Move the thread hop for UMA user actions from content:: to base::.

chromecast/browser/cast_browser_main_parts.cc

 // Copyright 2014 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 "chromecast/browser/cast_browser_main_parts.h"
 
 #include <stddef.h>
 #include <string.h>
 
 #include <string>
@@ skipping 425 matching lines @@
           url_request_context_factory_->GetSystemGetter(),
           video_plane_controller_.get()));
   cast_browser_process_->cast_service()->Initialize();
 
 #if !defined(OS_ANDROID)
   media_resource_tracker()->InitializeMediaLib();
 #endif
   ::media::InitializeMediaLibrary();
 
   // Initializing metrics service and network delegates must happen after cast
-  // service is intialized because CastMetricsServiceClient and
+  // service is initialized because CastMetricsServiceClient and
   // CastNetworkDelegate may use components initialized by cast service.
+  // Set the thread on which to execute RecordAction.

[Ilya Sherman, 2016/04/15 22:16:05]

nit: This comment seems rather redundant with the code; I'd omit it.

[beaudoin, 2016/04/18 14:29:52]

Done.

+  base::SetRecordActionTaskRunner(
+      BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI));
   cast_browser_process_->metrics_service_client()
       ->Initialize(cast_browser_process_->cast_service());
   url_request_context_factory_->InitializeNetworkDelegates();
 
   cast_browser_process_->cast_service()->Start();
 }
 
 bool CastBrowserMainParts::MainMessageLoopRun(int* result_code) {
 #if defined(OS_ANDROID)
   // Android does not use native main MessageLoop.
@@ skipping 40 matching lines @@
 void CastBrowserMainParts::PostDestroyThreads() {
 #if !defined(OS_ANDROID)
   media_resource_tracker_->FinalizeAndDestroy();
   media_resource_tracker_ = nullptr;
   media_pipeline_backend_manager_.reset();
 #endif
 }
 
 }  // namespace shell
 }  // namespace chromecast