diff options
author | Ira Lytvynenko (GitHub) <ILytvynenko@luxoft.com> | 2018-06-28 16:35:09 +0300 |
---|---|---|
committer | Ira Lytvynenko (GitHub) <ILytvynenko@luxoft.com> | 2018-07-16 16:37:44 +0300 |
commit | ca54bd15e1c4523bddcdc922f1857b3a03f01bd7 (patch) | |
tree | 2a826558c358f36e8a18ddae716536f5d0f5605f /src/components/application_manager/rpc_plugins/rc_rpc_plugin/include | |
parent | b722d26f0ffc75402844ff68ba1d7b9055cf5e82 (diff) | |
download | sdl_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')
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; |