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

Unified Diff: chrome/browser/sync/profile_sync_service_autofill_unittest.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/sync/profile_sync_service_autofill_unittest.cc
diff --git a/chrome/browser/sync/profile_sync_service_autofill_unittest.cc b/chrome/browser/sync/profile_sync_service_autofill_unittest.cc
index 411a2192e1daa2e138da95336d11c755c05f121c..7c6d2cf7c3369bbcbe8debccc95439dc24015b81 100644
--- a/chrome/browser/sync/profile_sync_service_autofill_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_autofill_unittest.cc
@@ -451,7 +451,7 @@ class ProfileSyncServiceAutofillTest : public AbstractProfileSyncServiceTest {
sync_pb::EntitySpecifics specifics;
AutocompleteSyncableService::WriteAutofillEntry(entry, &specifics);
sync_pb::AutofillSpecifics* autofill_specifics =
- specifics.MutableExtension(sync_pb::autofill);
+ specifics.mutable_autofill();
node.SetAutofillSpecifics(*autofill_specifics);
return true;
}
@@ -469,7 +469,7 @@ class ProfileSyncServiceAutofillTest : public AbstractProfileSyncServiceTest {
sync_pb::EntitySpecifics specifics;
AutofillProfileSyncableService::WriteAutofillProfile(profile, &specifics);
sync_pb::AutofillProfileSpecifics* profile_specifics =
- specifics.MutableExtension(sync_pb::autofill_profile);
+ specifics.mutable_autofill_profile();
node.SetAutofillProfileSpecifics(*profile_specifics);
return true;
}
@@ -660,8 +660,7 @@ class FakeServerUpdater : public base::RefCountedThreadSafe<FakeServerUpdater> {
}
sync_pb::EntitySpecifics entity_specifics;
- entity_specifics.MutableExtension(sync_pb::autofill)->
- CopyFrom(new_autofill);
+ entity_specifics.mutable_autofill()->CopyFrom(new_autofill);
{
// Tell main thread we've started
« no previous file with comments | « chrome/browser/sync/internal_api/write_node.cc ('k') | chrome/browser/sync/profile_sync_service_preference_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698