Omnibox - Do Not Allow HTTP/HTTPS Equivalence if User Explicitly Entered A Scheme

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 "components/omnibox/search_provider.h"
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
@@ skipping 2477 matching lines @@
     { "a.com", "[\"a.com\",[\"info\"],[],[],"
                 "{\"google:suggestrelevance\":[9999]}]",
       { { "a.com", AutocompleteMatchType::SEARCH_WHAT_YOU_TYPED, true  },
         { "info",  AutocompleteMatchType::SEARCH_SUGGEST,        false },
         kEmptyMatch, kEmptyMatch } },
     { "a.com", "[\"a.com\",[\"info\"],[],[],"
                 "{\"google:suggestrelevance\":[9999]}]",
       { { "a.com", AutocompleteMatchType::SEARCH_WHAT_YOU_TYPED, true  },
         { "info",  AutocompleteMatchType::SEARCH_SUGGEST,        false },
         kEmptyMatch, kEmptyMatch } },
+
+    // Ensure that if the user explicitly enters a scheme, a navsuggest
+    // result for a URL with a different scheme is not inlineable.
+    { "http://a.com", "[\"http://a.com\","
+               "[\"http://a.com/1\", \"https://a.com/\"],[],[],"
+                "{\"google:suggesttype\":[\"NAVIGATION\", \"NAVIGATION\"],"
+                 "\"google:suggestrelevance\":[9000, 8000]}]",
+      { { "http://a.com/1", AutocompleteMatchType::NAVSUGGEST,   true  },
+        { "https://a.com", AutocompleteMatchType::NAVSUGGEST,    false },
+        { "http://a.com",   AutocompleteMatchType::SEARCH_WHAT_YOU_TYPED,
+                                                                 true  },
+        kEmptyMatch } },
   };
 
   for (size_t i = 0; i < arraysize(cases); ++i) {
     // Send the query twice in order to have a synchronous pass after the first
     // response is received.  This is necessary because SearchProvider doesn't
     // allow an asynchronous response to change the default match.
     for (size_t j = 0; j < 2; ++j) {
       QueryForInputAndWaitForFetcherResponses(
           ASCIIToUTF16(cases[i].input), false, cases[i].json, std::string());
     }
@@ skipping 1001 matching lines @@
   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);
 }
 
 TEST_F(SearchProviderTest, AnswersCache) {
-  AutocompleteResult result;
+  AutocompleteResult result(client_.get());
   ACMatches matches;
   AutocompleteMatch match1;
   match1.answer_contents = base::ASCIIToUTF16("m1");
   match1.answer_type = base::ASCIIToUTF16("2334");
   match1.fill_into_edit = base::ASCIIToUTF16("weather los angeles");
 
   AutocompleteMatch non_answer_match1;
   non_answer_match1.fill_into_edit = base::ASCIIToUTF16("weather laguna beach");
 
   // Test that an answer in the first slot populates the cache.
@@ skipping 66 matching lines @@
 }
 
 TEST_F(SearchProviderTest, DoesNotProvideOnFocus) {
   AutocompleteInput input(
       base::ASCIIToUTF16("f"), base::string16::npos, std::string(), GURL(),
       metrics::OmniboxEventProto::INVALID_SPEC, false, true, true, true, true,
       ChromeAutocompleteSchemeClassifier(&profile_));
   provider_->Start(input, false);
   EXPECT_TRUE(provider_->matches().empty());
 }