summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRomanReznichenkoLuxoft <85177915+RomanReznichenkoLuxoft@users.noreply.github.com>2022-01-11 17:13:23 +0200
committerGitHub <noreply@github.com>2022-01-11 10:13:23 -0500
commit7aecb63a17bdc9f38d75cb381143b7eacd9d85fc (patch)
treefb2acd0157f24f385ffbec9079a06e2e1ead1ba2
parenteb14596e25cf6f49dd2ab21d39f32d3453615faa (diff)
downloadsdl_core-7aecb63a17bdc9f38d75cb381143b7eacd9d85fc.tar.gz
changed the way you add soft buttons in the subscription method (#3839)
-rw-r--r--src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/show_constant_tbt_response.cc2
-rw-r--r--src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/update_turn_list_response.cc2
-rw-r--r--src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/simple_response_commands_test.cc1
-rw-r--r--src/components/application_manager/src/application_impl.cc3
4 files changed, 2 insertions, 6 deletions
diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/show_constant_tbt_response.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/show_constant_tbt_response.cc
index a2362a74bc..7d546079e8 100644
--- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/show_constant_tbt_response.cc
+++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/show_constant_tbt_response.cc
@@ -60,8 +60,6 @@ ShowConstantTBTResponse::~ShowConstantTBTResponse() {}
void ShowConstantTBTResponse::Run() {
SDL_LOG_AUTO_TRACE();
- application_manager_.UnsubscribeAppFromSoftButtons(message_);
-
rpc_service_.SendMessageToMobile(message_);
}
diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/update_turn_list_response.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/update_turn_list_response.cc
index bf2658128d..bedf91b45c 100644
--- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/update_turn_list_response.cc
+++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/update_turn_list_response.cc
@@ -59,8 +59,6 @@ UpdateTurnListResponse::~UpdateTurnListResponse() {}
void UpdateTurnListResponse::Run() {
SDL_LOG_AUTO_TRACE();
- application_manager_.UnsubscribeAppFromSoftButtons(message_);
-
rpc_service_.SendMessageToMobile(message_);
}
diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/simple_response_commands_test.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/simple_response_commands_test.cc
index d091fdc8c5..a6bdf2a874 100644
--- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/simple_response_commands_test.cc
+++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/simple_response_commands_test.cc
@@ -149,7 +149,6 @@ TYPED_TEST(MobileResponseWithUnsubscribeCommandsTest,
std::shared_ptr<typename TestFixture::UnsubscribeCommand> command =
this->template CreateCommand<typename TestFixture::UnsubscribeCommand>();
- EXPECT_CALL(this->app_mngr_, UnsubscribeAppFromSoftButtons(_));
EXPECT_CALL(this->mock_rpc_service_, SendMessageToMobile(NotNull(), _));
command->Init();
diff --git a/src/components/application_manager/src/application_impl.cc b/src/components/application_manager/src/application_impl.cc
index 42afd93459..9b6e4457d0 100644
--- a/src/components/application_manager/src/application_impl.cc
+++ b/src/components/application_manager/src/application_impl.cc
@@ -1165,7 +1165,8 @@ void ApplicationImpl::SubscribeToSoftButtons(
return;
}
- WindowSoftButtons new_window_soft_buttons = *subscribed_window_buttons;
+ WindowSoftButtons new_window_soft_buttons;
+ new_window_soft_buttons.first = window_id;
new_window_soft_buttons.second.insert(window_softbuttons.second.begin(),
window_softbuttons.second.end());
command_soft_buttons.erase(subscribed_window_buttons);