diff options
author | Andrey Oleynik <aoleynik@luxoft.com> | 2017-08-21 13:54:39 +0300 |
---|---|---|
committer | Andrey Oleynik <aoleynik@luxoft.com> | 2017-08-21 16:02:16 +0300 |
commit | 08789ae20c7105cbac527c4b044fb2fff9284d07 (patch) | |
tree | b2fc088e85a38d390d22f5c32c35b9e8de254cba /src/components/application_manager | |
parent | 829b726b535638b33528fa3da5c593360300954d (diff) | |
download | sdl_core-08789ae20c7105cbac527c4b044fb2fff9284d07.tar.gz |
Removes unused interfaces from AccessRemote class
Diffstat (limited to 'src/components/application_manager')
6 files changed, 0 insertions, 65 deletions
diff --git a/src/components/application_manager/include/application_manager/core_service.h b/src/components/application_manager/include/application_manager/core_service.h index 29af7454b9..56d135efcb 100644 --- a/src/components/application_manager/include/application_manager/core_service.h +++ b/src/components/application_manager/include/application_manager/core_service.h @@ -89,16 +89,6 @@ class CoreService : public Service { const std::string& module) FINAL; /** - * Sets access to functional group which contains given RPC for application - * @param app_id id of application - * @param module type - * @param allowed true if driver has given access - */ - void SetAccess(const ApplicationId& app_id, - const std::string& module, - bool allowed) FINAL; - - /** * Checks if application has remote control functions * @param app application * @return true if application has remote control functions diff --git a/src/components/application_manager/include/application_manager/policies/policy_handler.h b/src/components/application_manager/include/application_manager/policies/policy_handler.h index bce7484fdf..f451c19813 100644 --- a/src/components/application_manager/include/application_manager/policies/policy_handler.h +++ b/src/components/application_manager/include/application_manager/policies/policy_handler.h @@ -163,18 +163,6 @@ class PolicyHandler : public PolicyHandlerInterface, bool CheckModule(const PTString& app_id, const PTString& module) OVERRIDE; /** - * Sets access to equipment of vehicle for application by RPC - * @param device_id unique identifier of device - * @param app_id policy id application - * @param module type - * @param allowed true if access is allowed - */ - void SetAccess(const PTString& device_id, - const PTString& app_id, - const PTString& module, - bool allowed) OVERRIDE; - - /** * @brief Notifies Remote apps about change in permissions * @param device_id Device on which app is running * @param application_id ID of app whose permissions are changed diff --git a/src/components/application_manager/include/application_manager/service.h b/src/components/application_manager/include/application_manager/service.h index a4eddc5447..be2570db3e 100644 --- a/src/components/application_manager/include/application_manager/service.h +++ b/src/components/application_manager/include/application_manager/service.h @@ -89,16 +89,6 @@ class Service { const std::string& module) = 0; /** - * Sets access to functional group which contains given RPC for application - * @param app_id id of application - * @param module type - * @param allowed true if driver has given access - */ - virtual void SetAccess(const ApplicationId& app_id, - const std::string& module, - bool allowed) = 0; - - /** * @brief Get pointer to application by application id * @param app_id application id * return pointer to application diff --git a/src/components/application_manager/src/core_service.cc b/src/components/application_manager/src/core_service.cc index 43a63b7d8d..4e41744e27 100644 --- a/src/components/application_manager/src/core_service.cc +++ b/src/components/application_manager/src/core_service.cc @@ -108,20 +108,6 @@ bool CoreService::CheckModule(const ApplicationId& app_id, return false; } -void CoreService::SetAccess(const ApplicationId& app_id, - const std::string& module, - bool allowed) { -#ifdef SDL_REMOTE_CONTROL - ApplicationSharedPtr app = GetApplication(app_id); - if (app) { - std::string device_handle = MessageHelper::GetDeviceMacAddressForHandle( - app->device(), application_manager_); - application_manager_.GetPolicyHandler().SetAccess( - device_handle, app->policy_app_id(), module, allowed); - } -#endif // SDL_REMOTE_CONTROL -} - bool CoreService::IsRemoteControlApplication(ApplicationSharedPtr app) const { #ifdef SDL_REMOTE_CONTROL return application_manager_.GetPolicyHandler().CheckHMIType( diff --git a/src/components/application_manager/src/policies/policy_handler.cc b/src/components/application_manager/src/policies/policy_handler.cc index 574f362e42..ae4fb033e9 100644 --- a/src/components/application_manager/src/policies/policy_handler.cc +++ b/src/components/application_manager/src/policies/policy_handler.cc @@ -2006,14 +2006,6 @@ bool PolicyHandler::CheckModule(const PTString& app_id, return policy_manager_->CheckModule(app_id, module); } -void PolicyHandler::SetAccess(const PTString& device_id, - const PTString& app_id, - const PTString& module, - bool allowed) { - POLICY_LIB_CHECK_VOID(); - policy_manager_->SetAccess(device_id, app_id, module, allowed); -} - void PolicyHandler::OnRemoteAppPermissionsChanged( const std::string& device_id, const std::string& application_id) { POLICY_LIB_CHECK_VOID(); diff --git a/src/components/application_manager/test/rc_policy_handler_test.cc b/src/components/application_manager/test/rc_policy_handler_test.cc index 760500af95..efc3f5d46d 100644 --- a/src/components/application_manager/test/rc_policy_handler_test.cc +++ b/src/components/application_manager/test/rc_policy_handler_test.cc @@ -288,17 +288,6 @@ TEST_F(RCPolicyHandlerTest, CheckAccess_ValidParams_SUCCESS) { policy_handler_.CheckAccess(kDeviceId_, kPolicyAppId_, module)); } -TEST_F(RCPolicyHandlerTest, SetAccess_ValidParams_SUCCESS) { - EnablePolicyAndPolicyManagerMock(); - - const PTString module("module"); - const bool allowed(true); - EXPECT_CALL(*mock_policy_manager_, - SetAccess(kDeviceId_, kPolicyAppId_, module, allowed)); - - policy_handler_.SetAccess(kDeviceId_, kPolicyAppId_, module, allowed); -} - TEST_F(RCPolicyHandlerTest, CheckModule_SUCCESS) { EnablePolicyAndPolicyManagerMock(); |