Moved JsonPrefStore to use SequencedWorkerPool

chrome/browser/prefs/pref_service_mock_builder.cc

Index: chrome/browser/prefs/pref_service_mock_builder.cc
diff --git a/chrome/browser/prefs/pref_service_mock_builder.cc b/chrome/browser/prefs/pref_service_mock_builder.cc
index edfe6960265ccde184651915367fae9d90f9a5c8..22b3f20b6f9c96790508e70da9f4eccfb74824fa 100644
--- a/chrome/browser/prefs/pref_service_mock_builder.cc
+++ b/chrome/browser/prefs/pref_service_mock_builder.cc
@@ -78,15 +78,16 @@ PrefServiceMockBuilder::WithCommandLine(CommandLine* command_line) {
 PrefServiceMockBuilder&
 PrefServiceMockBuilder::WithUserFilePrefs(const FilePath& prefs_file) {

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

hmmm...is there any way to remove this overload, and have everything use the one
below instead?

If there's too many places that call this, then maybe just add a TODO and have
it call the utility function from JsonPrefStore.

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

I've did that on 2 more places and the only one remaining is in chrome_frame
which I am scared to touch (not even sure if my CL is even working on those
tests yet). I added TODO comment in the header. 

   return WithUserFilePrefs(prefs_file,
-                           BrowserThread::GetMessageLoopProxyForThread(
-                               BrowserThread::FILE));
+                           BrowserThread::GetBlockingPool());
 }
 
 PrefServiceMockBuilder&
 PrefServiceMockBuilder::WithUserFilePrefs(
     const FilePath& prefs_file,
-    base::MessageLoopProxy* message_loop_proxy) {
-  user_prefs_ = new JsonPrefStore(prefs_file, message_loop_proxy);
+    base::SequencedWorkerPool* blocking_pool) {
+  user_prefs_ = JsonPrefStore::Create(
+      prefs_file,
+      JsonPrefStore::GetTaskRunnerForFile(prefs_file, blocking_pool));
   return *this;
 }