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

Side by Side Diff: components/search_engines/default_search_manager_unittest.cc

Issue 2487633003: Change behaivor to decide whether a search engine should be shown in the default list (Closed)
Patch Set: Update based on Peter and Nicolas's comments. Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/search_engines/default_search_manager.h" 5 #include "components/search_engines/default_search_manager.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <memory> 9 #include <memory>
10 10
(...skipping 23 matching lines...) Expand all
34 const TemplateURLData* actual) { 34 const TemplateURLData* actual) {
35 ASSERT_TRUE(expected != NULL); 35 ASSERT_TRUE(expected != NULL);
36 ASSERT_TRUE(actual != NULL); 36 ASSERT_TRUE(actual != NULL);
37 37
38 EXPECT_EQ(expected->short_name(), actual->short_name()); 38 EXPECT_EQ(expected->short_name(), actual->short_name());
39 EXPECT_EQ(expected->keyword(), actual->keyword()); 39 EXPECT_EQ(expected->keyword(), actual->keyword());
40 EXPECT_EQ(expected->url(), actual->url()); 40 EXPECT_EQ(expected->url(), actual->url());
41 EXPECT_EQ(expected->suggestions_url, actual->suggestions_url); 41 EXPECT_EQ(expected->suggestions_url, actual->suggestions_url);
42 EXPECT_EQ(expected->favicon_url, actual->favicon_url); 42 EXPECT_EQ(expected->favicon_url, actual->favicon_url);
43 EXPECT_EQ(expected->alternate_urls, actual->alternate_urls); 43 EXPECT_EQ(expected->alternate_urls, actual->alternate_urls);
44 EXPECT_EQ(expected->show_in_default_list, actual->show_in_default_list);
45 EXPECT_EQ(expected->safe_for_autoreplace, actual->safe_for_autoreplace); 44 EXPECT_EQ(expected->safe_for_autoreplace, actual->safe_for_autoreplace);
46 EXPECT_EQ(expected->input_encodings, actual->input_encodings); 45 EXPECT_EQ(expected->input_encodings, actual->input_encodings);
47 EXPECT_EQ(expected->search_terms_replacement_key, 46 EXPECT_EQ(expected->search_terms_replacement_key,
48 actual->search_terms_replacement_key); 47 actual->search_terms_replacement_key);
49 } 48 }
50 49
51 // TODO(caitkp): TemplateURLData-ify this. 50 // TODO(caitkp): TemplateURLData-ify this.
52 void SetOverrides(user_prefs::TestingPrefServiceSyncable* prefs, bool update) { 51 void SetOverrides(user_prefs::TestingPrefServiceSyncable* prefs, bool update) {
53 prefs->SetUserPref(prefs::kSearchProviderOverridesVersion, 52 prefs->SetUserPref(prefs::kSearchProviderOverridesVersion,
54 new base::FundamentalValue(1)); 53 new base::FundamentalValue(1));
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
127 const std::string& type) { 126 const std::string& type) {
128 std::unique_ptr<TemplateURLData> data(new TemplateURLData()); 127 std::unique_ptr<TemplateURLData> data(new TemplateURLData());
129 data->SetShortName(base::UTF8ToUTF16(std::string(type).append("name"))); 128 data->SetShortName(base::UTF8ToUTF16(std::string(type).append("name")));
130 data->SetKeyword(base::UTF8ToUTF16(std::string(type).append("key"))); 129 data->SetKeyword(base::UTF8ToUTF16(std::string(type).append("key")));
131 data->SetURL(std::string("http://").append(type).append("foo/{searchTerms}")); 130 data->SetURL(std::string("http://").append(type).append("foo/{searchTerms}"));
132 data->suggestions_url = std::string("http://").append(type).append("sugg"); 131 data->suggestions_url = std::string("http://").append(type).append("sugg");
133 data->alternate_urls.push_back( 132 data->alternate_urls.push_back(
134 std::string("http://").append(type).append("foo/alt")); 133 std::string("http://").append(type).append("foo/alt"));
135 data->favicon_url = GURL("http://icon1"); 134 data->favicon_url = GURL("http://icon1");
136 data->safe_for_autoreplace = true; 135 data->safe_for_autoreplace = true;
137 data->show_in_default_list = true;
138 data->input_encodings = base::SplitString( 136 data->input_encodings = base::SplitString(
139 "UTF-8;UTF-16", ";", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL); 137 "UTF-8;UTF-16", ";", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
140 data->date_created = base::Time(); 138 data->date_created = base::Time();
141 data->last_modified = base::Time(); 139 data->last_modified = base::Time();
142 return data; 140 return data;
143 } 141 }
144 142
145 } // namespace 143 } // namespace
146 144
147 class DefaultSearchManagerTest : public testing::Test { 145 class DefaultSearchManagerTest : public testing::Test {
(...skipping 21 matching lines...) Expand all
169 DefaultSearchManager manager(pref_service(), 167 DefaultSearchManager manager(pref_service(),
170 DefaultSearchManager::ObserverCallback()); 168 DefaultSearchManager::ObserverCallback());
171 TemplateURLData data; 169 TemplateURLData data;
172 data.SetShortName(base::UTF8ToUTF16("name1")); 170 data.SetShortName(base::UTF8ToUTF16("name1"));
173 data.SetKeyword(base::UTF8ToUTF16("key1")); 171 data.SetKeyword(base::UTF8ToUTF16("key1"));
174 data.SetURL("http://foo1/{searchTerms}"); 172 data.SetURL("http://foo1/{searchTerms}");
175 data.suggestions_url = "http://sugg1"; 173 data.suggestions_url = "http://sugg1";
176 data.alternate_urls.push_back("http://foo1/alt"); 174 data.alternate_urls.push_back("http://foo1/alt");
177 data.favicon_url = GURL("http://icon1"); 175 data.favicon_url = GURL("http://icon1");
178 data.safe_for_autoreplace = true; 176 data.safe_for_autoreplace = true;
179 data.show_in_default_list = true;
180 data.input_encodings = base::SplitString( 177 data.input_encodings = base::SplitString(
181 "UTF-8;UTF-16", ";", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL); 178 "UTF-8;UTF-16", ";", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
182 data.date_created = base::Time(); 179 data.date_created = base::Time();
183 data.last_modified = base::Time(); 180 data.last_modified = base::Time();
184 181
185 manager.SetUserSelectedDefaultSearchEngine(data); 182 manager.SetUserSelectedDefaultSearchEngine(data);
186 TemplateURLData* read_data = manager.GetDefaultSearchEngine(NULL); 183 TemplateURLData* read_data = manager.GetDefaultSearchEngine(NULL);
187 ExpectSimilar(&data, read_data); 184 ExpectSimilar(&data, read_data);
188 } 185 }
189 186
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
324 321
325 ExpectSimilar(extension_data_3.get(), 322 ExpectSimilar(extension_data_3.get(),
326 manager.GetDefaultSearchEngine(&source)); 323 manager.GetDefaultSearchEngine(&source));
327 EXPECT_EQ(DefaultSearchManager::FROM_EXTENSION, source); 324 EXPECT_EQ(DefaultSearchManager::FROM_EXTENSION, source);
328 325
329 manager.ClearExtensionControlledDefaultSearchEngine(); 326 manager.ClearExtensionControlledDefaultSearchEngine();
330 327
331 ExpectSimilar(data.get(), manager.GetDefaultSearchEngine(&source)); 328 ExpectSimilar(data.get(), manager.GetDefaultSearchEngine(&source));
332 EXPECT_EQ(DefaultSearchManager::FROM_USER, source); 329 EXPECT_EQ(DefaultSearchManager::FROM_USER, source);
333 } 330 }
OLDNEW
« no previous file with comments | « components/search_engines/default_search_manager.cc ('k') | components/search_engines/default_search_pref_migration.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698