Remove Ordinals Setters and Getters from 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 c0905049b4138e763afae4066cb812ba15ff6985..19da62407338740a82b06da85e744772315e96d9 100644
--- a/chrome/browser/extensions/extension_service.h
+++ b/chrome/browser/extensions/extension_service.h
@@ -125,6 +125,8 @@ class ExtensionServiceInterface : public SyncableService {
       const std::string& extension_id,
       extension_misc::UnloadedExtensionReason reason) = 0;
 
+  virtual void SyncExtensionChangeIfNeeded(const Extension& extension) = 0;
+
   virtual bool is_ready() = 0;
 };
 
@@ -227,17 +229,6 @@ class ExtensionService
   virtual void SetAppNotificationDisabled(const std::string& extension_id,
       bool value);
 
-  // Getters and setters for the position of Apps in the NTP. The setters
-  // will trigger a sync if needed.
-  // The getters return invalid StringOridinals for non-app extensions and
-  // setting ordinals for non-apps is an error.
-  StringOrdinal GetAppLaunchOrdinal(const std::string& extension_id) const;
-  void SetAppLaunchOrdinal(const std::string& extension_id,
-                           const StringOrdinal& app_launch_ordinal);
-  StringOrdinal GetPageOrdinal(const std::string& extension_id) const;
-  void SetPageOrdinal(const std::string& extension_id,
-                      const StringOrdinal& page_ordinal);
-
   // Updates the app launcher value for the moved extension so that it is now
   // located after the given predecessor and before the successor. This will
   // trigger a sync if needed. Empty strings are used to indicate no successor
@@ -375,6 +366,10 @@ class ExtensionService
   // Scan the extension directory and clean up the cruft.
   void GarbageCollectExtensions();
 
+  // Notifies Sync (if needed) of a newly-installed extension or a change to
+  // an existing extension.
+  virtual void SyncExtensionChangeIfNeeded(const Extension& extension) OVERRIDE;
+
   // The App that represents the web store.
   const Extension* GetWebStoreApp();
 
@@ -634,10 +629,6 @@ class ExtensionService
   };
   typedef std::list<NaClModuleInfo> NaClModuleInfoList;
 
-  // Notifies Sync (if needed) of a newly-installed extension or a change to
-  // an existing extension.
-  void SyncExtensionChangeIfNeeded(const Extension& extension);
-
   // Get the appropriate SyncBundle, given some representation of Sync data.
   SyncBundle* GetSyncBundleForExtension(const Extension& extension);
   SyncBundle* GetSyncBundleForExtensionSyncData(