[Sync] Add intelligent re-encryption support.

chrome/browser/sync/util/cryptographer.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <algorithm>
 
 #include "base/base64.h"
 #include "chrome/browser/sync/util/cryptographer.h"
 #include "chrome/browser/password_manager/encryptor.h"
 
@@ skipping 39 matching lines @@
 
 bool Cryptographer::CanDecrypt(const sync_pb::EncryptedData& data) const {
   return nigoris_.end() != nigoris_.find(data.key_name());
 }
 
 bool Cryptographer::CanDecryptUsingDefaultKey(
     const sync_pb::EncryptedData& data) const {
   return default_nigori_ && (data.key_name() == default_nigori_->first);
 }
 
-bool Cryptographer::Encrypt(const ::google::protobuf::MessageLite& message,
-                            sync_pb::EncryptedData* encrypted) const {
-  if (!encrypted || !default_nigori_) {
+bool Cryptographer::Encrypt(
+    const ::google::protobuf::MessageLite& message,
+    sync_pb::EncryptedData* encrypted) const {
+  DCHECK(encrypted);
+  if (!default_nigori_) {
     LOG(ERROR) << "Cryptographer not ready, failed to encrypt.";
     return false;
   }
 
   std::string serialized;
   if (!message.SerializeToString(&serialized)) {
     LOG(ERROR) << "Message is invalid/missing a required field.";
     return false;
   }
 
+  if (CanDecryptUsingDefaultKey(*encrypted)) {
+    const std::string& original_serialized = DecryptToString(*encrypted);
+    if (original_serialized == serialized) {
+      DVLOG(2) << "Re-encryption unnecessary, encrypted data already matches.";
+      return true;
+    }
+  }
+
+  return EncryptImpl(serialized, default_nigori_, encrypted);

[akalin, 2011/12/12 22:05:22]

do we still need the Encrypt/EncryptImpl divide?

[Nicolas Zea, 2011/12/13 00:43:30]

Done.

+}
+
+bool Cryptographer::EncryptImpl(const std::string& serialized,
+                                const NigoriMap::value_type* nigori,
+                                sync_pb::EncryptedData* encrypted) const {
   encrypted->set_key_name(default_nigori_->first);
-  if (!default_nigori_->second->Encrypt(serialized,
-                                        encrypted->mutable_blob())) {
+  if (!nigori->second->Encrypt(serialized,
+                               encrypted->mutable_blob())) {
     LOG(ERROR) << "Failed to encrypt data.";
     return false;
   }
   return true;
 }
 
 bool Cryptographer::Decrypt(const sync_pb::EncryptedData& encrypted,
                             ::google::protobuf::MessageLite* message) const {
   DCHECK(message);
   std::string plaintext = DecryptToString(encrypted);
@@ skipping 328 matching lines @@
         continue;
       }
       nigoris_[key.name()] = make_linked_ptr(new_nigori.release());
     }
   }
   DCHECK(nigoris_.end() != nigoris_.find(default_key_name));
   default_nigori_ = &*nigoris_.find(default_key_name);
 }
 
 }  // namespace browser_sync