Index: components/search_engines/template_url_service.cc |
diff --git a/components/search_engines/template_url_service.cc b/components/search_engines/template_url_service.cc |
index 22e2db0b3fa16c320df785db633dc5f7ab4c8bfa..738a40e2b9764626568d03dc99f86e0f46ed467e 100644 |
--- a/components/search_engines/template_url_service.cc |
+++ b/components/search_engines/template_url_service.cc |
@@ -395,6 +395,12 @@ bool TemplateURLService::CanAddAutogeneratedKeyword( |
CanAddAutogeneratedKeywordForHost(url.host()); |
} |
+bool TemplateURLService::ShowInDefaultList(const TemplateURL* t_url) { |
+ return (t_url == default_search_provider_ || t_url->prepopulate_id() > 0 || |
+ t_url->created_by_policy()) && |
Peter Kasting
2016/11/14 19:59:16
Nit: Make sure git cl format would format things t
ltian
2016/11/15 19:43:41
Done.
|
+ t_url->SupportsReplacement(search_terms_data()); |
+} |
+ |
void TemplateURLService::AddMatchingKeywords( |
const base::string16& prefix, |
bool supports_replacement_only, |
@@ -477,8 +483,6 @@ TemplateURL* TemplateURLService::AddExtensionControlledTURL( |
DCHECK_EQ(kInvalidTemplateURLID, template_url->id()); |
DCHECK(info); |
DCHECK_NE(TemplateURL::NORMAL, template_url->type()); |
- DCHECK_EQ(info->wants_to_be_default_engine, |
- template_url->show_in_default_list()); |
DCHECK( |
!FindTemplateURLForExtension(info->extension_id, template_url->type())); |
template_url->extension_info_.swap(info); |
@@ -1244,7 +1248,6 @@ syncer::SyncData TemplateURLService::CreateSyncDataFromTemplateURL( |
se_specifics->set_date_created(turl.date_created().ToInternalValue()); |
se_specifics->set_input_encodings( |
base::JoinString(turl.input_encodings(), ";")); |
- se_specifics->set_show_in_default_list(turl.show_in_default_list()); |
se_specifics->set_suggestions_url(turl.suggestions_url()); |
se_specifics->set_prepopulate_id(turl.prepopulate_id()); |
se_specifics->set_instant_url(turl.instant_url()); |
@@ -1324,7 +1327,6 @@ TemplateURLService::CreateTemplateURLFromTemplateURLAndSyncData( |
data.instant_url_post_params = specifics.instant_url_post_params(); |
data.image_url_post_params = specifics.image_url_post_params(); |
data.favicon_url = GURL(specifics.favicon_url()); |
- data.show_in_default_list = specifics.show_in_default_list(); |
data.safe_for_autoreplace = specifics.safe_for_autoreplace(); |
data.input_encodings = base::SplitString( |
specifics.input_encodings(), ";", |
@@ -1641,8 +1643,7 @@ bool TemplateURLService::CanAddAutogeneratedKeywordForHost( |
} |
bool TemplateURLService::CanReplace(const TemplateURL* t_url) { |
- return (t_url != default_search_provider_ && !t_url->show_in_default_list() && |
- t_url->safe_for_autoreplace()); |
+ return !ShowInDefaultList(t_url) && t_url->safe_for_autoreplace(); |
} |
TemplateURL* TemplateURLService::FindNonExtensionTemplateURLForKeyword( |
@@ -1967,7 +1968,6 @@ bool TemplateURLService::ApplyDefaultSearchChangeNoMetrics( |
FindPrepopulatedTemplateURL(data->prepopulate_id); |
} |
TemplateURLData new_data(*data); |
- new_data.show_in_default_list = true; |
if (default_search_provider_) { |
UpdateNoNotify(default_search_provider_, TemplateURL(new_data)); |
} else { |