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

Unified Diff: chrome/browser/sync/glue/preference_model_associator.cc

Issue 6905044: Refactor preference syncing. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Really fix. And rebase Created 9 years, 7 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/glue/preference_model_associator.cc
diff --git a/chrome/browser/sync/glue/preference_model_associator.cc b/chrome/browser/sync/glue/preference_model_associator.cc
deleted file mode 100644
index 7d75c3861fce2476368be1ed7d4d4d10eb33382b..0000000000000000000000000000000000000000
--- a/chrome/browser/sync/glue/preference_model_associator.cc
+++ /dev/null
@@ -1,338 +0,0 @@
-// 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 "chrome/browser/sync/glue/preference_model_associator.h"
-
-#include "base/json/json_reader.h"
-#include "base/logging.h"
-#include "base/utf_string_conversions.h"
-#include "base/values.h"
-#include "chrome/browser/prefs/pref_service.h"
-#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/sync/engine/syncapi.h"
-#include "chrome/browser/sync/glue/synchronized_preferences.h"
-#include "chrome/browser/sync/profile_sync_service.h"
-#include "chrome/browser/sync/protocol/preference_specifics.pb.h"
-#include "chrome/common/pref_names.h"
-#include "content/browser/browser_thread.h"
-#include "content/common/json_value_serializer.h"
-#include "content/common/notification_service.h"
-
-namespace browser_sync {
-
-PreferenceModelAssociator::PreferenceModelAssociator(
- ProfileSyncService* sync_service)
- : sync_service_(sync_service),
- preferences_node_id_(sync_api::kInvalidId) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- DCHECK(sync_service_);
-
- // Add the list of kSynchronizedPreferences to our local
- // synced_preferences set, taking care to filter out any preferences
- // that are not registered.
- PrefService* pref_service = sync_service_->profile()->GetPrefs();
- for (size_t i = 0; i < arraysize(kSynchronizedPreferences); ++i) {
- if (pref_service->FindPreference(kSynchronizedPreferences[i]))
- synced_preferences_.insert(kSynchronizedPreferences[i]);
- }
-}
-
-PreferenceModelAssociator::~PreferenceModelAssociator() {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-}
-
-bool PreferenceModelAssociator::InitPrefNodeAndAssociate(
- sync_api::WriteTransaction* trans,
- const sync_api::BaseNode& root,
- const PrefService::Preference* pref) {
- DCHECK(pref);
-
- PrefService* pref_service = sync_service_->profile()->GetPrefs();
- base::JSONReader reader;
- std::string tag = pref->name();
- sync_api::WriteNode node(trans);
- if (node.InitByClientTagLookup(syncable::PREFERENCES, tag)) {
- // The server has a value for the preference.
- const sync_pb::PreferenceSpecifics& preference(
- node.GetPreferenceSpecifics());
- DCHECK_EQ(tag, preference.name());
-
- if (pref->IsUserModifiable()) {
- scoped_ptr<Value> value(
- reader.JsonToValue(preference.value(), false, false));
- std::string pref_name = preference.name();
- if (!value.get()) {
- LOG(ERROR) << "Failed to deserialize preference value: "
- << reader.GetErrorMessage();
- return false;
- }
-
- // Merge the server value of this preference with the local value.
- scoped_ptr<Value> new_value(MergePreference(*pref, *value));
-
- // Update the local preference based on what we got from the
- // sync server.
- if (new_value->IsType(Value::TYPE_NULL)) {
- pref_service->ClearPref(pref_name.c_str());
- } else if (!new_value->IsType(pref->GetType())) {
- LOG(WARNING) << "Synced value for " << preference.name()
- << " is of type " << new_value->GetType()
- << " which doesn't match pref type " << pref->GetType();
- } else if (!pref->GetValue()->Equals(new_value.get())) {
- pref_service->Set(pref_name.c_str(), *new_value);
- }
-
- AfterUpdateOperations(pref_name);
-
- // If the merge resulted in an updated value, write it back to
- // the sync node.
- if (!value->Equals(new_value.get()) &&
- !WritePreferenceToNode(pref->name(), *new_value, &node))
- return false;
- }
- Associate(pref, node.GetId());
- } else if (pref->IsUserControlled()) {
- // The server doesn't have a value, but we have a user-controlled value,
- // so we push it to the server.
- sync_api::WriteNode write_node(trans);
- if (!write_node.InitUniqueByCreation(syncable::PREFERENCES, root, tag)) {
- LOG(ERROR) << "Failed to create preference sync node.";
- return false;
- }
-
- // Update the sync node with the local value for this preference.
- if (!WritePreferenceToNode(pref->name(), *pref->GetValue(), &write_node))
- return false;
-
- Associate(pref, write_node.GetId());
- }
-
- return true;
-}
-
-bool PreferenceModelAssociator::AssociateModels() {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- PrefService* pref_service = sync_service_->profile()->GetPrefs();
-
- int64 root_id;
- if (!GetSyncIdForTaggedNode(kPreferencesTag, &root_id)) {
- LOG(ERROR) << "Server did not create the top-level preferences node. We "
- << "might be running against an out-of-date server.";
- return false;
- }
-
- sync_api::WriteTransaction trans(sync_service_->GetUserShare());
- sync_api::ReadNode root(&trans);
- if (!root.InitByIdLookup(root_id)) {
- LOG(ERROR) << "Server did not create the top-level preferences node. We "
- << "might be running against an out-of-date server.";
- return false;
- }
-
- for (std::set<std::string>::iterator it = synced_preferences_.begin();
- it != synced_preferences_.end(); ++it) {
- const PrefService::Preference* pref =
- pref_service->FindPreference((*it).c_str());
- DCHECK(pref);
- InitPrefNodeAndAssociate(&trans, root, pref);
- }
- return true;
-}
-
-bool PreferenceModelAssociator::DisassociateModels() {
- id_map_.clear();
- id_map_inverse_.clear();
- return true;
-}
-
-bool PreferenceModelAssociator::SyncModelHasUserCreatedNodes(bool* has_nodes) {
- DCHECK(has_nodes);
- *has_nodes = false;
- int64 preferences_sync_id;
- if (!GetSyncIdForTaggedNode(kPreferencesTag, &preferences_sync_id)) {
- LOG(ERROR) << "Server did not create the top-level preferences node. We "
- << "might be running against an out-of-date server.";
- return false;
- }
- sync_api::ReadTransaction trans(sync_service_->GetUserShare());
-
- sync_api::ReadNode preferences_node(&trans);
- if (!preferences_node.InitByIdLookup(preferences_sync_id)) {
- LOG(ERROR) << "Server did not create the top-level preferences node. We "
- << "might be running against an out-of-date server.";
- return false;
- }
-
- // The sync model has user created nodes if the preferences folder has any
- // children.
- *has_nodes = sync_api::kInvalidId != preferences_node.GetFirstChildId();
- return true;
-}
-
-const PrefService::Preference*
-PreferenceModelAssociator::GetChromeNodeFromSyncId(int64 sync_id) {
- return NULL;
-}
-
-bool PreferenceModelAssociator::InitSyncNodeFromChromeId(
- const std::string& node_id,
- sync_api::BaseNode* sync_node) {
- return false;
-}
-
-int64 PreferenceModelAssociator::GetSyncIdFromChromeId(
- const std::string& preference_name) {
- PreferenceNameToSyncIdMap::const_iterator iter =
- id_map_.find(preference_name);
- return iter == id_map_.end() ? sync_api::kInvalidId : iter->second;
-}
-
-void PreferenceModelAssociator::Associate(
- const PrefService::Preference* preference, int64 sync_id) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- DCHECK_NE(sync_api::kInvalidId, sync_id);
- DCHECK(id_map_.find(preference->name()) == id_map_.end());
- DCHECK(id_map_inverse_.find(sync_id) == id_map_inverse_.end());
- id_map_[preference->name()] = sync_id;
- id_map_inverse_[sync_id] = preference->name();
-}
-
-void PreferenceModelAssociator::Disassociate(int64 sync_id) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- SyncIdToPreferenceNameMap::iterator iter = id_map_inverse_.find(sync_id);
- if (iter == id_map_inverse_.end())
- return;
- id_map_.erase(iter->second);
- id_map_inverse_.erase(iter);
-}
-
-bool PreferenceModelAssociator::GetSyncIdForTaggedNode(const std::string& tag,
- int64* sync_id) {
- sync_api::ReadTransaction trans(sync_service_->GetUserShare());
- sync_api::ReadNode sync_node(&trans);
- if (!sync_node.InitByTagLookup(tag.c_str()))
- return false;
- *sync_id = sync_node.GetId();
- return true;
-}
-
-Value* PreferenceModelAssociator::MergePreference(
- const PrefService::Preference& local_pref,
- const Value& server_value) {
- const std::string& name(local_pref.name());
- if (name == prefs::kURLsToRestoreOnStartup ||
- name == prefs::kDesktopNotificationAllowedOrigins ||
- name == prefs::kDesktopNotificationDeniedOrigins) {
- return MergeListValues(*local_pref.GetValue(), server_value);
- }
-
- if (name == prefs::kContentSettingsPatterns ||
- name == prefs::kGeolocationContentSettings) {
- return MergeDictionaryValues(*local_pref.GetValue(), server_value);
- }
-
- // If this is not a specially handled preference, server wins.
- return server_value.DeepCopy();
-}
-
-bool PreferenceModelAssociator::WritePreferenceToNode(
- const std::string& name,
- const Value& value,
- sync_api::WriteNode* node) {
- std::string serialized;
- JSONStringValueSerializer json(&serialized);
- if (!json.Serialize(value)) {
- LOG(ERROR) << "Failed to serialize preference value.";
- return false;
- }
-
- sync_pb::PreferenceSpecifics preference;
- preference.set_name(name);
- preference.set_value(serialized);
- node->SetPreferenceSpecifics(preference);
- // TODO(viettrungluu): eliminate conversion (it's temporary)
- node->SetTitle(UTF8ToWide(name));
- return true;
-}
-
-Value* PreferenceModelAssociator::MergeListValues(const Value& from_value,
- const Value& to_value) {
- if (from_value.GetType() == Value::TYPE_NULL)
- return to_value.DeepCopy();
- if (to_value.GetType() == Value::TYPE_NULL)
- return from_value.DeepCopy();
-
- DCHECK(from_value.GetType() == Value::TYPE_LIST);
- DCHECK(to_value.GetType() == Value::TYPE_LIST);
- const ListValue& from_list_value = static_cast<const ListValue&>(from_value);
- const ListValue& to_list_value = static_cast<const ListValue&>(to_value);
- ListValue* result = to_list_value.DeepCopy();
-
- for (ListValue::const_iterator i = from_list_value.begin();
- i != from_list_value.end(); ++i) {
- Value* value = (*i)->DeepCopy();
- result->AppendIfNotPresent(value);
- }
- return result;
-}
-
-Value* PreferenceModelAssociator::MergeDictionaryValues(
- const Value& from_value,
- const Value& to_value) {
- if (from_value.GetType() == Value::TYPE_NULL)
- return to_value.DeepCopy();
- if (to_value.GetType() == Value::TYPE_NULL)
- return from_value.DeepCopy();
-
- DCHECK(from_value.GetType() == Value::TYPE_DICTIONARY);
- DCHECK(to_value.GetType() == Value::TYPE_DICTIONARY);
- const DictionaryValue& from_dict_value =
- static_cast<const DictionaryValue&>(from_value);
- const DictionaryValue& to_dict_value =
- static_cast<const DictionaryValue&>(to_value);
- DictionaryValue* result = to_dict_value.DeepCopy();
-
- for (DictionaryValue::key_iterator key = from_dict_value.begin_keys();
- key != from_dict_value.end_keys(); ++key) {
- Value* from_value;
- bool success = from_dict_value.GetWithoutPathExpansion(*key, &from_value);
- DCHECK(success);
-
- Value* to_key_value;
- if (result->GetWithoutPathExpansion(*key, &to_key_value)) {
- if (to_key_value->GetType() == Value::TYPE_DICTIONARY) {
- Value* merged_value = MergeDictionaryValues(*from_value, *to_key_value);
- result->SetWithoutPathExpansion(*key, merged_value);
- }
- // Note that for all other types we want to preserve the "to"
- // values so we do nothing here.
- } else {
- result->SetWithoutPathExpansion(*key, from_value->DeepCopy());
- }
- }
- return result;
-}
-
-void PreferenceModelAssociator::AfterUpdateOperations(
- const std::string& pref_name) {
- // The bookmark bar visibility preference requires a special
- // notification to update the UI.
- if (0 == pref_name.compare(prefs::kShowBookmarkBar)) {
- NotificationService::current()->Notify(
- NotificationType::BOOKMARK_BAR_VISIBILITY_PREF_CHANGED,
- Source<PreferenceModelAssociator>(this),
- NotificationService::NoDetails());
- }
-}
-
-bool PreferenceModelAssociator::CryptoReadyIfNecessary() {
- // We only access the cryptographer while holding a transaction.
- sync_api::ReadTransaction trans(sync_service_->GetUserShare());
- syncable::ModelTypeSet encrypted_types;
- sync_service_->GetEncryptedDataTypes(&encrypted_types);
- return encrypted_types.count(syncable::PREFERENCES) == 0 ||
- sync_service_->IsCryptographerReady(&trans);
-}
-
-} // namespace browser_sync

Powered by Google App Engine
This is Rietveld 408576698