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

Unified Diff: blimp/client/core/blimp_client_context_impl.cc

Issue 2255533002: Add TabControlFeature to BlimpClientContextImpl (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: sync to head Created 4 years, 4 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
Index: blimp/client/core/blimp_client_context_impl.cc
diff --git a/blimp/client/core/blimp_client_context_impl.cc b/blimp/client/core/blimp_client_context_impl.cc
index 46ba505076f7823a8e839e48d8fbc31b10051ae3..9fe0c91afeafa90f4fb7306c2c392a2df3ca0805 100644
--- a/blimp/client/core/blimp_client_context_impl.cc
+++ b/blimp/client/core/blimp_client_context_impl.cc
@@ -10,6 +10,7 @@
#include "base/threading/sequenced_task_runner_handle.h"
#include "blimp/client/core/contents/blimp_contents_impl.h"
#include "blimp/client/core/contents/blimp_contents_manager.h"
+#include "blimp/client/core/contents/tab_control_feature.h"
#include "blimp/client/core/session/cross_thread_network_event_observer.h"
#include "blimp/client/public/blimp_client_context_delegate.h"
@@ -48,7 +49,9 @@ BlimpClientContextImpl::BlimpClientContextImpl(
: BlimpClientContext(),
io_thread_task_runner_(io_thread_task_runner),
file_thread_task_runner_(file_thread_task_runner),
- blimp_contents_manager_(new BlimpContentsManager),
+ tab_control_feature_(new TabControlFeature),
+ blimp_contents_manager_(new BlimpContentsManager(
+ tab_control_feature_.get())),
weak_factory_(this) {
net_components_.reset(new ClientNetworkComponents(
base::MakeUnique<CrossThreadNetworkEventObserver>(
@@ -59,6 +62,8 @@ BlimpClientContextImpl::BlimpClientContextImpl(
thread_pipe_manager_ = base::MakeUnique<ThreadPipeManager>(
io_thread_task_runner_, net_components_->GetBrowserConnectionHandler());
+ RegisterFeatures();
+
// Initialize must only be posted after the calls features have been
// registered.
io_thread_task_runner_->PostTask(
@@ -113,6 +118,10 @@ void BlimpClientContextImpl::OnConnected() {}
void BlimpClientContextImpl::OnDisconnected(int result) {}
+TabControlFeature* BlimpClientContextImpl::GetTabControlFeature() const {
+ return tab_control_feature_.get();
+}
+
GURL BlimpClientContextImpl::GetAssignerURL() {
return GURL(kDefaultAssignerUrl);
}
@@ -137,5 +146,12 @@ void BlimpClientContextImpl::ConnectWithAssignment(
base::Unretained(net_components_.get()), assignment));
}
+void BlimpClientContextImpl::RegisterFeatures() {
+ // Register features' message senders and receivers.
+ tab_control_feature_->set_outgoing_message_processor(
+ thread_pipe_manager_->RegisterFeature(BlimpMessage::kTabControl,
+ tab_control_feature_.get()));
+}
+
} // namespace client
} // namespace blimp
« no previous file with comments | « blimp/client/core/blimp_client_context_impl.h ('k') | blimp/client/core/blimp_client_context_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698