Add a TLD whitelist for Google Search URLs.

chrome/browser/android/url_utilities.cc

Index: chrome/browser/android/url_utilities.cc
diff --git a/chrome/browser/android/url_utilities.cc b/chrome/browser/android/url_utilities.cc
index cc7d3d1f63fb9a8ee1e22a72e24f1b250e482d16..584759a0de1b2d9d92fc1a70e27d5fe207c74498 100644
--- a/chrome/browser/android/url_utilities.cc
+++ b/chrome/browser/android/url_utilities.cc
@@ -26,6 +26,44 @@ static const char* const g_downloadable_schemes[] = {
 static const char* const g_fallback_valid_schemes[] = {
     "http", "https", nullptr };
 
+// TODO(mariakhomenko): figure out how to keep this list updated.

[battre (please use the other), 2016/11/11 22:54:10]

IsCanonicalHostGoogleHostname in components/google/core/browser/google_util.cc
could also benefit from this.

+static const char* const g_google_tld_list[] = {"ac", "ad", "ae", "af", "ag",
+    "al", "am", "as", "at", "aw", "az", "ba", "be", "bf", "bg", "bi", "biz",
+    "bj", "bm", "bn", "bo", "bs", "bt", "by", "bz", "ca", "cat", "cc", "cd",
+    "cf", "cg", "ch", "ci", "cl", "cm", "cn", "co", "co.ao", "co.at", "co.ba",
+    "co.bi", "co.bw", "co.ci", "co.ck", "co.cr", "co.gg", "co.gl", "co.gy",
+    "co.hu", "co.id", "co.il", "co.im", "co.in", "co.it", "co.je", "co.jp",
+    "co.ke", "co.kr", "co.ls", "co.ma", "co.mu", "co.mw", "co.mz", "co.nz",
+    "co.pn", "co.rs",  "co.th", "co.tt", "co.tz", "co.ua", "co.ug", "co.uk",
+    "co.uz", "co.ve", "co.vi", "co.za", "co.zm", "co.zw", "com", "com.af",
+    "com.ag", "com.ai", "com.ar", "com.au", "com.az", "com.bd", "com.bh",
+    "com.bi", "com.bn", "com.bo", "com.br", "com.bs", "com.by", "com.bz",
+    "com.cn", "com.co", "com.cu", "com.cy", "com.do", "com.dz", "com.ec",
+    "com.eg", "com.er", "com.et", "com.fj", "com.ge", "com.gh", "com.gi",
+    "com.gl", "com.gp", "com.gr", "com.gt", "com.gy", "com.hk", "com.hn",
+    "com.hr", "com.ht", "com.iq", "com.jm", "com.jo", "com.kg", "com.kh",
+    "com.ki", "com.kw", "com.kz", "com.lb", "com.lc", "com.lk", "com.lv",
+    "com.ly", "com.mk", "com.mm", "com.mt", "com.mu", "com.mw", "com.mx",
+    "com.my", "com.na", "com.nc", "com.nf", "com.ng", "com.ni", "com.np",
+    "com.nr", "com.om", "com.pa", "com.pe", "com.pg", "com.ph", "com.pk",
+    "com.pl", "com.pr", "com.ps", "com.pt", "com.py", "com.qa", "com.ru",
+    "com.sa", "com.sb", "com.sc", "com.sg", "com.sl", "com.sv", "com.tj",
+    "com.tm", "com.tn", "com.tr", "com.tt", "com.tw", "com.ua", "com.uy",
+    "com.uz", "com.vc", "com.ve", "com.vi", "com.vn", "com.ws", "cv", "cx",
+    "cz", "de", "dj", "dk", "dm", "do", "dz", "ec", "ee", "es", "eu", "fi",
+    "fm", "fr", "ga", "gd", "ge", "gf", "gg", "gl", "gm", "gp", "gr", "gw",
+    "gy", "hk", "hn", "hr", "ht", "hu", "ie", "im", "in", "info", "in.rs", "io",
+    "iq", "is", "it", "it.ao", "je", "jo", "jobs", "jp", "kg", "ki", "kids.us",
+    "km", "kn", "kr", "kz", "la", "li", "lk", "lt", "lu", "lv", "ma", "md",
+    "me", "mg", "mh", "mk", "ml", "mn", "mobi", "mr", "ms", "mu", "mv", "mw",
+    "mx", "name", "ne", "ne.jp", "net", "net.in", "net.nz", "nf", "ng", "nl",
+    "no", "nom.es", "nr", "nu", "off.ai", "org", "org.af", "org.es", "org.in",
+    "org.nz", "org.uk", "pf", "ph", "pk", "pl", "pn", "pr", "pro", "ps", "pt",
+    "qa", "re", "ro", "rs", "ru", "rw", "sc", "se", "sg", "sh", "si", "sk",
+    "sl", "sm", "sn", "so", "sr", "st", "sz", "td", "tel", "tg", "tk", "tl",
+    "tm", "tn", "to", "tt", "tv", "tw", "ua", "ug", "us", "uz", "vc", "vg",
+    "vn", "vu", "ws", "yt"};
+
 GURL ConvertJavaStringToGURL(JNIEnv* env, jstring url) {
   return url ? GURL(ConvertJavaStringToUTF8(env, url)) : GURL();
 }
@@ -103,7 +141,26 @@ static jboolean IsGoogleSearchUrl(JNIEnv* env,
   GURL gurl = ConvertJavaStringToGURL(env, url);
   if (gurl.is_empty())
     return false;
-  return google_util::IsGoogleSearchUrl(gurl);
+  bool is_search = google_util::IsGoogleSearchUrl(gurl);

[battre (please use the other), 2016/11/11 22:54:10]

Wouldn't it make sense to move the logic into this?

+  if (!is_search)
+    return is_search;
+
+  size_t length = net::registry_controlled_domains::GetRegistryLength(
+      gurl,
+      net::registry_controlled_domains::EXCLUDE_UNKNOWN_REGISTRIES,
+      net::registry_controlled_domains::EXCLUDE_PRIVATE_REGISTRIES);
+  if ((length == 0) || (length == std::string::npos))
+    return false;
+
+  std::string tld(gurl.host().substr(gurl.host().length() - length,
+                                     std::string::npos));
+  // TODO(mariakhomenko): binary search instead?
+  for (size_t i = 0; g_google_tld_list[i]; i++) {
+    if (g_google_tld_list[i] == tld) {
+      return true;
+    }
+  }
+  return false;
 }
 
 static jboolean IsGoogleHomePageUrl(JNIEnv* env,