[Sync] Switch Extension Settings DTC to use ProfilesyncComponentsFactory

chrome/browser/extensions/settings/settings_apitest.cc

Index: chrome/browser/extensions/settings/settings_apitest.cc
diff --git a/chrome/browser/extensions/settings/settings_apitest.cc b/chrome/browser/extensions/settings/settings_apitest.cc
index 093930ee818f899c1f1ad08261aa2c3c64be0c37..aef13f8142738cbca43260555a6c6487dc18f07a 100644
--- a/chrome/browser/extensions/settings/settings_apitest.cc
+++ b/chrome/browser/extensions/settings/settings_apitest.cc
@@ -69,26 +69,26 @@ class ExtensionSettingsApiTest : public ExtensionApiTest {
   }
 
   void InitSync(SyncChangeProcessor* sync_processor) {
-    browser()->profile()->GetExtensionService()->
-        settings_frontend()->RunWithSyncableService(
+    content::BrowserThread::PostTask(content::BrowserThread::FILE, FROM_HERE,
+        base::Bind(
+            &ExtensionSettingsApiTest::InitSyncWithSyncableService,
+            this,
+            sync_processor,
             // TODO(kalman): test both EXTENSION_SETTINGS and APP_SETTINGS.
-            syncable::EXTENSION_SETTINGS,
-            base::Bind(
-                &ExtensionSettingsApiTest::InitSyncWithSyncableService,
-                this,
-                sync_processor));
+            browser()->profile()->GetExtensionService()->settings_frontend()->
+                GetBackendForSync(syncable::EXTENSION_SETTINGS)));

[not at google - send to devlin, 2012/02/16 22:58:44]

These tests all run on a single thread with the required message loops, so the
PostTasking should be unnecessary now; just

InitSyncWithSyncableService(
    sync_processor,
    b()->p()->gES()->s_f()->GetBackendForSync(s::E_S));

and hopefully no need for the MessageLoop:: stuff. Same below.

[Nicolas Zea, 2012/02/23 00:27:58]

Done.

[not at google - send to devlin, 2012/02/23 02:15:38]

.. so you do still need the MessageLoop stuff then? It just got moved above the
block.

[Nicolas Zea, 2012/02/24 21:38:04]

Yes, there's apparently other parts of the code that post tasks, and they need
to have run before this (was getting crashes otherwise).

     MessageLoop::current()->RunAllPending();
   }
 
   void SendChanges(const SyncChangeList& change_list) {
-    browser()->profile()->GetExtensionService()->
-        settings_frontend()->RunWithSyncableService(
+    content::BrowserThread::PostTask(content::BrowserThread::FILE, FROM_HERE,
+        base::Bind(
+            &ExtensionSettingsApiTest::SendChangesToSyncableService,
+            this,
+            change_list,
             // TODO(kalman): test both EXTENSION_SETTINGS and APP_SETTINGS.
-            syncable::EXTENSION_SETTINGS,
-            base::Bind(
-                &ExtensionSettingsApiTest::SendChangesToSyncableService,
-                this,
-                change_list));
+            browser()->profile()->GetExtensionService()->settings_frontend()->
+                GetBackendForSync(syncable::EXTENSION_SETTINGS)));
     MessageLoop::current()->RunAllPending();
   }