Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7166)

Unified Diff: chrome/browser/extensions/extension_sync_data.cc

Issue 9460047: sync: remove use of protobuf extensions in protocol to reduce static init overhead. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fred's review Created 8 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 f3bb6c88cb29d13d510562c7f1c9dc16e2e1e551..843e1c6747b0625fc8b62120d0fe31476fc6805c 100644
--- a/chrome/browser/extensions/extension_sync_data.cc
+++ b/chrome/browser/extensions/extension_sync_data.cc
@@ -8,6 +8,7 @@
#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"
+#include "chrome/browser/sync/protocol/sync.pb.h"
ExtensionSyncData::ExtensionSyncData()
: uninstalled_(false),
@@ -92,11 +93,10 @@ SyncData ExtensionSyncData::GetSyncData() const {
switch (type_) {
case Extension::SYNC_TYPE_EXTENSION:
- PopulateExtensionSpecifics(specifics.MutableExtension(
- sync_pb::extension));
+ PopulateExtensionSpecifics(specifics.mutable_extension());
break;
case Extension::SYNC_TYPE_APP:
- PopulateAppSpecifics(specifics.MutableExtension(sync_pb::app));
+ PopulateAppSpecifics(specifics.mutable_app());
break;
default:
LOG(FATAL) << "Attempt to get non-syncable data.";
@@ -156,13 +156,11 @@ void ExtensionSyncData::PopulateFromExtensionSpecifics(
void ExtensionSyncData::PopulateFromSyncData(const SyncData& sync_data) {
const sync_pb::EntitySpecifics& entity_specifics = sync_data.GetSpecifics();
- if (entity_specifics.HasExtension(sync_pb::extension)) {
- PopulateFromExtensionSpecifics(
- entity_specifics.GetExtension(sync_pb::extension));
+ if (entity_specifics.has_extension()) {
+ PopulateFromExtensionSpecifics(entity_specifics.extension());
type_ = Extension::SYNC_TYPE_EXTENSION;
- } else if (entity_specifics.HasExtension(sync_pb::app)) {
- PopulateFromAppSpecifics(
- entity_specifics.GetExtension(sync_pb::app));
+ } else if (entity_specifics.has_app()) {
+ PopulateFromAppSpecifics(entity_specifics.app());
type_ = Extension::SYNC_TYPE_APP;
} else {
LOG(FATAL) << "Attempt to sync bad EntitySpecifics.";
« no previous file with comments | « chrome/browser/extensions/extension_service_unittest.cc ('k') | chrome/browser/extensions/extension_sync_data_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698