Loosen RFC 1738 compliance check to allow underscores where we already allowe...

chrome/browser/autocomplete/autocomplete.cc

 // Copyright (c) 2009 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/autocomplete/autocomplete.h"
 
 #include <algorithm>
 
 #include "app/l10n_util.h"
 #include "base/basictypes.h"
@@ skipping 157 matching lines @@
     return QUERY;
 
   // Likewise, the RCDS can reject certain obviously-invalid hosts.  (We also
   // use the registry length later below.)
   const std::wstring host(text.substr(parts->host.begin, parts->host.len));
   const size_t registry_length =
       net::RegistryControlledDomainService::GetRegistryLength(host, false);
   if (registry_length == std::wstring::npos)
     return QUERY;  // Could be a broken IP address, etc.
 
-  // See if the hostname is valid per RFC 1738.  While IE and GURL allow
-  // hostnames to contain many other characters (perhaps for weird intranet
-  // machines), it's extremely unlikely that a user would be trying to type
-  // those in for anything other than a search query.
+  // See if the hostname is valid.  While IE and GURL allow hostnames to contain
+  // many other characters (perhaps for weird intranet machines), it's extremely
+  // unlikely that a user would be trying to type those in for anything other
+  // than a search query.
   url_canon::CanonHostInfo host_info;
   const std::string canonicalized_host(net::CanonicalizeHost(host, &host_info));
   if ((host_info.family == url_canon::CanonHostInfo::NEUTRAL) &&
-      !net::IsCanonicalizedHostRFC1738Compliant(canonicalized_host))
+      !net::IsCanonicalizedHostCompliant(canonicalized_host))
     return QUERY;
 
   // Presence of a port means this is likely a URL, if the port is really a port
   // number.  If it's just garbage after a colon, this is a query.
   if (parts->port.is_nonempty()) {
     int port;
     return (StringToInt(WideToUTF16(
                 text.substr(parts->port.begin, parts->port.len)), &port) &&
             (port >= 0) && (port <= 65535)) ? URL : QUERY;
   }
@@ skipping 745 matching lines @@
 void AutocompleteController::CheckIfDone() {
   for (ACProviders::const_iterator i(providers_.begin()); i != providers_.end();
        ++i) {
     if (!(*i)->done()) {
       done_ = false;
       return;
     }
   }
   done_ = true;
 }