Apps/Extensions Sync refactoring -- delete most of the old glue, implement new sync API.

chrome/browser/extensions/extension_sync_data.cc

Index: chrome/browser/extensions/extension_sync_data.cc
diff --git a/chrome/browser/extensions/extension_sync_data.cc b/chrome/browser/extensions/extension_sync_data.cc
index f315552d67e783d60f0b083345a41b4113dd11a4..74b9164237d2adb1ffbd7d1bc521b9ef0458fd0a 100644
--- a/chrome/browser/extensions/extension_sync_data.cc
+++ b/chrome/browser/extensions/extension_sync_data.cc
@@ -5,25 +5,115 @@
 #include "chrome/browser/extensions/extension_sync_data.h"
 
 #include "base/logging.h"
+#include "chrome/browser/extensions/extension_service.h"
+#include "chrome/browser/sync/protocol/app_specifics.pb.h"
+#include "chrome/browser/sync/protocol/extension_specifics.pb.h"
 
 ExtensionSyncData::ExtensionSyncData()
-    : uninstalled(false), enabled(false), incognito_enabled(false) {}
+    : uninstalled_(false), enabled_(false), incognito_enabled_(false) {}
 
 ExtensionSyncData::~ExtensionSyncData() {}
 
-void ExtensionSyncData::Merge(const ExtensionSyncData& new_data) {
-  CHECK_EQ(id, new_data.id);
-  CHECK(!uninstalled);
-  CHECK(!new_data.uninstalled);
+ExtensionSyncData::ExtensionSyncData(const SyncData& sync_data)
+    : uninstalled_(false), enabled_(false), incognito_enabled_(false) {
+  PopulateFromSyncData(sync_data);
+}
+
+ExtensionSyncData::ExtensionSyncData(const SyncChange& sync_change) {
+  PopulateFromSyncData(sync_change.sync_data());
+  uninstalled_ = (sync_change.change_type() == SyncChange::ACTION_DELETE);
+}
+
+ExtensionSyncData::ExtensionSyncData(const Extension& extension,
+                                     bool enabled,
+                                     bool incognito_enabled)
+    : uninstalled_(false) {
+  id_ = extension.id();
+  uninstalled_ = false;
+  enabled_ = enabled;
+  incognito_enabled_ = incognito_enabled;
+  version_ = *extension.version();
+  update_url_ = extension.update_url();
+  name_ = extension.name();
+  type_ = extension.GetSyncType();
+}
+
+void ExtensionSyncData::PopulateSyncSpecifics(
+    sync_pb::ExtensionSpecifics* specifics) const {
+  DCHECK(Extension::IdIsValid(id_));
+  specifics->set_id(id_);
+  specifics->set_update_url(update_url_.spec());
+  specifics->set_version(version_.GetString());
+  specifics->set_enabled(enabled_);
+  specifics->set_incognito_enabled(incognito_enabled_);
+  specifics->set_name(name_);
+}
 
-  // Copy version-independent properties.
-  enabled = new_data.enabled;
-  incognito_enabled = new_data.incognito_enabled;
+SyncData ExtensionSyncData::GetSyncData() const {
+  sync_pb::EntitySpecifics specifics;
+  sync_pb::ExtensionSpecifics* ext_specifics;
 
-  // Copy version-dependent properties if version <= new_data.version.
-  if (version.CompareTo(new_data.version) <= 0) {
-    version = new_data.version;
-    update_url = new_data.update_url;
-    name = new_data.name;
+  switch (type_) {
+    case Extension::SYNC_TYPE_EXTENSION:
+      ext_specifics = specifics.MutableExtension(sync_pb::extension);
+      break;
+    case Extension::SYNC_TYPE_APP:
+      ext_specifics =
+          specifics.MutableExtension(sync_pb::app)->mutable_extension();
+      break;
+    default:
+      NOTREACHED();
   }
+
+  PopulateSyncSpecifics(ext_specifics);
+
+  return SyncData::CreateLocalData(id_, name_, specifics);
 }
+
+SyncChange ExtensionSyncData::GetSyncChange(
+    SyncChange::SyncChangeType change_type) const {
+  return SyncChange(change_type, GetSyncData());
+}
+
+void ExtensionSyncData::PopulateFromExtensionSpecifics(
+    const sync_pb::ExtensionSpecifics& specifics) {
+  if (!Extension::IdIsValid(specifics.id())) {
+    NOTREACHED();
+  }
+
+  scoped_ptr<Version> specifics_version(
+      Version::GetVersionFromString(specifics.version()));
+  if (!specifics_version.get()) {
+    NOTREACHED();
+  }
+
+  // The update URL must be either empty or valid.
+  GURL specifics_update_url(specifics.update_url());
+  if (!specifics_update_url.is_empty() && !specifics_update_url.is_valid()) {
+    NOTREACHED();
+  }
+
+  id_ = specifics.id();
+  update_url_ = specifics_update_url;
+  version_ = *specifics_version;
+  enabled_ = specifics.enabled();
+  incognito_enabled_ = specifics.incognito_enabled();
+  name_ = specifics.name();
+}
+
+void ExtensionSyncData::PopulateFromSyncData(const SyncData& sync_data) {
+  const sync_pb::EntitySpecifics& ent_spec = sync_data.GetSpecifics();
+  sync_pb::ExtensionSpecifics es;

[asargent_no_longer_on_chrome, 2011/08/12 20:50:39]

style nit: "es" and "ent_spec" are probably too short. 

http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#General_Naming...

Maybe s/es/specifics/ and s/ent_spec/entity/ ?

+  if (ent_spec.HasExtension(sync_pb::extension)) {
+    es = ent_spec.GetExtension(sync_pb::extension);
+    type_ = Extension::SYNC_TYPE_EXTENSION;
+  } else if (ent_spec.HasExtension(sync_pb::app)) {
+    es = ent_spec.GetExtension(sync_pb::app).extension();
+    type_ = Extension::SYNC_TYPE_APP;
+  } else {
+    NOTREACHED();
+  }
+  PopulateFromExtensionSpecifics(es);
+  uninstalled_ = false;
+}
+