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

Unified Diff: chrome/browser/prefs/pref_model_associator.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/prefs/pref_model_associator.cc
diff --git a/chrome/browser/prefs/pref_model_associator.cc b/chrome/browser/prefs/pref_model_associator.cc
index 0f48fd0b663827a716417edf87eb3f035f513ab7..dd28d81952f2b686853e73ca6ca4c8db7f5d34b1 100644
--- a/chrome/browser/prefs/pref_model_associator.cc
+++ b/chrome/browser/prefs/pref_model_associator.cc
@@ -13,6 +13,7 @@
#include "base/values.h"
#include "chrome/browser/sync/api/sync_change.h"
#include "chrome/browser/sync/protocol/preference_specifics.pb.h"
+#include "chrome/browser/sync/protocol/sync.pb.h"
#include "chrome/common/chrome_notification_types.h"
#include "chrome/common/pref_names.h"
@@ -46,7 +47,7 @@ void PrefModelAssociator::InitPrefAndAssociate(
// The server has a value for the preference, we have to reconcile it with
// ours.
const sync_pb::PreferenceSpecifics& preference =
- sync_pref.GetSpecifics().GetExtension(sync_pb::preference);
+ sync_pref.GetSpecifics().preference();
DCHECK_EQ(pref->name(), preference.name());
scoped_ptr<Value> value(
@@ -124,8 +125,7 @@ SyncError PrefModelAssociator::MergeDataAndStartSyncing(
sync_iter != initial_sync_data.end();
++sync_iter) {
DCHECK_EQ(PREFERENCES, sync_iter->GetDataType());
- std::string sync_pref_name = sync_iter->GetSpecifics().
- GetExtension(sync_pb::preference).name();
+ std::string sync_pref_name = sync_iter->GetSpecifics().preference().name();
if (remaining_preferences.count(sync_pref_name) == 0) {
// We're not syncing this preference locally, ignore the sync data.
// TODO(zea): Eventually we want to be able to have the syncable service
@@ -198,8 +198,7 @@ bool PrefModelAssociator::CreatePrefSyncData(
}
sync_pb::EntitySpecifics specifics;
- sync_pb::PreferenceSpecifics* pref_specifics = specifics.MutableExtension(
- sync_pb::preference);
+ sync_pb::PreferenceSpecifics* pref_specifics = specifics.mutable_preference();
pref_specifics->set_name(name);
pref_specifics->set_value(serialized);
*sync_data = SyncData::CreateLocalData(name, name, specifics);
@@ -306,7 +305,7 @@ SyncError PrefModelAssociator::ProcessSyncChanges(
std::string name;
sync_pb::PreferenceSpecifics pref_specifics =
- iter->sync_data().GetSpecifics().GetExtension(sync_pb::preference);
+ iter->sync_data().GetSpecifics().preference();
scoped_ptr<Value> value(ReadPreferenceSpecifics(pref_specifics,
&name));
« no previous file with comments | « chrome/browser/extensions/settings/settings_sync_util.cc ('k') | chrome/browser/search_engines/template_url_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698