Reject some previuosly-escaped chars in hostnames.

components/safe_browsing_db/v4_protocol_manager_util.cc

 // Copyright 2016 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/safe_browsing_db/v4_protocol_manager_util.h"
 
 #include "base/base64.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/rand_util.h"
 #include "base/strings/string_util.h"
@@ skipping 330 matching lines @@
       urls->push_back(hosts[h] + paths[p]);
     }
   }
 }
 
 // static
 void V4ProtocolManagerUtil::CanonicalizeUrl(const GURL& url,
                                             std::string* canonicalized_hostname,
                                             std::string* canonicalized_path,
                                             std::string* canonicalized_query) {
-  DCHECK(url.is_valid());
+  DCHECK(url.is_valid()) << url.possibly_invalid_spec();
 
   // We only canonicalize "normal" URLs.
   if (!url.IsStandard())
     return;
 
   // Following canonicalization steps are excluded since url parsing takes care
   // of those :-
   // 1. Remove any tab (0x09), CR (0x0d), and LF (0x0a) chars from url.
   //    (Exclude escaped version of these chars).
   // 2. Normalize hostname to 4 dot-seperated decimal values.
@@ skipping 160 matching lines @@
 // static
 void V4ProtocolManagerUtil::SetClientInfoFromConfig(
     ClientInfo* client_info,
     const V4ProtocolConfig& config) {
   DCHECK(client_info);
   client_info->set_client_id(config.client_name);
   client_info->set_client_version(config.version);
 }
 
 }  // namespace safe_browsing