diff options
-rw-r--r-- | base/values.cc | 30 | ||||
-rw-r--r-- | base/values.h | 8 | ||||
-rw-r--r-- | base/values_unittest.cc | 3 | ||||
-rw-r--r-- | chrome/browser/chromeos/login/user_manager.cc | 3 | ||||
-rw-r--r-- | chrome/browser/geolocation/geolocation_content_settings_map.cc | 2 | ||||
-rw-r--r-- | chrome/browser/host_zoom_map_unittest.cc | 11 |
6 files changed, 6 insertions, 51 deletions
diff --git a/base/values.cc b/base/values.cc index c4f33c9..1a1695c 100644 --- a/base/values.cc +++ b/base/values.cc @@ -747,36 +747,6 @@ bool DictionaryValue::GetListWithoutPathExpansion(const std::string& key, return true; } -// TODO(viettrungluu): Deprecated and to be removed: -bool DictionaryValue::GetWithoutPathExpansion(const std::wstring& key, - Value** out_value) const { - return GetWithoutPathExpansion(WideToUTF8(key), out_value); -} - -// TODO(viettrungluu): Deprecated and to be removed: -bool DictionaryValue::GetIntegerWithoutPathExpansion(const std::wstring& key, - int* out_value) const { - return GetIntegerWithoutPathExpansion(WideToUTF8(key), out_value); -} - -// TODO(viettrungluu): Deprecated and to be removed: -bool DictionaryValue::GetStringWithoutPathExpansion( - const std::wstring& key, - std::string* out_value) const { - return GetStringWithoutPathExpansion(WideToUTF8(key), out_value); -} - -// TODO(viettrungluu): Deprecated and to be removed: -bool DictionaryValue::GetStringWithoutPathExpansion( - const std::wstring& key, - std::wstring* out_value) const { - Value* value; - if (!GetWithoutPathExpansion(WideToUTF8(key), &value)) - return false; - - return value->GetAsString(out_value); -} - bool DictionaryValue::Remove(const std::string& path, Value** out_value) { DCHECK(IsStringUTF8(path)); std::string current_path(path); diff --git a/base/values.h b/base/values.h index a750422..54230c9 100644 --- a/base/values.h +++ b/base/values.h @@ -323,14 +323,6 @@ class DictionaryValue : public Value { DictionaryValue** out_value) const; bool GetListWithoutPathExpansion(const std::string& key, ListValue** out_value) const; - /*DEPRECATED*/bool GetWithoutPathExpansion(const std::wstring& key, - Value** out_value) const; - /*DEPRECATED*/bool GetIntegerWithoutPathExpansion(const std::wstring& key, - int* out_value) const; - /*DEPRECATED*/bool GetStringWithoutPathExpansion( - const std::wstring& key, std::string* out_value) const; - /*DEPRECATED*/bool GetStringWithoutPathExpansion( - const std::wstring& key, std::wstring* out_value) const; // Removes the Value with the specified path from this dictionary (or one // of its child dictionaries, if the path is more than just a local key). diff --git a/base/values_unittest.cc b/base/values_unittest.cc index 82d6082..81bd675 100644 --- a/base/values_unittest.cc +++ b/base/values_unittest.cc @@ -508,9 +508,6 @@ TEST_F(ValuesTest, DictionaryWithoutPathExpansionDeprecated) { EXPECT_TRUE(dict.HasKey(L"this.isnt.expanded")); Value* value3; EXPECT_FALSE(dict.Get(L"this.isnt.expanded", &value3)); - Value* value4; - ASSERT_TRUE(dict.GetWithoutPathExpansion(L"this.isnt.expanded", &value4)); - EXPECT_EQ(Value::TYPE_NULL, value4->GetType()); } TEST_F(ValuesTest, DeepCopy) { diff --git a/chrome/browser/chromeos/login/user_manager.cc b/chrome/browser/chromeos/login/user_manager.cc index baba41a..ca8cb5a 100644 --- a/chrome/browser/chromeos/login/user_manager.cc +++ b/chrome/browser/chromeos/login/user_manager.cc @@ -130,8 +130,7 @@ std::vector<UserManager::User> UserManager::GetUsers() const { std::string image_path; if (image_it == user_images_.end()) { if (prefs_images && - prefs_images->GetStringWithoutPathExpansion( - ASCIIToWide(email), &image_path)) { + prefs_images->GetStringWithoutPathExpansion(email, &image_path)) { // Insert the default image so we don't send another request if // GetUsers is called twice. user_images_[email] = user.image(); diff --git a/chrome/browser/geolocation/geolocation_content_settings_map.cc b/chrome/browser/geolocation/geolocation_content_settings_map.cc index b9e84dc5..d734a24 100644 --- a/chrome/browser/geolocation/geolocation_content_settings_map.cc +++ b/chrome/browser/geolocation/geolocation_content_settings_map.cc @@ -70,7 +70,7 @@ ContentSetting GeolocationContentSettingsMap::GetContentSetting( requesting_origin.spec(), &requesting_origin_settings)) { int setting; if (requesting_origin_settings->GetIntegerWithoutPathExpansion( - UTF8ToWide(embedding_origin.spec()), &setting)) + embedding_origin.spec(), &setting)) return IntToContentSetting(setting); // Check for any-embedder setting if (requesting_origin != embedding_origin && diff --git a/chrome/browser/host_zoom_map_unittest.cc b/chrome/browser/host_zoom_map_unittest.cc index 4b20a09..bd503ba 100644 --- a/chrome/browser/host_zoom_map_unittest.cc +++ b/chrome/browser/host_zoom_map_unittest.cc @@ -63,8 +63,7 @@ TEST_F(HostZoomMapTest, LoadNoPrefs) { TEST_F(HostZoomMapTest, Load) { DictionaryValue* dict = prefs_->GetMutableDictionary(prefs::kPerHostZoomLevels); - dict->SetWithoutPathExpansion(UTF8ToWide(host_), - Value::CreateIntegerValue(kZoomLevel)); + dict->SetWithoutPathExpansion(host_, Value::CreateIntegerValue(kZoomLevel)); scoped_refptr<HostZoomMap> map(new HostZoomMap(&profile_)); EXPECT_EQ(kZoomLevel, map->GetZoomLevel(url_)); } @@ -78,14 +77,13 @@ TEST_F(HostZoomMapTest, SetZoomLevel) { const DictionaryValue* dict = prefs_->GetDictionary(prefs::kPerHostZoomLevels); int zoom_level = 0; - EXPECT_TRUE(dict->GetIntegerWithoutPathExpansion(UTF8ToWide(host_), - &zoom_level)); + EXPECT_TRUE(dict->GetIntegerWithoutPathExpansion(host_, &zoom_level)); EXPECT_EQ(kZoomLevel, zoom_level); SetPrefObserverExpectation(); map->SetZoomLevel(url_, 0); EXPECT_EQ(0, map->GetZoomLevel(url_)); - EXPECT_FALSE(dict->HasKey(UTF8ToWide(host_))); + EXPECT_FALSE(dict->HasKey(host_)); prefs_->RemovePrefObserver(prefs::kPerHostZoomLevels, &pref_observer_); } @@ -106,8 +104,7 @@ TEST_F(HostZoomMapTest, ReloadOnPrefChange) { map->SetZoomLevel(url_, kZoomLevel); DictionaryValue dict; - dict.SetWithoutPathExpansion(UTF8ToWide(host_), - Value::CreateIntegerValue(0)); + dict.SetWithoutPathExpansion(host_, Value::CreateIntegerValue(0)); prefs_->Set(prefs::kPerHostZoomLevels, dict); EXPECT_EQ(0, map->GetZoomLevel(url_)); } |