Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1299)

Unified Diff: blimp/client/core/contents/tab_control_feature.cc

Issue 2325893002: [blimp] Add support for having multiple tabs (Closed)
Patch Set: Add tablet and non-blimp support Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « blimp/client/core/contents/navigation_feature.cc ('k') | blimp/engine/session/blimp_engine_session.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: blimp/client/core/contents/tab_control_feature.cc
diff --git a/blimp/client/core/contents/tab_control_feature.cc b/blimp/client/core/contents/tab_control_feature.cc
index 3559b37f00b2082f3a10f55f7f83e839b60f7ff3..bf21af61f6645239e54fb26c1ccf2bfe30f15b74 100644
--- a/blimp/client/core/contents/tab_control_feature.cc
+++ b/blimp/client/core/contents/tab_control_feature.cc
@@ -43,6 +43,7 @@ void TabControlFeature::CreateTab(int tab_id) {
TabControlMessage* tab_control;
std::unique_ptr<BlimpMessage> message = CreateBlimpMessage(&tab_control);
tab_control->mutable_create_tab();
+ message->set_target_tab_id(tab_id);
outgoing_message_processor_->ProcessMessage(std::move(message),
net::CompletionCallback());
}
@@ -51,6 +52,7 @@ void TabControlFeature::CloseTab(int tab_id) {
TabControlMessage* tab_control;
std::unique_ptr<BlimpMessage> message = CreateBlimpMessage(&tab_control);
tab_control->mutable_close_tab();
+ message->set_target_tab_id(tab_id);
outgoing_message_processor_->ProcessMessage(std::move(message),
net::CompletionCallback());
}
« no previous file with comments | « blimp/client/core/contents/navigation_feature.cc ('k') | blimp/engine/session/blimp_engine_session.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698