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

Unified Diff: chrome/browser/sync/internal_api/write_node.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: fix nigori access in testserver 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/internal_api/write_node.cc
diff --git a/chrome/browser/sync/internal_api/write_node.cc b/chrome/browser/sync/internal_api/write_node.cc
index 00f955077e065001fb25a796611240831d64fdec..bc65bd430cf10a0b0c585c9620b72c5608810c14 100644
--- a/chrome/browser/sync/internal_api/write_node.cc
+++ b/chrome/browser/sync/internal_api/write_node.cc
@@ -103,7 +103,7 @@ bool WriteNode::UpdateEntryWithEncryption(
// else the server will try to do it for us.
if (type == syncable::BOOKMARKS) {
sync_pb::BookmarkSpecifics* bookmark_specifics =
- generated_specifics.MutableExtension(sync_pb::bookmark);
+ generated_specifics.mutable_bookmark();
if (!entry->Get(syncable::IS_DIR))
bookmark_specifics->set_url(kEncryptedString);
bookmark_specifics->set_title(kEncryptedString);
@@ -172,7 +172,7 @@ void WriteNode::SetTitle(const std::wstring& title) {
// TODO(zea): refactor bookmarks to not need this functionality.
if (GetModelType() == syncable::BOOKMARKS) {
sync_pb::EntitySpecifics specifics = GetEntitySpecifics();
- specifics.MutableExtension(sync_pb::bookmark)->set_title(new_legal_title);
+ specifics.mutable_bookmark()->set_title(new_legal_title);
SetEntitySpecifics(specifics); // Does it's own encryption checking.
}
@@ -200,21 +200,21 @@ void WriteNode::SetURL(const GURL& url) {
void WriteNode::SetAppSpecifics(
const sync_pb::AppSpecifics& new_value) {
sync_pb::EntitySpecifics entity_specifics;
- entity_specifics.MutableExtension(sync_pb::app)->CopyFrom(new_value);
+ entity_specifics.mutable_app()->CopyFrom(new_value);
SetEntitySpecifics(entity_specifics);
}
void WriteNode::SetAutofillSpecifics(
const sync_pb::AutofillSpecifics& new_value) {
sync_pb::EntitySpecifics entity_specifics;
- entity_specifics.MutableExtension(sync_pb::autofill)->CopyFrom(new_value);
+ entity_specifics.mutable_autofill()->CopyFrom(new_value);
SetEntitySpecifics(entity_specifics);
}
void WriteNode::SetAutofillProfileSpecifics(
const sync_pb::AutofillProfileSpecifics& new_value) {
sync_pb::EntitySpecifics entity_specifics;
- entity_specifics.MutableExtension(sync_pb::autofill_profile)->
+ entity_specifics.mutable_autofill_profile()->
CopyFrom(new_value);
SetEntitySpecifics(entity_specifics);
}
@@ -222,14 +222,14 @@ void WriteNode::SetAutofillProfileSpecifics(
void WriteNode::SetBookmarkSpecifics(
const sync_pb::BookmarkSpecifics& new_value) {
sync_pb::EntitySpecifics entity_specifics;
- entity_specifics.MutableExtension(sync_pb::bookmark)->CopyFrom(new_value);
+ entity_specifics.mutable_bookmark()->CopyFrom(new_value);
SetEntitySpecifics(entity_specifics);
}
void WriteNode::SetNigoriSpecifics(
const sync_pb::NigoriSpecifics& new_value) {
sync_pb::EntitySpecifics entity_specifics;
- entity_specifics.MutableExtension(sync_pb::nigori)->CopyFrom(new_value);
+ entity_specifics.mutable_nigori()->CopyFrom(new_value);
SetEntitySpecifics(entity_specifics);
}
@@ -253,7 +253,7 @@ void WriteNode::SetPasswordSpecifics(
&entity_specifics);
}
sync_pb::PasswordSpecifics* password_specifics =
- entity_specifics.MutableExtension(sync_pb::password);
+ entity_specifics.mutable_password();
// This will only update password_specifics if the underlying unencrypted blob
// was different from |data| or was not encrypted with the proper passphrase.
if (!cryptographer->Encrypt(data, password_specifics->mutable_encrypted())) {
@@ -267,14 +267,14 @@ void WriteNode::SetPasswordSpecifics(
void WriteNode::SetThemeSpecifics(
const sync_pb::ThemeSpecifics& new_value) {
sync_pb::EntitySpecifics entity_specifics;
- entity_specifics.MutableExtension(sync_pb::theme)->CopyFrom(new_value);
+ entity_specifics.mutable_theme()->CopyFrom(new_value);
SetEntitySpecifics(entity_specifics);
}
void WriteNode::SetSessionSpecifics(
const sync_pb::SessionSpecifics& new_value) {
sync_pb::EntitySpecifics entity_specifics;
- entity_specifics.MutableExtension(sync_pb::session)->CopyFrom(new_value);
+ entity_specifics.mutable_session()->CopyFrom(new_value);
SetEntitySpecifics(entity_specifics);
}
@@ -324,14 +324,14 @@ void WriteNode::ResetFromSpecifics() {
void WriteNode::SetTypedUrlSpecifics(
const sync_pb::TypedUrlSpecifics& new_value) {
sync_pb::EntitySpecifics entity_specifics;
- entity_specifics.MutableExtension(sync_pb::typed_url)->CopyFrom(new_value);
+ entity_specifics.mutable_typed_url()->CopyFrom(new_value);
SetEntitySpecifics(entity_specifics);
}
void WriteNode::SetExtensionSpecifics(
const sync_pb::ExtensionSpecifics& new_value) {
sync_pb::EntitySpecifics entity_specifics;
- entity_specifics.MutableExtension(sync_pb::extension)->CopyFrom(new_value);
+ entity_specifics.mutable_extension()->CopyFrom(new_value);
SetEntitySpecifics(entity_specifics);
}

Powered by Google App Engine
This is Rietveld 408576698