Migrate Linux Blimp client to use BlimpClientContext

blimp/client/app/linux/blimp_main.cc

Index: blimp/client/app/linux/blimp_main.cc
diff --git a/blimp/client/app/linux/blimp_main.cc b/blimp/client/app/linux/blimp_main.cc
index 37c7c957fab2e1b06fb4030ec406acce22ff6490..2bb5fa0419da04affc54ed5e292e23dd96682647 100644
--- a/blimp/client/app/linux/blimp_main.cc
+++ b/blimp/client/app/linux/blimp_main.cc
@@ -1,4 +1,4 @@
-// Copyright 2015 The Chromium Authors. All rights reserved.
+// 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.
 
@@ -10,18 +10,26 @@
 #include "base/threading/thread.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "blimp/client/app/blimp_startup.h"
-#include "blimp/client/app/linux/blimp_client_session_linux.h"
-#include "blimp/client/core/contents/navigation_feature.h"
-#include "blimp/client/core/contents/tab_control_feature.h"
-#include "blimp/client/core/session/assignment_source.h"
+#include "blimp/client/app/linux/blimp_client_context_delegate_linux.h"
+#include "blimp/client/app/linux/blimp_display_manager.h"
+#include "blimp/client/app/linux/blimp_display_manager_delegate_main.h"
+#include "blimp/client/public/blimp_client_context.h"
+#include "blimp/client/public/contents/blimp_contents_view.h"
+#include "blimp/client/public/contents/blimp_navigation_controller.h"
+#include "blimp/client/support/compositor/blimp_embedder_compositor.h"
+#include "blimp/client/support/compositor/compositor_dependencies_impl.h"
+#include "blimp/common/get_client_auth_token.h"
+#include "ui/events/platform/platform_event_source.h"
 #include "ui/gfx/x/x11_connection.h"
 
 namespace {
-const char kDummyLoginToken[] = "";
 const char kDefaultUrl[] = "https://www.google.com";
-const int kDummyTabId = 0;
+const int kWindowWidth = 800;
+const int kWindowHeight = 600;
 }
 
+std::unique_ptr<base::Thread> CreateIOThread();
+
 int main(int argc, const char**argv) {
   base::AtExitManager at_exit;
   base::CommandLine::Init(argc, argv);
@@ -31,15 +39,45 @@ int main(int argc, const char**argv) {
   blimp::client::InitializeLogging();
   blimp::client::InitializeMainMessageLoop();
 
-  blimp::client::BlimpClientSessionLinux session;
-  session.GetTabControlFeature()->CreateTab(kDummyTabId);
-  session.Connect(kDummyLoginToken);
+  std::unique_ptr<base::Thread> io_thread = CreateIOThread();
+
+  blimp::client::CompositorDependencies* compositor_dependencies =
+      new blimp::client::CompositorDependenciesImpl();
+
+  std::unique_ptr<blimp::client::BlimpClientContext> context =
+      base::WrapUnique<blimp::client::BlimpClientContext>(
+          blimp::client::BlimpClientContext::Create(
+              io_thread->task_runner(), io_thread->task_runner(),
+              base::WrapUnique(compositor_dependencies)));
+
+  context->SetDelegate(new blimp::client::BlimpClientContextDelegateLinux());

[David Trainor- moved to gerrit, 2016/09/26 20:54:26]

This will leak the delegate.  It isn't owned by BlimpClientContext.  Use a
std::unique_ptr and pass .get().  Make sure the destruction order is correct
(destroy BlimpClientContext before BlimpClientContextDelegate).

[steimel, 2016/09/27 17:35:01]

Done.

+
+  context->Connect();
+
+  std::unique_ptr<blimp::client::BlimpContents> contents =
+      context->CreateBlimpContents(nullptr);
 
   // If there is a non-switch argument to the command line, load that url.
   base::CommandLine::StringVector args =
       base::CommandLine::ForCurrentProcess()->GetArgs();
   std::string url = args.size() > 0 ? args[0] : kDefaultUrl;
-  session.GetNavigationFeature()->NavigateToUrlText(kDummyTabId, url);
+  contents->GetNavigationController().LoadURL(GURL(url));
+
+  std::unique_ptr<ui::PlatformEventSource> event_source =
+      ui::PlatformEventSource::CreateDefault();
 
+  blimp::client::BlimpDisplayManager display_manager(
+      gfx::Size(kWindowWidth, kWindowHeight),
+      new blimp::client::BlimpDisplayManagerDelegateMain(),
+      compositor_dependencies, std::move(contents));
   base::RunLoop().Run();
 }
+
+std::unique_ptr<base::Thread> CreateIOThread() {

[David Trainor- moved to gerrit, 2016/09/26 20:54:26]

Put this up in the anonymous namespace block above or inline it if it's only
called once.

[steimel, 2016/09/27 17:35:01]

Done.

+  std::unique_ptr<base::Thread> io_thread =
+      base::MakeUnique<base::Thread>("BlimpIOThread");
+  base::Thread::Options options;
+  options.message_loop_type = base::MessageLoop::TYPE_IO;
+  io_thread->StartWithOptions(options);
+  return io_thread;
+}