Remove the GoogleURLTracker infobar functionality entirely.

components/google/core/browser/google_util.cc

 // Copyright 2014 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 "components/google/core/browser/google_util.h"
 
 #include <string>
 #include <vector>
 
 #include "base/command_line.h"
@@ skipping 61 matching lines @@
 }
 
 }  // namespace
 
 
 namespace google_util {
 
 // Global functions -----------------------------------------------------------
 
 bool HasGoogleSearchQueryParam(const std::string& str) {
-  url::Component query(0, str.length()), key, value;
+  url::Component query(0, static_cast<int>(str.length())), key, value;

[Ilya Sherman, 2015/01/06 00:39:58]

nit: Would checked_cast be better?

[Peter Kasting, 2015/01/06 00:54:26]

I don't think it's necessary.  This just makes explicit a conversion that was
previously implicit, and which we have in many other places; if we can ever get
strings > 2 GB in parts of the UI a whole bunch of code is going to break.  From
glancing at ExtractQueryKeyValue() I think the consequence in that case is just
going to be that this doesn't extract anything anyway, which seems like a fine
failure mode.

   while (url::ExtractQueryKeyValue(str.c_str(), &query, &key, &value)) {
     if ((key.len == 1) && (str[key.begin] == 'q') && value.is_nonempty())
       return true;
   }
   return false;
 }
 
 GURL LinkDoctorBaseURL() {
   if (gUseMockLinkDoctorBaseURLForTesting)
     return GURL("http://mock.linkdoctor.url/for?testing");
@@ skipping 115 matching lines @@
 }
 
 bool IsYoutubeDomainUrl(const GURL& url,
                         SubdomainPermission subdomain_permission,
                         PortPermission port_permission) {
   return IsValidURL(url, port_permission) &&
       IsValidHostName(url.host(), "youtube", subdomain_permission);
 }
 
 }  // namespace google_util