Omnibox: Fix URL-What-You-Typed Allowed-To-Be-Default-Match Issues

chrome/browser/autocomplete/search_provider_unittest.cc

 // Copyright 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/autocomplete/search_provider.h"
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
@@ skipping 2188 matching lines @@
     const bool allowed_to_be_default_match_in_prevent_inline_mode;
   } cases[] = {
     // Do not inline matches that do not contain the input; trim http as needed.
     { "x",                 "http://www.abc.com",
                                   "www.abc.com",  std::string(), false, false },
     { "https:",            "http://www.abc.com",
                                   "www.abc.com",  std::string(), false, false },
     { "http://www.abc.com/a", "http://www.abc.com",
                               "http://www.abc.com",  std::string(), false,
                                                                     false },
-    { "http://www.abc.com",   "https://www.abc.com",
-                              "https://www.abc.com", std::string(), false,
-                                                                    false },
-    { "http://abc.com",       "ftp://abc.com",
-                              "ftp://abc.com",       std::string(), false,
-                                                                    false },
-    { "https://www.abc.com",  "http://www.abc.com",
-                                     "www.abc.com",  std::string(), false,
-                                                                    false },
-    { "ftp://abc.com",        "http://abc.com",
-                                     "abc.com",      std::string(), false,
-                                                                    false },
 
     // Do not inline matches with invalid input prefixes; trim http as needed.
     { "ttp",              "http://www.abc.com",
                                  "www.abc.com", std::string(), false, false },
     { "://w",             "http://www.abc.com",
                                  "www.abc.com", std::string(), false, false },
     { "ww.",              "http://www.abc.com",
                                  "www.abc.com", std::string(), false, false },
     { ".ab",              "http://www.abc.com",
                                  "www.abc.com", std::string(), false, false },
@@ skipping 24 matching lines @@
                              "http://www.abc.com", std::string(), true, true },
 
     // Inputs with trailing whitespace should inline when possible.
     { "abc.com ",      "http://www.abc.com",
                               "www.abc.com",      std::string(), true,  true },
     { "abc.com/ ",     "http://www.abc.com",
                               "www.abc.com",      std::string(), true,  true },
     { "abc.com ",      "http://www.abc.com/bar",
                               "www.abc.com/bar",  "/bar",        false, false },
 
+    // A suggestion that's equivalent to what the input gets fixed up to
+    // should be inlined.
+    { "abc.com:",      "http://abc.com/",
+                              "abc.com",      "", true, true },
+    { "abc.com:",      "http://www.abc.com",
+                              "www.abc.com",  "", true, true },
+
     // Inline matches when the input is a leading substring of the scheme.
     { "h",             "http://www.abc.com",
                        "http://www.abc.com", "ttp://www.abc.com", true, false },
     { "http",          "http://www.abc.com",
                        "http://www.abc.com", "://www.abc.com",    true, false },
 
     // Inline matches when the input is a leading substring of the full URL.
     { "http:",             "http://www.abc.com",
                            "http://www.abc.com", "//www.abc.com", true, false },
     { "http://w",          "http://www.abc.com",
@@ skipping 923 matching lines @@
 
   // The expiration time is always updated.
   provider_->GetSessionToken();
   base::TimeTicks expiration_time_1 = provider_->token_expiration_time_;
   base::PlatformThread::Sleep(kSmallDelta);
   provider_->GetSessionToken();
   base::TimeTicks expiration_time_2 = provider_->token_expiration_time_;
   EXPECT_GT(expiration_time_2, expiration_time_1);
   EXPECT_GE(expiration_time_2, expiration_time_1 + kSmallDelta);
 }