[Sync] Fix encrypting of sync titles.

chrome/browser/sync/internal_api/syncapi_unittest.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.
 
 // Unit tests for the SyncApi. Note that a lot of the underlying
 // functionality is provided by the Syncable layer, which has its own
 // unit tests. We'll test SyncApi specific things in this harness.
 
 #include <cstddef>
 #include <map>
@@ skipping 50 matching lines @@
 using browser_sync::JsEventHandler;
 using browser_sync::JsReplyHandler;
 using browser_sync::MockJsEventHandler;
 using browser_sync::MockJsReplyHandler;
 using browser_sync::ModelSafeRoutingInfo;
 using browser_sync::ModelSafeWorker;
 using browser_sync::ModelSafeWorkerRegistrar;
 using browser_sync::sessions::SyncSessionSnapshot;
 using browser_sync::WeakHandle;
 using syncable::GetAllRealModelTypes;
+using syncable::kEncryptedString;
 using syncable::ModelType;
 using syncable::ModelTypeSet;
 using test::ExpectDictDictionaryValue;
 using test::ExpectDictStringValue;
 using testing::_;
 using testing::AnyNumber;
 using testing::AtLeast;
 using testing::InSequence;
 using testing::Invoke;
 using testing::SaveArg;
@@ skipping 314 matching lines @@
 
     ReadNode password_node(&trans);
     EXPECT_TRUE(password_node.InitByClientTagLookup(syncable::PASSWORDS,
                                                     "foo"));
     const sync_pb::PasswordSpecificsData& data =
         password_node.GetPasswordSpecifics();
     EXPECT_EQ("secret", data.password_value());
   }
 }
 
+TEST_F(SyncApiTest, WriteEncryptedTitle) {
+  KeyParams params = {"localhost", "username", "passphrase"};
+  {
+    ReadTransaction trans(FROM_HERE, test_user_share_.user_share());
+    trans.GetCryptographer()->AddKey(params);
+    trans.GetCryptographer()->set_encrypt_everything();
+  }
+  {
+    WriteTransaction trans(FROM_HERE, test_user_share_.user_share());
+    ReadNode root_node(&trans);
+    root_node.InitByRootLookup();
+
+    WriteNode bookmark_node(&trans);
+    EXPECT_TRUE(bookmark_node.InitUniqueByCreation(syncable::BOOKMARKS,
+                                                   root_node, "foo"));
+    bookmark_node.SetTitle(UTF8ToWide("foo"));
+
+    WriteNode pref_node(&trans);
+    EXPECT_TRUE(pref_node.InitUniqueByCreation(syncable::PREFERENCES,
+                                               root_node, "bar"));
+    pref_node.SetTitle(UTF8ToWide("bar"));
+  }
+  {
+    ReadTransaction trans(FROM_HERE, test_user_share_.user_share());
+    ReadNode root_node(&trans);
+    root_node.InitByRootLookup();
+
+    ReadNode bookmark_node(&trans);
+    EXPECT_TRUE(bookmark_node.InitByClientTagLookup(syncable::BOOKMARKS,
+                                                    "foo"));
+    EXPECT_EQ("foo", bookmark_node.GetTitle());
+    EXPECT_EQ(kEncryptedString,
+              bookmark_node.GetEntry()->Get(syncable::NON_UNIQUE_NAME));
+
+    ReadNode pref_node(&trans);
+    EXPECT_TRUE(pref_node.InitByClientTagLookup(syncable::PREFERENCES,
+                                                "bar"));
+    EXPECT_EQ(kEncryptedString, pref_node.GetTitle());
+  }
+}
+
 TEST_F(SyncApiTest, BaseNodeSetSpecifics) {
   int64 child_id = MakeNode(test_user_share_.user_share(),
                             syncable::BOOKMARKS, "testtag");
   WriteTransaction trans(FROM_HERE, test_user_share_.user_share());
   WriteNode node(&trans);
   EXPECT_TRUE(node.InitByIdLookup(child_id));
 
   sync_pb::EntitySpecifics entity_specifics;
   entity_specifics.MutableExtension(sync_pb::bookmark)->
       set_url("http://www.google.com");
@@ skipping 1092 matching lines @@
     ReadNode node(&trans);
     EXPECT_TRUE(node.InitByIdLookup(node1));
     EXPECT_EQ(syncable::BOOKMARKS, node.GetModelType());
     EXPECT_EQ(title, node.GetTitle());
     EXPECT_EQ(title, node.GetBookmarkSpecifics().title());
     EXPECT_EQ(url, node.GetBookmarkSpecifics().url());
   }
 }
 
 }  // namespace browser_sync