Change behaivor to decide whether a search engine should be shown in the default list

components/search_engines/template_url_service.h

Index: components/search_engines/template_url_service.h
diff --git a/components/search_engines/template_url_service.h b/components/search_engines/template_url_service.h
index da7c5169a0f09b3d1ca09bff53e07c62fefc88f4..fd86a1215d99f447e3f63137956a08a16986f7c6 100644
--- a/components/search_engines/template_url_service.h
+++ b/components/search_engines/template_url_service.h
@@ -133,6 +133,12 @@ class TemplateURLService : public WebDataServiceConsumer,
                                   const GURL& url,
                                   TemplateURL** template_url_to_replace);
 
+  // Returns true if the template_url is in the default list. An TemplateURL is
+  // in the default list if it is the default search provider or its
+  // prepopulated
+  // id is greater than 0 or it is created by policy.

[Ian Wen, 2016/11/09 19:58:52]

Again, comments not well formatted.

[Peter Kasting, 2016/11/10 06:41:07]

Nit: How about this comment, which is more about why than what:

Returns whether |template_url| should be shown in the list of engines most
likely to be selected as a default engine.  This is meant to highlight the
current default, as well as the other most likely choices of default engine,
separately from a full list of all TemplateURLs (which might be very long).

[ltian, 2016/11/11 03:52:13]

Done.

[ltian, 2016/11/11 03:52:13]

Good point. I will add this in together with a description how to decide whether
a template_url should be in this list.

+  bool ShowInDefaultList(const TemplateURL* template_url);
+
   // Adds to |matches| all TemplateURLs whose keywords begin with |prefix|,
   // sorted shortest-keyword-first. If |supports_replacement_only| is true, only
   // TemplateURLs that support replacement are returned.