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

Unified Diff: chrome/browser/prefs/pref_value_map.cc

Issue 11243002: Move the bits of Prefs where production code has only trivially easy (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address review comments, fix gypi problem Created 8 years, 2 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
« no previous file with comments | « chrome/browser/prefs/pref_value_map.h ('k') | chrome/browser/prefs/pref_value_map_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prefs/pref_value_map.cc
diff --git a/chrome/browser/prefs/pref_value_map.cc b/chrome/browser/prefs/pref_value_map.cc
deleted file mode 100644
index 1968a143ce9086efeb469be6f4e89241bc2af9e2..0000000000000000000000000000000000000000
--- a/chrome/browser/prefs/pref_value_map.cc
+++ /dev/null
@@ -1,151 +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/prefs/pref_value_map.h"
-
-#include "base/logging.h"
-#include "base/memory/scoped_ptr.h"
-#include "base/stl_util.h"
-#include "base/values.h"
-
-PrefValueMap::PrefValueMap() {}
-
-PrefValueMap::~PrefValueMap() {
- Clear();
-}
-
-bool PrefValueMap::GetValue(const std::string& key, const Value** value) const {
- const Map::const_iterator entry = prefs_.find(key);
- if (entry != prefs_.end()) {
- if (value)
- *value = entry->second;
- return true;
- }
-
- return false;
-}
-
-bool PrefValueMap::GetValue(const std::string& key, Value** value) {
- const Map::const_iterator entry = prefs_.find(key);
- if (entry != prefs_.end()) {
- if (value)
- *value = entry->second;
- return true;
- }
-
- return false;
-}
-
-bool PrefValueMap::SetValue(const std::string& key, Value* value) {
- DCHECK(value);
- scoped_ptr<Value> value_ptr(value);
- const Map::iterator entry = prefs_.find(key);
- if (entry != prefs_.end()) {
- if (Value::Equals(entry->second, value))
- return false;
- delete entry->second;
- entry->second = value_ptr.release();
- } else {
- prefs_[key] = value_ptr.release();
- }
-
- return true;
-}
-
-bool PrefValueMap::RemoveValue(const std::string& key) {
- const Map::iterator entry = prefs_.find(key);
- if (entry != prefs_.end()) {
- delete entry->second;
- prefs_.erase(entry);
- return true;
- }
-
- return false;
-}
-
-void PrefValueMap::Clear() {
- STLDeleteValues(&prefs_);
- prefs_.clear();
-}
-
-void PrefValueMap::Swap(PrefValueMap* other) {
- prefs_.swap(other->prefs_);
-}
-
-PrefValueMap::iterator PrefValueMap::begin() {
- return prefs_.begin();
-}
-
-PrefValueMap::iterator PrefValueMap::end() {
- return prefs_.end();
-}
-
-PrefValueMap::const_iterator PrefValueMap::begin() const {
- return prefs_.begin();
-}
-
-PrefValueMap::const_iterator PrefValueMap::end() const {
- return prefs_.end();
-}
-
-bool PrefValueMap::GetBoolean(const std::string& key,
- bool* value) const {
- const Value* stored_value = NULL;
- return GetValue(key, &stored_value) && stored_value->GetAsBoolean(value);
-}
-
-void PrefValueMap::SetBoolean(const std::string& key, bool value) {
- SetValue(key, Value::CreateBooleanValue(value));
-}
-
-bool PrefValueMap::GetString(const std::string& key,
- std::string* value) const {
- const Value* stored_value = NULL;
- return GetValue(key, &stored_value) && stored_value->GetAsString(value);
-}
-
-void PrefValueMap::SetString(const std::string& key,
- const std::string& value) {
- SetValue(key, Value::CreateStringValue(value));
-}
-
-bool PrefValueMap::GetInteger(const std::string& key, int* value) const {
- const Value* stored_value = NULL;
- return GetValue(key, &stored_value) && stored_value->GetAsInteger(value);
-}
-
-void PrefValueMap::SetInteger(const std::string& key, const int value) {
- SetValue(key, Value::CreateIntegerValue(value));
-}
-
-void PrefValueMap::GetDifferingKeys(
- const PrefValueMap* other,
- std::vector<std::string>* differing_keys) const {
- differing_keys->clear();
-
- // Walk over the maps in lockstep, adding everything that is different.
- Map::const_iterator this_pref(prefs_.begin());
- Map::const_iterator other_pref(other->prefs_.begin());
- while (this_pref != prefs_.end() && other_pref != other->prefs_.end()) {
- const int diff = this_pref->first.compare(other_pref->first);
- if (diff == 0) {
- if (!this_pref->second->Equals(other_pref->second))
- differing_keys->push_back(this_pref->first);
- ++this_pref;
- ++other_pref;
- } else if (diff < 0) {
- differing_keys->push_back(this_pref->first);
- ++this_pref;
- } else if (diff > 0) {
- differing_keys->push_back(other_pref->first);
- ++other_pref;
- }
- }
-
- // Add the remaining entries.
- for ( ; this_pref != prefs_.end(); ++this_pref)
- differing_keys->push_back(this_pref->first);
- for ( ; other_pref != other->prefs_.end(); ++other_pref)
- differing_keys->push_back(other_pref->first);
-}
« no previous file with comments | « chrome/browser/prefs/pref_value_map.h ('k') | chrome/browser/prefs/pref_value_map_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698