summaryrefslogtreecommitdiff
path: root/src/components/application_manager/rpc_plugins/rc_rpc_plugin/include
diff options
context:
space:
mode:
authorIra Lytvynenko (GitHub) <ILytvynenko@luxoft.com>2018-06-28 16:35:09 +0300
committerIra Lytvynenko (GitHub) <ILytvynenko@luxoft.com>2018-07-16 16:37:44 +0300
commitca54bd15e1c4523bddcdc922f1857b3a03f01bd7 (patch)
tree2a826558c358f36e8a18ddae716536f5d0f5605f /src/components/application_manager/rpc_plugins/rc_rpc_plugin/include
parentb722d26f0ffc75402844ff68ba1d7b9055cf5e82 (diff)
downloadsdl_core-ca54bd15e1c4523bddcdc922f1857b3a03f01bd7.tar.gz
Fix logic according to review comments
In case of RC app registration OnRCStatus notification is sent only to the newly registered app
Diffstat (limited to 'src/components/application_manager/rpc_plugins/rc_rpc_plugin/include')
-rw-r--r--src/components/application_manager/rpc_plugins/rc_rpc_plugin/include/rc_rpc_plugin/resource_allocation_manager.h8
-rw-r--r--src/components/application_manager/rpc_plugins/rc_rpc_plugin/include/rc_rpc_plugin/resource_allocation_manager_impl.h4
2 files changed, 10 insertions, 2 deletions
diff --git a/src/components/application_manager/rpc_plugins/rc_rpc_plugin/include/rc_rpc_plugin/resource_allocation_manager.h b/src/components/application_manager/rpc_plugins/rc_rpc_plugin/include/rc_rpc_plugin/resource_allocation_manager.h
index 5140d10722..20bd438494 100644
--- a/src/components/application_manager/rpc_plugins/rc_rpc_plugin/include/rc_rpc_plugin/resource_allocation_manager.h
+++ b/src/components/application_manager/rpc_plugins/rc_rpc_plugin/include/rc_rpc_plugin/resource_allocation_manager.h
@@ -160,9 +160,15 @@ class ResourceAllocationManager {
/**
* @brief Create and send OnRCStatusNotification to mobile and HMI
* @param event trigger for notification sending
+ * @param application - app that should receive notification
+ * in case of registration; in cases of RC enabling/disabling
+ * or module allocation - application is just empty shared ptr,
+ * because in these cases all registered RC apps should
+ * receive a notification
*/
virtual void SendOnRCStatusNotifications(
- NotificationTrigger::eType event) = 0;
+ NotificationTrigger::eType event,
+ application_manager::ApplicationSharedPtr application) = 0;
virtual bool is_rc_enabled() const = 0;
diff --git a/src/components/application_manager/rpc_plugins/rc_rpc_plugin/include/rc_rpc_plugin/resource_allocation_manager_impl.h b/src/components/application_manager/rpc_plugins/rc_rpc_plugin/include/rc_rpc_plugin/resource_allocation_manager_impl.h
index 368c5fa785..d6124a026b 100644
--- a/src/components/application_manager/rpc_plugins/rc_rpc_plugin/include/rc_rpc_plugin/resource_allocation_manager_impl.h
+++ b/src/components/application_manager/rpc_plugins/rc_rpc_plugin/include/rc_rpc_plugin/resource_allocation_manager_impl.h
@@ -118,7 +118,9 @@ class ResourceAllocationManagerImpl : public ResourceAllocationManager {
RCAppExtensionPtr GetApplicationExtention(
application_manager::ApplicationSharedPtr application) FINAL;
- void SendOnRCStatusNotifications(NotificationTrigger::eType event) FINAL;
+ void SendOnRCStatusNotifications(
+ NotificationTrigger::eType event,
+ application_manager::ApplicationSharedPtr application) FINAL;
bool is_rc_enabled() const FINAL;