Switch to using the new Link Doctor API.

chrome/browser/google/google_util.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/google/google_util.h"
 
 #include <string>
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/strings/string16.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/google/google_url_tracker.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/installer/util/google_update_settings.h"
 #include "net/base/registry_controlled_domains/registry_controlled_domain.h"
 #include "net/base/url_util.h"
 #include "url/gurl.h"
 
 #if defined(OS_MACOSX)
 #include "chrome/browser/mac/keystone_glue.h"
 #elif defined(OS_CHROMEOS)
 #include "chrome/browser/google/google_util_chromeos.h"
 #endif
 
+// Only use Link Doctor on official builds.  It uses an API key, too, but
+// seems best to just disable it, for more responsive error pages and to reduce
+// server load.
 #if defined(GOOGLE_CHROME_BUILD)
-#include "chrome/browser/google/linkdoctor_internal/linkdoctor_internal.h"
-#endif
-
-#ifndef LINKDOCTOR_SERVER_REQUEST_URL
-#define LINKDOCTOR_SERVER_REQUEST_URL std::string()
+#define LINKDOCTOR_SERVER_REQUEST_URL "https://www.googleapis.com/rpc"
+#else
+#define LINKDOCTOR_SERVER_REQUEST_URL ""
 #endif
 
 
 // Helpers --------------------------------------------------------------------
 
 namespace {
 
 const char* brand_for_testing = NULL;
 bool gUseMockLinkDoctorBaseURLForTesting = false;
 
@@ skipping 21 matching lines @@
 GURL LinkDoctorBaseURL() {
   if (gUseMockLinkDoctorBaseURLForTesting)
     return GURL("http://mock.linkdoctor.url/for?testing");
   return GURL(LINKDOCTOR_SERVER_REQUEST_URL);
 }
 
 void SetMockLinkDoctorBaseURLForTesting() {
   gUseMockLinkDoctorBaseURLForTesting = true;
 }
 
-GURL AppendGoogleLocaleParam(const GURL& url) {
+std::string GetGoogleLocale() {
   // Google does not yet recognize 'nb' for Norwegian Bokmal, but it uses

[Deprecated (see juliatuttle), 2014/02/08 01:51:31]

nit: Move this comment one line down, right above the if.  (Also, is there a
central place listing "Google locale vs. real locale" differences?)

[mmenke, 2014/02/08 01:57:49]

Done.

   // 'no' for that.
   std::string locale = g_browser_process->GetApplicationLocale();
   if (locale == "nb")
-    locale = "no";
-  return net::AppendQueryParameter(url, "hl", locale);
+    return "no";
+  return locale;
+}
+
+GURL AppendGoogleLocaleParam(const GURL& url) {
+  return net::AppendQueryParameter(url, "hl", GetGoogleLocale());
 }
 
 std::string StringAppendGoogleLocaleParam(const std::string& url) {
   GURL original_url(url);
   DCHECK(original_url.is_valid());
   GURL localized_url = AppendGoogleLocaleParam(original_url);
   return localized_url.spec();
 }
 
-GURL AppendGoogleTLDParam(Profile* profile, const GURL& url) {
-  const std::string google_domain(
-      net::registry_controlled_domains::GetDomainAndRegistry(
-          GoogleURLTracker::GoogleURL(profile),
-          net::registry_controlled_domains::EXCLUDE_PRIVATE_REGISTRIES));
-  const size_t first_dot = google_domain.find('.');
-  if (first_dot == std::string::npos) {
+std::string GetGoogleCountryCode(Profile* profile) {
+  const std::string google_hostname =
+      GoogleURLTracker::GoogleURL(profile).host();
+  const size_t last_dot = google_hostname.find_last_of('.');
+  if (last_dot == std::string::npos) {
     NOTREACHED();
-    return url;
   }
-  return net::AppendQueryParameter(url, "sd",
-                                   google_domain.substr(first_dot + 1));
+  std::string country_code = google_hostname.substr(last_dot + 1);
+  // Assume the com TLD implies the US.
+  if (country_code == "com")
+    return "us";
+  // Google uses the Unicode Common Locale Data Repository (CLDR), and the CLDR
+  // code for the UK is "gb".
+  if (country_code == "uk")
+    return "gb";
+  // Catalonia does not have a CLDR country code, since it's a region in Spain,
+  // so use Spain instead.
+  if (country_code == "cat")

[Deprecated (see juliatuttle), 2014/02/08 01:51:31]

...and of course nyan.cat is as expected.  I love the Internet.

+    return "es";
+  return country_code;
+}
+
+GURL GetGoogleSearchURL(Profile* profile) {
+  // The url returned by the tracker does not include the "/search" or the
+  // "q=" query string.
+  std::string search_path = "search";
+  std::string query_string = "q=";
+  GURL::Replacements replacements;
+  replacements.SetPathStr(search_path);
+  replacements.SetQueryStr(query_string);
+  return GoogleURLTracker::GoogleURL(profile).ReplaceComponents(replacements);
 }
 
 #if defined(OS_WIN)
 
 bool GetBrand(std::string* brand) {
   if (brand_for_testing) {
     brand->assign(brand_for_testing);
     return true;
   }
 
@@ skipping 171 matching lines @@
   DCHECK(brand_for_testing == NULL);
   brand_for_testing = brand_.c_str();
 }
 
 BrandForTesting::~BrandForTesting() {
   brand_for_testing = NULL;
 }
 
 
 }  // namespace google_util