Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1538)

Unified Diff: chrome/browser/autocomplete/search_provider_unittest.cc

Issue 1098843004: Omnibox - Do Not Allow HTTP/HTTPS Equivalence if User Explicitly Entered A Scheme (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: refactored, as discussed Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/autocomplete/search_provider_unittest.cc
diff --git a/chrome/browser/autocomplete/search_provider_unittest.cc b/chrome/browser/autocomplete/search_provider_unittest.cc
index 8217327c3debba4305dabdfbfc6d9961082f38a0..95c96186a0a9c2ec5c2090109540d211884c4c78 100644
--- a/chrome/browser/autocomplete/search_provider_unittest.cc
+++ b/chrome/browser/autocomplete/search_provider_unittest.cc
@@ -2500,6 +2500,18 @@ TEST_F(SearchProviderTest, DefaultProviderSuggestRelevanceScoringUrlInput) {
{ { "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) {

Powered by Google App Engine
This is Rietveld 408576698