summaryrefslogtreecommitdiff
path: root/src/components/application_manager
diff options
context:
space:
mode:
authorAndrey Oleynik <aoleynik@luxoft.com>2017-08-21 13:54:39 +0300
committerAndrey Oleynik <aoleynik@luxoft.com>2017-08-21 16:02:16 +0300
commit08789ae20c7105cbac527c4b044fb2fff9284d07 (patch)
treeb2fc088e85a38d390d22f5c32c35b9e8de254cba /src/components/application_manager
parent829b726b535638b33528fa3da5c593360300954d (diff)
downloadsdl_core-08789ae20c7105cbac527c4b044fb2fff9284d07.tar.gz
Removes unused interfaces from AccessRemote class
Diffstat (limited to 'src/components/application_manager')
-rw-r--r--src/components/application_manager/include/application_manager/core_service.h10
-rw-r--r--src/components/application_manager/include/application_manager/policies/policy_handler.h12
-rw-r--r--src/components/application_manager/include/application_manager/service.h10
-rw-r--r--src/components/application_manager/src/core_service.cc14
-rw-r--r--src/components/application_manager/src/policies/policy_handler.cc8
-rw-r--r--src/components/application_manager/test/rc_policy_handler_test.cc11
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();