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

Unified Diff: components/search_engines/default_search_manager_unittest.cc

Issue 1135163002: Omnibox - Strip Extra Whitespace from Custom Search Engine Names (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix more tests that don't set short_name Created 5 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
« no previous file with comments | « components/search_engines/default_search_manager.cc ('k') | components/search_engines/keyword_table.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/search_engines/default_search_manager_unittest.cc
diff --git a/components/search_engines/default_search_manager_unittest.cc b/components/search_engines/default_search_manager_unittest.cc
index a6718d63ed8bd8ff3a6a92747c5070139eb4355f..02e15a69c80ce69181e779b056b8ca05a331a5a4 100644
--- a/components/search_engines/default_search_manager_unittest.cc
+++ b/components/search_engines/default_search_manager_unittest.cc
@@ -30,7 +30,7 @@ void ExpectSimilar(const TemplateURLData* expected,
ASSERT_TRUE(expected != NULL);
ASSERT_TRUE(actual != NULL);
- EXPECT_EQ(expected->short_name, actual->short_name);
+ EXPECT_EQ(expected->short_name(), actual->short_name());
EXPECT_EQ(expected->keyword(), actual->keyword());
EXPECT_EQ(expected->url(), actual->url());
EXPECT_EQ(expected->suggestions_url, actual->suggestions_url);
@@ -86,7 +86,7 @@ void SetPolicy(user_prefs::TestingPrefServiceSyncable* prefs,
EXPECT_FALSE(data->url().empty());
}
scoped_ptr<base::DictionaryValue> entry(new base::DictionaryValue);
- entry->SetString(DefaultSearchManager::kShortName, data->short_name);
+ entry->SetString(DefaultSearchManager::kShortName, data->short_name());
entry->SetString(DefaultSearchManager::kKeyword, data->keyword());
entry->SetString(DefaultSearchManager::kURL, data->url());
entry->SetString(DefaultSearchManager::kFaviconURL, data->favicon_url.spec());
@@ -120,7 +120,7 @@ void SetPolicy(user_prefs::TestingPrefServiceSyncable* prefs,
scoped_ptr<TemplateURLData> GenerateDummyTemplateURLData(std::string type) {
scoped_ptr<TemplateURLData> data(new TemplateURLData());
- data->short_name = base::UTF8ToUTF16(std::string(type).append("name"));
+ data->SetShortName(base::UTF8ToUTF16(std::string(type).append("name")));
data->SetKeyword(base::UTF8ToUTF16(std::string(type).append("key")));
data->SetURL(std::string("http://").append(type).append("foo/{searchTerms}"));
data->suggestions_url = std::string("http://").append(type).append("sugg");
@@ -162,7 +162,7 @@ TEST_F(DefaultSearchManagerTest, ReadAndWritePref) {
DefaultSearchManager manager(pref_service(),
DefaultSearchManager::ObserverCallback());
TemplateURLData data;
- data.short_name = base::UTF8ToUTF16("name1");
+ data.SetShortName(base::UTF8ToUTF16("name1"));
data.SetKeyword(base::UTF8ToUTF16("key1"));
data.SetURL("http://foo1/{searchTerms}");
data.suggestions_url = "http://sugg1";
@@ -242,8 +242,8 @@ TEST_F(DefaultSearchManagerTest, DefaultSearchSetByOverrides) {
ExpectSimilar(prepopulated_urls[default_search_index],
manager.GetDefaultSearchEngine(&source));
EXPECT_EQ(DefaultSearchManager::FROM_FALLBACK, source);
- EXPECT_NE(manager.GetDefaultSearchEngine(NULL)->short_name,
- first_default.short_name);
+ EXPECT_NE(manager.GetDefaultSearchEngine(NULL)->short_name(),
+ first_default.short_name());
EXPECT_NE(manager.GetDefaultSearchEngine(NULL)->keyword(),
first_default.keyword());
}
« no previous file with comments | « components/search_engines/default_search_manager.cc ('k') | components/search_engines/keyword_table.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698