Index: chrome/browser/search_engines/template_url_service_unittest.cc |
diff --git a/chrome/browser/search_engines/template_url_service_unittest.cc b/chrome/browser/search_engines/template_url_service_unittest.cc |
index fe23c2ab3dca76f11b426b6e55e247f101b41242..daa34b62d1d1fb143adbd0237308e62704b1b214 100644 |
--- a/chrome/browser/search_engines/template_url_service_unittest.cc |
+++ b/chrome/browser/search_engines/template_url_service_unittest.cc |
@@ -70,7 +70,7 @@ std::unique_ptr<TemplateURL> CreateKeywordWithDate( |
const std::string& alternate_url, |
const std::string& favicon_url, |
bool safe_for_autoreplace, |
- bool show_in_default_list, |
+ int prepopulate_id, |
const std::string& encodings, |
Time date_created, |
Time last_modified, |
@@ -84,7 +84,7 @@ std::unique_ptr<TemplateURL> CreateKeywordWithDate( |
data.alternate_urls.push_back(alternate_url); |
data.favicon_url = GURL(favicon_url); |
data.safe_for_autoreplace = safe_for_autoreplace; |
- data.show_in_default_list = show_in_default_list; |
+ data.prepopulate_id = prepopulate_id; |
data.input_encodings = base::SplitString( |
encodings, ";", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL); |
data.date_created = date_created; |
@@ -106,7 +106,7 @@ TemplateURL* AddKeywordWithDate( |
Time last_modified) { |
TemplateURL* t_url = model->Add(CreateKeywordWithDate( |
model, short_name, keyword, url, suggest_url, alternate_url, favicon_url, |
- safe_for_autoreplace, false, encodings, date_created, last_modified)); |
+ safe_for_autoreplace, 0, encodings, date_created, last_modified)); |
EXPECT_NE(0, t_url->id()); |
return t_url; |
} |
@@ -122,7 +122,6 @@ void ExpectSimilar(const TemplateURL* expected, const TemplateURL* actual) { |
EXPECT_EQ(expected->suggestions_url(), actual->suggestions_url()); |
EXPECT_EQ(expected->favicon_url(), actual->favicon_url()); |
EXPECT_EQ(expected->alternate_urls(), actual->alternate_urls()); |
- EXPECT_EQ(expected->show_in_default_list(), actual->show_in_default_list()); |
EXPECT_EQ(expected->safe_for_autoreplace(), actual->safe_for_autoreplace()); |
EXPECT_EQ(expected->input_encodings(), actual->input_encodings()); |
EXPECT_EQ(expected->search_terms_replacement_key(), |
@@ -233,7 +232,7 @@ void TemplateURLServiceTest::AssertEquals(const TemplateURL& expected, |
ASSERT_EQ(expected.suggestions_url(), actual.suggestions_url()); |
ASSERT_EQ(expected.favicon_url(), actual.favicon_url()); |
ASSERT_EQ(expected.alternate_urls(), actual.alternate_urls()); |
- ASSERT_EQ(expected.show_in_default_list(), actual.show_in_default_list()); |
+ ASSERT_EQ(expected.prepopulate_id(), actual.prepopulate_id()); |
ASSERT_EQ(expected.safe_for_autoreplace(), actual.safe_for_autoreplace()); |
ASSERT_EQ(expected.input_encodings(), actual.input_encodings()); |
ASSERT_EQ(expected.id(), actual.id()); |
@@ -260,7 +259,6 @@ std::unique_ptr<TemplateURL> TemplateURLServiceTest::CreatePreloadedTemplateURL( |
data.SetKeyword(ASCIIToUTF16("unittest")); |
data.SetURL("http://www.unittest.com/{searchTerms}"); |
data.favicon_url = GURL("http://favicon.url"); |
- data.show_in_default_list = true; |
data.safe_for_autoreplace = safe_for_autoreplace; |
data.input_encodings.push_back("UTF-8"); |
data.date_created = Time::FromTimeT(100); |
@@ -686,7 +684,7 @@ TEST_F(TemplateURLServiceTest, DefaultSearchProvider) { |
model()->SetUserSelectedDefaultSearchProvider(t_url); |
ASSERT_EQ(t_url, model()->GetDefaultSearchProvider()); |
ASSERT_TRUE(t_url->safe_for_autoreplace()); |
- ASSERT_TRUE(t_url->show_in_default_list()); |
+ ASSERT_TRUE(model()->ShowInDefaultList(t_url)); |
// Setting the default search provider should have caused notification. |
VerifyObserverCount(1); |
@@ -866,7 +864,7 @@ TEST_F(TemplateURLServiceTest, RepairSearchEnginesWithManagedDefault) { |
data.SetKeyword(ASCIIToUTF16(kKeyword)); |
data.SetURL(kSearchURL); |
data.favicon_url = GURL(kIconURL); |
- data.show_in_default_list = true; |
+ data.prepopulate_id = 999999; |
Peter Kasting
2016/11/10 06:41:06
Nit: Define a constant for this somewhere (kPrepop
ltian
2016/11/11 03:52:13
kNotPrepopulatedId might not be necessary because
|
data.input_encodings = base::SplitString( |
kEncodings, ";", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL); |
data.alternate_urls.push_back(kAlternateURL); |
@@ -1246,7 +1244,7 @@ TEST_F(TemplateURLServiceTest, TestManagedDefaultSearch) { |
data.SetKeyword(ASCIIToUTF16(kKeyword)); |
data.SetURL(kSearchURL); |
data.favicon_url = GURL(kIconURL); |
- data.show_in_default_list = true; |
+ data.prepopulate_id = 999999; |
data.input_encodings = base::SplitString( |
kEncodings, ";", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL); |
data.alternate_urls.push_back(kAlternateURL); |
@@ -1255,7 +1253,7 @@ TEST_F(TemplateURLServiceTest, TestManagedDefaultSearch) { |
const TemplateURL* actual_managed_default = |
model()->GetDefaultSearchProvider(); |
ExpectSimilar(expected_managed_default1.get(), actual_managed_default); |
- EXPECT_TRUE(actual_managed_default->show_in_default_list()); |
+ EXPECT_TRUE(model()->ShowInDefaultList(actual_managed_default)); |
// Update the managed preference and check that the model has changed. |
const char kNewName[] = "test2"; |
@@ -1275,12 +1273,12 @@ TEST_F(TemplateURLServiceTest, TestManagedDefaultSearch) { |
data2.SetKeyword(ASCIIToUTF16(kNewKeyword)); |
data2.SetURL(kNewSearchURL); |
data2.suggestions_url = kNewSuggestURL; |
- data2.show_in_default_list = true; |
+ data2.prepopulate_id = 999999; |
std::unique_ptr<TemplateURL> expected_managed_default2( |
new TemplateURL(data2)); |
actual_managed_default = model()->GetDefaultSearchProvider(); |
ExpectSimilar(expected_managed_default2.get(), actual_managed_default); |
- EXPECT_EQ(actual_managed_default->show_in_default_list(), true); |
+ EXPECT_EQ(model()->ShowInDefaultList(actual_managed_default), true); |
// Remove all the managed prefs and check that we are no longer managed. |
test_util()->RemoveManagedDefaultSearchPreferences(); |
@@ -1292,7 +1290,7 @@ TEST_F(TemplateURLServiceTest, TestManagedDefaultSearch) { |
const TemplateURL* actual_final_managed_default = |
model()->GetDefaultSearchProvider(); |
ExpectSimilar(regular_default, actual_final_managed_default); |
- EXPECT_EQ(actual_final_managed_default->show_in_default_list(), true); |
+ EXPECT_EQ(model()->ShowInDefaultList(actual_final_managed_default), true); |
// Disable the default search provider through policy. |
test_util()->SetManagedDefaultSearchPreferences(false, std::string(), |
@@ -1314,7 +1312,7 @@ TEST_F(TemplateURLServiceTest, TestManagedDefaultSearch) { |
// Verify that the default manager we are getting is the managed one. |
actual_managed_default = model()->GetDefaultSearchProvider(); |
ExpectSimilar(expected_managed_default1.get(), actual_managed_default); |
- EXPECT_EQ(actual_managed_default->show_in_default_list(), true); |
+ EXPECT_EQ(model()->ShowInDefaultList(actual_managed_default), true); |
// Clear the model and disable the default search provider through policy. |
// Verify that there is no default search provider after loading the model. |
@@ -1418,8 +1416,8 @@ TEST_F(TemplateURLServiceTest, DefaultExtensionEngine) { |
std::unique_ptr<TemplateURL> ext_dse = CreateKeywordWithDate( |
model(), "ext", "ext", "http://www.search.com/s?q={searchTerms}", |
- std::string(), std::string(), std::string(), true, true, "UTF-8", Time(), |
- Time(), TemplateURL::NORMAL_CONTROLLED_BY_EXTENSION); |
+ std::string(), std::string(), std::string(), true, 999999, "UTF-8", |
+ Time(), Time(), TemplateURL::NORMAL_CONTROLLED_BY_EXTENSION); |
std::unique_ptr<TemplateURL::AssociatedExtensionInfo> extension_info( |
new TemplateURL::AssociatedExtensionInfo("ext")); |
extension_info->wants_to_be_default_engine = true; |
@@ -1444,7 +1442,7 @@ TEST_F(TemplateURLServiceTest, ExtensionEnginesNotPersist) { |
std::unique_ptr<TemplateURL> ext_dse = CreateKeywordWithDate( |
model(), "ext1", "ext1", "http://www.ext1.com/s?q={searchTerms}", |
- std::string(), std::string(), std::string(), true, false, "UTF-8", Time(), |
+ std::string(), std::string(), std::string(), true, 0, "UTF-8", Time(), |
Time(), TemplateURL::NORMAL_CONTROLLED_BY_EXTENSION); |
std::unique_ptr<TemplateURL::AssociatedExtensionInfo> extension_info( |
new TemplateURL::AssociatedExtensionInfo("ext1")); |
@@ -1455,8 +1453,8 @@ TEST_F(TemplateURLServiceTest, ExtensionEnginesNotPersist) { |
ext_dse = CreateKeywordWithDate( |
model(), "ext2", "ext2", "http://www.ext2.com/s?q={searchTerms}", |
- std::string(), std::string(), std::string(), true, true, "UTF-8", Time(), |
- Time(), TemplateURL::NORMAL_CONTROLLED_BY_EXTENSION); |
+ std::string(), std::string(), std::string(), true, 999999, "UTF-8", |
+ Time(), Time(), TemplateURL::NORMAL_CONTROLLED_BY_EXTENSION); |
extension_info.reset(new TemplateURL::AssociatedExtensionInfo("ext2")); |
extension_info->wants_to_be_default_engine = true; |
TemplateURL* ext_dse_ptr = model()->AddExtensionControlledTURL( |
@@ -1489,7 +1487,6 @@ TEST_F(TemplateURLServiceTest, ExtensionEngineVsPolicy) { |
data.SetKeyword(ASCIIToUTF16(kKeyword)); |
data.SetURL(kSearchURL); |
data.favicon_url = GURL(kIconURL); |
- data.show_in_default_list = true; |
data.input_encodings = base::SplitString( |
kEncodings, ";", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL); |
data.alternate_urls.push_back(kAlternateURL); |
@@ -1502,8 +1499,8 @@ TEST_F(TemplateURLServiceTest, ExtensionEngineVsPolicy) { |
std::unique_ptr<TemplateURL> ext_dse = CreateKeywordWithDate( |
model(), "ext1", "ext1", "http://www.ext1.com/s?q={searchTerms}", |
- std::string(), std::string(), std::string(), true, true, "UTF-8", Time(), |
- Time(), TemplateURL::NORMAL_CONTROLLED_BY_EXTENSION); |
+ std::string(), std::string(), std::string(), true, 999999, "UTF-8", |
+ Time(), Time(), TemplateURL::NORMAL_CONTROLLED_BY_EXTENSION); |
std::unique_ptr<TemplateURL::AssociatedExtensionInfo> extension_info( |
new TemplateURL::AssociatedExtensionInfo("ext1")); |
extension_info->wants_to_be_default_engine = true; |