Create and integrate a metrics service client into Blimp engine.

blimp/engine/session/blimp_engine_session.cc

 // Copyright 2015 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/engine/session/blimp_engine_session.h"
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/memory/ptr_util.h"
@@ skipping 242 matching lines @@
 
   window_tree_client_.reset(
       new BlimpWindowTreeClient(window_tree_host_->window()));
 
   window_tree_host_->GetInputMethod()->AddObserver(this);
 
   window_tree_host_->SetBounds(gfx::Rect(screen_->GetPrimaryDisplay().size()));
 
   RegisterFeatures();
 
+  browser_context_->Initialize();

[Wez, 2016/04/26 01:38:19]

nit: Add a one-line comment to explain what this is for?

Seems strange that this only became required when we started doing metrics...

[Jess, 2016/04/26 21:24:11]

Done.

+
   // Initialize must only be posted after the RegisterFeature calls have
   // completed.
   content::BrowserThread::PostTask(
       content::BrowserThread::IO, FROM_HERE,
       base::Bind(&EngineNetworkComponents::Initialize,
                  base::Unretained(net_components_.get()),
                  engine_config_->client_token()));
 }
 
+void BlimpEngineSession::Finalize() {
+  browser_context_->Finalize();

[Wez, 2016/04/26 01:38:19]

Is it important that this happen before we enter the BlimpEngineSession dtor, or
could we move it there and avoid the explicit Finalize() API?

[Jess, 2016/04/26 21:24:11]

Done.

Also moved call to browser_context_->Initialize() into ctor.

+}
+
 void BlimpEngineSession::RegisterFeatures() {
   thread_pipe_manager_.reset(new ThreadPipeManager(
       content::BrowserThread::GetMessageLoopProxyForThread(
           content::BrowserThread::IO),
       content::BrowserThread::GetMessageLoopProxyForThread(
           content::BrowserThread::UI),
       net_components_->GetBrowserConnectionHandler()));
 
   // Register features' message senders and receivers.
   tab_control_message_sender_ =
@@ skipping 351 matching lines @@
 
   aura::Window* parent = window_tree_host_->window();
   aura::Window* content = web_contents_->GetNativeView();
   if (!parent->Contains(content))
     parent->AddChild(content);
   content->Show();
 }
 
 }  // namespace engine
 }  // namespace blimp