Create and integrate a metrics service client into Blimp engine.

blimp/engine/common/blimp_browser_context.h

 // 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.
 
 #ifndef BLIMP_ENGINE_COMMON_BLIMP_BROWSER_CONTEXT_H_
 #define BLIMP_ENGINE_COMMON_BLIMP_BROWSER_CONTEXT_H_
 
 #include <memory>
 
 #include "base/files/file_path.h"
 #include "base/macros.h"
+#include "blimp/engine/app/blimp_metrics_service_client.h"
 #include "blimp/engine/app/blimp_system_url_request_context_getter.h"
 #include "blimp/engine/app/blimp_url_request_context_getter.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/content_browser_client.h"
 #include "content/public/browser/resource_context.h"
 #include "net/url_request/url_request_job_factory.h"
 
 namespace net {
 class NetLog;
 }
 
 namespace blimp {
 namespace engine {
 
 class BlimpResourceContext;
 class PermissionManager;
 
 class BlimpBrowserContext : public content::BrowserContext {
  public:
   // Caller owns |net_log| and ensures it out-lives this browser context.
   BlimpBrowserContext(bool off_the_record, net::NetLog* net_log);
   ~BlimpBrowserContext() override;
 
+  // These map to BrowserMainParts::Pre/PostMainMessageLoopRun respectively.

[Wez, 2016/05/01 00:12:59]

Can you be more specific? Do you mean "These call through to..." or "These are
called by..."?

+  void Initialize();
+  void Finalize();
+
   // content::BrowserContext implementation.
   std::unique_ptr<content::ZoomLevelDelegate> CreateZoomLevelDelegate(
       const base::FilePath& partition_path) override;
   base::FilePath GetPath() const override;
   bool IsOffTheRecord() const override;
   content::ResourceContext* GetResourceContext() override;
   content::DownloadManagerDelegate* GetDownloadManagerDelegate() override;
   content::BrowserPluginGuestManager* GetGuestManager() override;
   storage::SpecialStoragePolicy* GetSpecialStoragePolicy() override;
   content::PushMessagingService* GetPushMessagingService() override;
@@ skipping 15 matching lines @@
 
   // Provides a URLRequestContextGetter for system requests (e.g. metrics
   // uploads).
   net::URLRequestContextGetter* GetSystemRequestContextGetter();
 
  private:
   // Performs initialization of the BlimpBrowserContext while IO is still
   // allowed on the current thread.
   void InitWhileIOAllowed();
 
+  // Used in metrics initialization to get a PrefService to store logs
+  // temporarily.

[Wez, 2016/05/01 00:12:59]

This comment is pretty confusing; why does metrics initialization need to store
temporary logs?

+  std::unique_ptr<PrefService> GetPrefService();
+
   std::unique_ptr<BlimpResourceContext> resource_context_;
   scoped_refptr<BlimpSystemURLRequestContextGetter> system_context_getter_;
   bool ignore_certificate_errors_;
   std::unique_ptr<content::PermissionManager> permission_manager_;
   bool off_the_record_;
   net::NetLog* net_log_;
   base::FilePath path_;
 
   DISALLOW_COPY_AND_ASSIGN(BlimpBrowserContext);
 };
 
 }  // namespace engine
 }  // namespace blimp
 
 #endif  // BLIMP_ENGINE_COMMON_BLIMP_BROWSER_CONTEXT_H_