Adjust preference sync code to only sync user modifiable preferences.

chrome/browser/sync/profile_sync_service_preference_unittest.cc

 // Copyright (c) 2010 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 <map>
 #include <string>
 
 #include "base/json/json_reader.h"
 #include "base/stl_util-inl.h"
 #include "base/task.h"
@@ skipping 155 matching lines @@
   std::string ValueString(const Value& value) {
     std::string serialized;
     JSONStringValueSerializer json(&serialized);
     json.Serialize(value);
     return serialized;
   }
 
   friend class AddPreferenceEntriesTask;
 
   scoped_ptr<TestingProfile> profile_;
-  PrefService* prefs_;
+  TestingPrefService* prefs_;
 
   PreferenceModelAssociator* model_associator_;
   PreferenceChangeProcessor* change_processor_;
   std::string example_url0_;
   std::string example_url1_;
   std::string example_url2_;
   std::wstring not_synced_preference_name_;
   std::string not_synced_preference_default_value_;
   std::string non_default_charset_value_;
 };
@@ skipping 217 matching lines @@
   record->action = SyncManager::ChangeRecord::ACTION_ADD;
   record->id = node_id;
   {
     sync_api::WriteTransaction trans(backend()->GetUserShareHandle());
     change_processor_->ApplyChangesFromSyncModel(&trans, record.get(), 1);
   }
 
   // Nothing interesting happens on the client when it gets an update
   // of an unknown preference.  We just should not crash.
 }
+
+TEST_F(ProfileSyncServicePreferenceTest, ManagedPreferences) {
+  // Make the homepage preference managed.
+  scoped_ptr<Value> managed_value(
+      Value::CreateStringValue(L"http://example.com"));
+  prefs_->SetManagedPref(prefs::kHomePage, managed_value->DeepCopy());
+
+  CreateRootTask task(this, syncable::PREFERENCES);
+  ASSERT_TRUE(StartSyncService(&task, false));
+  ASSERT_TRUE(task.success());
+
+  // Changing the homepage preference should not sync anything.
+  scoped_ptr<Value> user_value(
+      Value::CreateStringValue(L"http://chromium..com"));
+  prefs_->SetUserPref(prefs::kHomePage, user_value->DeepCopy());
+  EXPECT_EQ(NULL, GetSyncedValue(prefs::kHomePage));
+
+  // An incoming sync transaction shouldn't change the user value.
+  scoped_ptr<Value> sync_value(
+      Value::CreateStringValue(L"http://crbug.com"));
+  int64 node_id = SetSyncedValue(prefs::kHomePage, *sync_value);
+  ASSERT_NE(node_id, sync_api::kInvalidId);
+  scoped_ptr<SyncManager::ChangeRecord> record(new SyncManager::ChangeRecord);
+  record->action = SyncManager::ChangeRecord::ACTION_UPDATE;
+  record->id = node_id;
+  {
+    sync_api::WriteTransaction trans(backend()->GetUserShareHandle());
+    change_processor_->ApplyChangesFromSyncModel(&trans, record.get(), 1);
+  }
+  EXPECT_TRUE(managed_value->Equals(prefs_->GetManagedPref(prefs::kHomePage)));
+  EXPECT_TRUE(user_value->Equals(prefs_->GetUserPref(prefs::kHomePage)));
+}