[Sync] Move some extension-sync-related logic to ExtensionService

chrome/browser/extensions/extension_service.h

Index: chrome/browser/extensions/extension_service.h
diff --git a/chrome/browser/extensions/extension_service.h b/chrome/browser/extensions/extension_service.h
index 9fd84e226957f3cea94f4fd42e4ad643eb18060a..4896dab9b865b6c54e0d8cb948e77e728192ea5c 100644
--- a/chrome/browser/extensions/extension_service.h
+++ b/chrome/browser/extensions/extension_service.h
@@ -39,6 +39,7 @@
 class ExtensionBrowserEventRouter;
 class ExtensionPreferenceEventRouter;
 class ExtensionServiceBackend;
+struct ExtensionSyncData;
 class ExtensionToolbarModel;
 class ExtensionUpdater;
 class GURL;
@@ -81,9 +82,30 @@ class ExtensionServiceInterface {
 
   // Safe to call multiple times in a row.
   //
-  // TODO(akalin): Remove this method (and others) once we add
-  // ProcessSyncData().
+  // TODO(akalin): Remove this method (and others) once we refactor
+  // themes sync to not use it directly.
   virtual void CheckForUpdatesSoon() = 0;
+
+  // Take any actions required to make the local state of the
+  // extension match the state in |extension_sync_data| (including
+  // installing/uninstalling the extension).
+  //
+  // TODO(akalin): We'll eventually need a separate method for app
+  // sync.  See http://crbug.com/58077 and http://crbug.com/61447.
+  virtual void ProcessSyncData(
+      const ExtensionSyncData& extension_sync_data,
+      PendingExtensionInfo::ShouldAllowInstallPredicate
+          should_allow_install) = 0;
+
+  // TODO(akalin): Add a method like:
+  //
+  //   virtual void
+  //     GetInitialSyncData(bool (*filter)(Extension),
+  //                        map<string, ExtensionSyncData>* out) const;
+  //
+  // which would fill |out| with sync data for the extensions that
+  // match |filter|.  Sync would use this for the initial syncing
+  // step.
 };
 
 // Manages installed and running Chromium extensions.
@@ -345,6 +367,11 @@ class ExtensionService
 
   virtual void CheckForUpdatesSoon();
 
+  virtual void ProcessSyncData(
+      const ExtensionSyncData& extension_sync_data,
+      PendingExtensionInfo::ShouldAllowInstallPredicate
+          should_allow_install);
+
   void set_extensions_enabled(bool enabled) { extensions_enabled_ = enabled; }
   bool extensions_enabled() { return extensions_enabled_; }