Moved JsonPrefStore to use SequencedWorkerPool

chrome/browser/browser_process_impl.cc

Index: chrome/browser/browser_process_impl.cc
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index 2b33a94337a8886c4ff5f15ef947fec631bea4b2..bb16727045e0005be819e0727c58d9493d9662c7 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -131,7 +131,9 @@ using content::ChildProcessSecurityPolicy;
 using content::PluginService;
 using content::ResourceDispatcherHost;
 
-BrowserProcessImpl::BrowserProcessImpl(const CommandLine& command_line)
+BrowserProcessImpl::BrowserProcessImpl(
+    base::SequencedTaskRunner* local_state_task_runner,
+    const CommandLine& command_line)
     : created_metrics_service_(false),
       created_watchdog_thread_(false),
       created_browser_policy_connector_(false),
@@ -145,7 +147,8 @@ BrowserProcessImpl::BrowserProcessImpl(const CommandLine& command_line)
       checked_for_new_frames_(false),
       using_new_frames_(false),
       render_widget_snapshot_taker_(new RenderWidgetSnapshotTaker),
-      download_status_updater_(new DownloadStatusUpdater) {
+      download_status_updater_(new DownloadStatusUpdater),
+      local_state_task_runner_(local_state_task_runner) {

[Mattias Nissler (ping if slow), 2012/10/22 17:28:21]

This file uses well-known browser threads elsewhere, so it should probably
BrowserThread::GetBlockingPool() directly to obtain a task runner? Or is there a
good reason to pass this in?

[akalin, 2012/10/22 17:37:25]

To make sure that it uses the same one as the one in chrome_browser_main

[zel, 2012/10/24 02:20:11]

Yep.

   g_browser_process = this;
 
 #if defined(ENABLE_PRINTING)
@@ -712,10 +715,10 @@ void BrowserProcessImpl::CreateLocalState() {
   created_local_state_ = true;
 
   FilePath local_state_path;
-  PathService::Get(chrome::FILE_LOCAL_STATE, &local_state_path);
+  CHECK(PathService::Get(chrome::FILE_LOCAL_STATE, &local_state_path));
   local_state_.reset(
       PrefService::CreatePrefService(local_state_path, policy_service(), NULL,
-                                     false));
+                                     false, local_state_task_runner_));
 
   // Initialize the prefs of the local state.
   chrome::RegisterLocalState(local_state_.get());