summaryrefslogtreecommitdiffstats
path: root/chrome/browser/sync/engine
diff options
context:
space:
mode:
authorzea@chromium.org <zea@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-12-22 18:32:18 +0000
committerzea@chromium.org <zea@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-12-22 18:32:18 +0000
commit45dbaf9cae272a5942d41e2316fe83e9d0340cdc (patch)
tree36b8eb2738950bec5f8c01948ef6bb9e11ef16c2 /chrome/browser/sync/engine
parent7e11bf57a0d649706afe8d3f00e708095dc1305c (diff)
downloadchromium_src-45dbaf9cae272a5942d41e2316fe83e9d0340cdc.zip
chromium_src-45dbaf9cae272a5942d41e2316fe83e9d0340cdc.tar.gz
chromium_src-45dbaf9cae272a5942d41e2316fe83e9d0340cdc.tar.bz2
Merge 113550 - [Sync] Special case nigori when calculating commit id's.
We special case the nigori node because even though it is considered an "encrypted type", not all nigori node changes require valid encryption (ex: sync_tabs). BUG=106619 TEST=none. Review URL: http://codereview.chromium.org/8856007 TBR=zea@chromium.org Review URL: http://codereview.chromium.org/9025021 git-svn-id: svn://svn.chromium.org/chrome/branches/963/src@115563 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/sync/engine')
-rw-r--r--chrome/browser/sync/engine/get_commit_ids_command.cc6
1 files changed, 5 insertions, 1 deletions
diff --git a/chrome/browser/sync/engine/get_commit_ids_command.cc b/chrome/browser/sync/engine/get_commit_ids_command.cc
index 3000649..4c280ec 100644
--- a/chrome/browser/sync/engine/get_commit_ids_command.cc
+++ b/chrome/browser/sync/engine/get_commit_ids_command.cc
@@ -94,7 +94,11 @@ bool IsEntryReadyForCommit(const syncable::ModelTypeSet& encrypted_types,
}
syncable::ModelType type = entry.GetModelType();
- if (encrypted_types.count(type) > 0 &&
+ // We special case the nigori node because even though it is considered an
+ // "encrypted type", not all nigori node changes require valid encryption
+ // (ex: sync_tabs).
+ if (type != syncable::NIGORI &&
+ encrypted_types.count(type) > 0 &&
(passphrase_missing ||
syncable::EntryNeedsEncryption(encrypted_types, entry))) {
// This entry requires encryption but is not properly encrypted (possibly