Reject some previuosly-escaped chars in hostnames.

components/omnibox/browser/history_url_provider_unittest.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 "components/omnibox/browser/history_url_provider.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <memory>
@@ skipping 533 matching lines @@
   const UrlAndLegalDefault results_1[] = {
     { "http://wytmatch/", true }
   };
   RunTest(ASCIIToUTF16("wytmatch"), std::string(), false, results_1,
           arraysize(results_1));
 
   RunTest(ASCIIToUTF16("wytmatch foo bar"), std::string(), false, NULL, 0);
   RunTest(ASCIIToUTF16("wytmatch+foo+bar"), std::string(), false, NULL, 0);
 
   const UrlAndLegalDefault results_2[] = {
-    { "http://wytmatch+foo+bar.com/", true }
+    { "http://wytmatch_foo_bar.com/", true }
   };
-  RunTest(ASCIIToUTF16("wytmatch+foo+bar.com"), std::string(), false,
+  RunTest(ASCIIToUTF16("wytmatch_foo_bar.com"), std::string(), false,
           results_2, arraysize(results_2));
 }
 
 TEST_F(HistoryURLProviderTest, WhatYouTyped) {
   // Make sure we suggest a What You Typed match at the right times.
   RunTest(ASCIIToUTF16("wytmatch"), std::string(), false, NULL, 0);
   RunTest(ASCIIToUTF16("wytmatch foo bar"), std::string(), false, NULL, 0);
   RunTest(ASCIIToUTF16("wytmatch+foo+bar"), std::string(), false, NULL, 0);
   RunTest(ASCIIToUTF16("wytmatch+foo+bar.com"), std::string(), false, NULL, 0);
 
   const UrlAndLegalDefault results_1[] = {
     { "http://www.wytmatch.com/", true }
   };
   RunTest(ASCIIToUTF16("wytmatch"), "com", false, results_1,
           arraysize(results_1));
 
   const UrlAndLegalDefault results_2[] = {
-    { "http://wytmatch%20foo%20bar/", false }
+    { "http://wytmatch/", true }
   };
-  RunTest(ASCIIToUTF16("http://wytmatch foo bar"), std::string(), false,
+  RunTest(ASCIIToUTF16("http://wytmatch"), std::string(), false,
           results_2, arraysize(results_2));
-
-  const UrlAndLegalDefault results_3[] = {
-    { "https://wytmatch%20foo%20bar/", false }
-  };
-  RunTest(ASCIIToUTF16("https://wytmatch foo bar"), std::string(), false,
-          results_3, arraysize(results_3));
 }
 
 TEST_F(HistoryURLProviderTest, Fixup) {
   // Test for various past crashes we've had.
   RunTest(ASCIIToUTF16("\\"), std::string(), false, NULL, 0);
   RunTest(ASCIIToUTF16("#"), std::string(), false, NULL, 0);
   RunTest(ASCIIToUTF16("%20"), std::string(), false, NULL, 0);
   const UrlAndLegalDefault fixup_crash[] = {
     { "http://%EF%BD%A5@s/", false }
   };
@@ skipping 524 matching lines @@
     // the default experimental scoring.
     autocomplete_->scoring_params_.experimental_scoring_enabled = true;
     ASSERT_NO_FATAL_FAILURE(RunTest(ASCIIToUTF16(test_cases[i].input),
                                     std::string(), false, output, max_matches));
     for (int j = 0; j < max_matches; ++j) {
       EXPECT_EQ(test_cases[i].matches[j].experiment_relevance,
                 matches_[j].relevance);
     }
   }
 }