Moved JsonPrefStore to use SequencedWorkerPool

chrome/browser/browser_process_impl.h

 // Copyright (c) 2012 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.
 
 // When each service is created, we set a flag indicating this. At this point,
 // the service initialization could fail or succeed. This allows us to remember
 // if we tried to create a service, and not try creating it over and over if
 // the creation failed.
 
 #ifndef CHROME_BROWSER_BROWSER_PROCESS_IMPL_H_
@@ skipping 24 matching lines @@
 namespace policy {
 class BrowserPolicyConnector;
 class PolicyService;
 };
 
 // Real implementation of BrowserProcess that creates and returns the services.
 class BrowserProcessImpl : public BrowserProcess,
                            public base::NonThreadSafe,
                            public content::NotificationObserver {
  public:
-  explicit BrowserProcessImpl(const CommandLine& command_line);
+  explicit BrowserProcessImpl(

[akalin, 2012/10/18 23:52:24]

remove explicit

[zel, 2012/10/19 01:20:32]

Done.

+      base::SequencedTaskRunner* local_state_task_runner,

[akalin, 2012/10/18 23:52:24]

fwd-declare SequencedTaskRunner

[akalin, 2012/10/18 23:52:24]

add comment saying that local_state_task_runner must be a shutdown-blocking task
runner.

[zel, 2012/10/19 01:20:32]

Done.

[zel, 2012/10/19 01:20:32]

Done.

+      const CommandLine& command_line);
   virtual ~BrowserProcessImpl();
 
   // Called before the browser threads are created.
   void PreCreateThreads();
 
   // Called after the threads have been created but before the message loops
   // starts running. Allows the browser process to do any initialization that
   // requires all threads running.
   void PreMainMessageLoopRun();
 
@@ skipping 162 matching lines @@
   // nothing in the constructor so we don't have to worry about lazy init.
   scoped_ptr<RenderWidgetSnapshotTaker> render_widget_snapshot_taker_;
 
   // Download status updates (like a changing application icon on dock/taskbar)
   // are global per-application. DownloadStatusUpdater does no work in the ctor
   // so we don't have to worry about lazy initialization.
   scoped_ptr<DownloadStatusUpdater> download_status_updater_;
 
   scoped_refptr<DownloadRequestLimiter> download_request_limiter_;
 
+  // Sequenced task runner for local state related I/O tasks.
+  scoped_refptr<base::SequencedTaskRunner> local_state_task_runner_;

[akalin, 2012/10/18 23:52:24]

const scoped_refptr<...> ...

[zel, 2012/10/19 01:20:32]

Done.

+
   // Ensures that the observers of plugin/print disable/enable state
   // notifications are properly added and removed.
   PrefChangeRegistrar pref_change_registrar_;
 
   // Lives here so can safely log events on shutdown.
   scoped_ptr<ChromeNetLog> net_log_;
 
   // Ordered before resource_dispatcher_host_delegate_ due to destruction
   // ordering.
   scoped_ptr<prerender::PrerenderTracker> prerender_tracker_;
@@ skipping 26 matching lines @@
   scoped_refptr<PluginsResourceService> plugins_resource_service_;
 #endif
   // TODO(eroman): Remove this when done debugging 113031. This tracks
   // the callstack which released the final module reference count.
   base::debug::StackTrace release_last_reference_callstack_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserProcessImpl);
 };
 
 #endif  // CHROME_BROWSER_BROWSER_PROCESS_IMPL_H_