[Android] Allow setting recently visited search engines as default search engine

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 329131160eb90b139b80d0da60195f9957b665a5..a6b516fa8f6949ab409aaf14f26b7ae6a72db6fd 100644
--- a/components/search_engines/template_url_service.h
+++ b/components/search_engines/template_url_service.h
@@ -387,6 +387,10 @@ class TemplateURLService : public WebDataServiceConsumer,
   static SyncDataMap CreateGUIDToSyncDataMap(
       const syncer::SyncDataList& sync_data);
 
+  // Like GetTemplateURLForKeyword(), but ignores extension-provided keywords.
+  TemplateURL* FindNonExtensionTemplateURLForKeyword(

[Ian Wen, 2016/10/17 23:32:21]

It looks like GetTemplateURLForKeyword is already a public method in this class
and I think we should directly use that instead.

There is no extension on Android.

[ltian, 2016/10/18 01:01:56]

Done.

+      const base::string16& keyword);
+
 #if defined(UNIT_TEST)
   void set_clock(std::unique_ptr<base::Clock> clock) {
     clock_ = std::move(clock);
@@ -522,10 +526,6 @@ class TemplateURLService : public WebDataServiceConsumer,
   // in the default list and is marked as safe_for_autoreplace.
   bool CanReplace(const TemplateURL* t_url);
 
-  // Like GetTemplateURLForKeyword(), but ignores extension-provided keywords.
-  TemplateURL* FindNonExtensionTemplateURLForKeyword(
-      const base::string16& keyword);
-
   // Updates the information in |existing_turl| using the information from
   // |new_values|, but the ID for |existing_turl| is retained.  Notifying
   // observers is the responsibility of the caller.  Returns whether