Blimp Settings framework on the c++ side

blimp/client/app/linux/blimp_main.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 <string>
 
 #include "base/at_exit.h"
 #include "base/command_line.h"
 #include "base/run_loop.h"
 #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_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_navigation_controller.h"
 #include "blimp/client/support/compositor/compositor_dependencies_impl.h"
 #include "ui/gfx/x/x11_connection.h"
 
 namespace {
 const char kDefaultUrl[] = "https://www.google.com";
 constexpr int kWindowWidth = 800;
 constexpr int kWindowHeight = 600;
 }
 
 int main(int argc, const char**argv) {

[David Trainor- moved to gerrit, 2016/10/26 01:14:55]

Should we call RegisterPrefs and the AppendCommandSwitches here?

[Menglin, 2016/10/26 01:46:22]

Done.

   base::AtExitManager at_exit;
   base::CommandLine::Init(argc, argv);
 
   CHECK(gfx::InitializeThreadedX11());
 
   blimp::client::InitializeLogging();
   blimp::client::InitializeMainMessageLoop();
 
   base::Thread io_thread("BlimpIOThread");
   base::Thread::Options options;
   options.message_loop_type = base::MessageLoop::TYPE_IO;
   io_thread.StartWithOptions(options);
 
   // Creating this using "new" and passing to context using "WrapUnique" as
   // opposed to "MakeUnique" because we'll need to pass the compositor
   // dependencies to the display manager as well.
   blimp::client::CompositorDependencies* compositor_dependencies =
       new blimp::client::CompositorDependenciesImpl();
   // Creating the context delegate before the context so that the context is
   // destroyed before the delegate.
   std::unique_ptr<blimp::client::BlimpClientContextDelegate> context_delegate =
       base::MakeUnique<blimp::client::BlimpClientContextDelegateLinux>();
-  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)));
+  auto context = base::WrapUnique<blimp::client::BlimpClientContext>(
+      blimp::client::BlimpClientContext::Create(
+          io_thread.task_runner(), io_thread.task_runner(),
+          base::WrapUnique(compositor_dependencies),
+          static_cast<blimp::client::BlimpClientContextDelegateLinux*>(
+              context_delegate.get())->pref_service()));
   context->SetDelegate(context_delegate.get());
 
   context->Connect();
 
   // 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;
   std::unique_ptr<blimp::client::BlimpContents> contents =
       context->CreateBlimpContents(nullptr);
   contents->GetNavigationController().LoadURL(GURL(url));
 
   std::unique_ptr<blimp::client::BlimpDisplayManagerDelegate>
       display_manager_delegate =
           base::MakeUnique<blimp::client::BlimpDisplayManagerDelegateMain>();
   blimp::client::BlimpDisplayManager display_manager(
       display_manager_delegate.get(), compositor_dependencies);
   display_manager.SetWindowSize(gfx::Size(kWindowWidth, kWindowHeight));
   display_manager.SetBlimpContents(std::move(contents));
 
   base::RunLoop().Run();
 }