Add network setup code to BlimpClientContextImpl

blimp/client/core/blimp_client_context_impl.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "blimp/client/core/blimp_client_context_impl.h"
 
+#include "base/bind.h"
 #include "base/memory/ptr_util.h"
+#include "base/message_loop/message_loop.h"
 #include "base/supports_user_data.h"
+#include "base/threading/sequenced_task_runner_handle.h"
 #include "blimp/client/core/contents/blimp_contents_impl.h"
+#include "blimp/client/core/session/cross_thread_network_event_observer.h"
 #include "blimp/client/public/blimp_client_context_delegate.h"
 
 #if defined(OS_ANDROID)
 #include "blimp/client/core/android/blimp_client_context_impl_android.h"
 #endif  // OS_ANDROID
 
 namespace blimp {
 namespace client {
 
 namespace {
 
 #if defined(OS_ANDROID)
 const char kBlimpClientContextImplAndroidKey[] =
     "blimp_client_context_impl_android";
 #endif  // OS_ANDROID
 }
 
 // This function is declared in //blimp/client/public/blimp_client_context.h,
 // and either this function or the one in
 // //blimp/client/core/dummy_blimp_client_context.cc should be linked in to
 // any binary using BlimpClientContext::Create.
 // static
-BlimpClientContext* BlimpClientContext::Create() {
-  return new BlimpClientContextImpl();
+BlimpClientContext* BlimpClientContext::Create(
+    scoped_refptr<base::SingleThreadTaskRunner> io_thread_task_runner) {
+  return new BlimpClientContextImpl(io_thread_task_runner);
 }
 
-BlimpClientContextImpl::BlimpClientContextImpl() : BlimpClientContext() {}
+BlimpClientContextImpl::BlimpClientContextImpl(
+    scoped_refptr<base::SingleThreadTaskRunner> io_thread_task_runner)
+    : BlimpClientContext(),
+      io_thread_task_runner_(io_thread_task_runner),
+      weak_factory_(this) {
+  blimp_connection_statistics_ = new BlimpConnectionStatistics();
+  net_components_.reset(new ClientNetworkComponents(
+      base::WrapUnique(new CrossThreadNetworkEventObserver(
+          weak_factory_.GetWeakPtr(), base::SequencedTaskRunnerHandle::Get())),
+      base::WrapUnique(blimp_connection_statistics_)));
 
-BlimpClientContextImpl::~BlimpClientContextImpl() {}
+  RegisterFeatures();

[Kevin M, 2016/07/29 19:23:17]

* This seems like a misnomer, since this is really just setting up the
ThreadPipeManager?
* IMO this should just be inlined.

[nyquist, 2016/07/29 22:26:57]

True. How about this; I'll move the threadpipemanager out to the constructor,
and then when the features come in later, we can add RegisterFeatures if it
fits?

+
+  // Initialize must only be posted after the RegisterFeature calls have
+  // completed.
+  io_thread_task_runner_->PostTask(
+      FROM_HERE, base::Bind(&ClientNetworkComponents::Initialize,
+                            base::Unretained(net_components_.get())));
+}
+
+BlimpClientContextImpl::~BlimpClientContextImpl() {
+  io_thread_task_runner_->DeleteSoon(FROM_HERE, net_components_.release());
+}
+
+void BlimpClientContextImpl::RegisterFeatures() {
+  thread_pipe_manager_ = base::WrapUnique(new ThreadPipeManager(
+      io_thread_task_runner_, net_components_->GetBrowserConnectionHandler()));
+}
 
 #if defined(OS_ANDROID)
 
 base::android::ScopedJavaLocalRef<jobject>
 BlimpClientContextImpl::GetJavaObject() {
   return GetBlimpClientContextImplAndroid()->GetJavaObject();
 }
 
 BlimpClientContextImplAndroid*
 BlimpClientContextImpl::GetBlimpClientContextImplAndroid() {
@@ skipping 15 matching lines @@
   delegate_ = delegate;
 }
 
 std::unique_ptr<BlimpContents> BlimpClientContextImpl::CreateBlimpContents() {
   std::unique_ptr<BlimpContents> blimp_contents =
       base::WrapUnique(new BlimpContentsImpl);
   delegate_->AttachBlimpContentsHelpers(blimp_contents.get());
   return blimp_contents;
 }
 
+void BlimpClientContextImpl::OnConnected() {}
+
+void BlimpClientContextImpl::OnDisconnected(int result) {}
+
 }  // namespace client
 }  // namespace blimp