Index: chrome/browser/autocomplete/history_url_provider_unittest.cc |
diff --git a/chrome/browser/autocomplete/history_url_provider_unittest.cc b/chrome/browser/autocomplete/history_url_provider_unittest.cc |
deleted file mode 100644 |
index d4122f89865c63bdf8b7f29339ab8a0141d31554..0000000000000000000000000000000000000000 |
--- a/chrome/browser/autocomplete/history_url_provider_unittest.cc |
+++ /dev/null |
@@ -1,1118 +0,0 @@ |
-// 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 "base/macros.h" |
-#include "base/memory/scoped_ptr.h" |
-#include "base/message_loop/message_loop.h" |
-#include "base/prefs/pref_service.h" |
-#include "base/strings/string_util.h" |
-#include "base/strings/utf_string_conversions.h" |
-#include "base/time/time.h" |
-#include "chrome/browser/autocomplete/chrome_autocomplete_provider_client.h" |
-#include "chrome/browser/autocomplete/chrome_autocomplete_scheme_classifier.h" |
-#include "chrome/browser/history/history_service_factory.h" |
-#include "chrome/browser/search_engines/chrome_template_url_service_client.h" |
-#include "chrome/browser/search_engines/template_url_service_factory.h" |
-#include "chrome/common/pref_names.h" |
-#include "chrome/test/base/testing_browser_process.h" |
-#include "chrome/test/base/testing_profile.h" |
-#include "components/history/core/browser/history_service.h" |
-#include "components/history/core/browser/url_database.h" |
-#include "components/metrics/proto/omnibox_event.pb.h" |
-#include "components/metrics/proto/omnibox_input_type.pb.h" |
-#include "components/omnibox/browser/autocomplete_match.h" |
-#include "components/omnibox/browser/autocomplete_provider.h" |
-#include "components/omnibox/browser/autocomplete_provider_listener.h" |
-#include "components/omnibox/browser/autocomplete_result.h" |
-#include "components/omnibox/browser/history_quick_provider.h" |
-#include "components/search_engines/default_search_manager.h" |
-#include "components/search_engines/search_terms_data.h" |
-#include "components/search_engines/template_url.h" |
-#include "components/search_engines/template_url_service.h" |
-#include "components/url_formatter/url_fixer.h" |
-#include "content/public/test/test_browser_thread_bundle.h" |
-#include "testing/gtest/include/gtest/gtest.h" |
- |
-using base::ASCIIToUTF16; |
-using base::Time; |
-using base::TimeDelta; |
- |
-using content::TestBrowserThreadBundle; |
- |
-struct TestURLInfo { |
- const char* url; |
- const char* title; |
- int visit_count; |
- int typed_count; |
- int age_in_days; |
-} test_db[] = { |
- {"http://www.google.com/", "Google", 3, 3, 80}, |
- |
- // High-quality pages should get a host synthesized as a lower-quality match. |
- {"http://slashdot.org/favorite_page.html", "Favorite page", 200, 100, 80}, |
- |
- // Less popular pages should have hosts synthesized as higher-quality |
- // matches. |
- {"http://kerneltrap.org/not_very_popular.html", "Less popular", 4, 0, 80}, |
- |
- // Unpopular pages should not appear in the results at all. |
- {"http://freshmeat.net/unpopular.html", "Unpopular", 1, 0, 80}, |
- |
- // If a host has a match, we should pick it up during host synthesis. |
- {"http://news.google.com/?ned=us&topic=n", "Google News - U.S.", 2, 2, 80}, |
- {"http://news.google.com/", "Google News", 1, 1, 80}, |
- |
- // Matches that are normally not inline-autocompletable should be |
- // autocompleted if they are shorter substitutes for longer matches that would |
- // have been inline autocompleted. |
- {"http://synthesisatest.com/foo/", "Test A", 1, 1, 80}, |
- {"http://synthesisbtest.com/foo/", "Test B", 1, 1, 80}, |
- {"http://synthesisbtest.com/foo/bar.html", "Test B Bar", 2, 2, 80}, |
- |
- // Suggested short URLs must be "good enough" and must match user input. |
- {"http://foo.com/", "Dir", 5, 5, 80}, |
- {"http://foo.com/dir/", "Dir", 2, 2, 80}, |
- {"http://foo.com/dir/another/", "Dir", 5, 1, 80}, |
- {"http://foo.com/dir/another/again/", "Dir", 10, 0, 80}, |
- {"http://foo.com/dir/another/again/myfile.html", "File", 10, 2, 80}, |
- |
- // We throw in a lot of extra URLs here to make sure we're testing the |
- // history database's query, not just the autocomplete provider. |
- {"http://startest.com/y/a", "A", 2, 2, 80}, |
- {"http://startest.com/y/b", "B", 5, 2, 80}, |
- {"http://startest.com/x/c", "C", 5, 2, 80}, |
- {"http://startest.com/x/d", "D", 5, 5, 80}, |
- {"http://startest.com/y/e", "E", 4, 2, 80}, |
- {"http://startest.com/y/f", "F", 3, 2, 80}, |
- {"http://startest.com/y/g", "G", 3, 2, 80}, |
- {"http://startest.com/y/h", "H", 3, 2, 80}, |
- {"http://startest.com/y/i", "I", 3, 2, 80}, |
- {"http://startest.com/y/j", "J", 3, 2, 80}, |
- {"http://startest.com/y/k", "K", 3, 2, 80}, |
- {"http://startest.com/y/l", "L", 3, 2, 80}, |
- {"http://startest.com/y/m", "M", 3, 2, 80}, |
- |
- // A file: URL is useful for testing that fixup does the right thing w.r.t. |
- // the number of trailing slashes on the user's input. |
- {"file:///C:/foo.txt", "", 2, 2, 80}, |
- |
- // Results with absurdly high typed_counts so that very generic queries like |
- // "http" will give consistent results even if more data is added above. |
- {"http://bogussite.com/a", "Bogus A", 10002, 10000, 80}, |
- {"http://bogussite.com/b", "Bogus B", 10001, 10000, 80}, |
- {"http://bogussite.com/c", "Bogus C", 10000, 10000, 80}, |
- |
- // Domain name with number. |
- {"http://www.17173.com/", "Domain with number", 3, 3, 80}, |
- |
- // URLs to test exact-matching behavior. |
- {"http://go/", "Intranet URL", 1, 1, 80}, |
- {"http://gooey/", "Intranet URL 2", 5, 5, 80}, |
- |
- // URLs for testing offset adjustment. |
- {"http://www.\xEA\xB5\x90\xEC\x9C\xA1.kr/", "Korean", 2, 2, 80}, |
- {"http://spaces.com/path%20with%20spaces/foo.html", "Spaces", 2, 2, 80}, |
- {"http://ms/c++%20style%20guide", "Style guide", 2, 2, 80}, |
- |
- // URLs for testing ctrl-enter behavior. |
- {"http://binky/", "Intranet binky", 2, 2, 80}, |
- {"http://winky/", "Intranet winky", 2, 2, 80}, |
- {"http://www.winky.com/", "Internet winky", 5, 0, 80}, |
- |
- // URLs used by EmptyVisits. |
- {"http://pandora.com/", "Pandora", 2, 2, 80}, |
- // This entry is explicitly added more recently than |
- // history::kLowQualityMatchAgeLimitInDays. |
- // {"http://pa/", "pa", 0, 0, 80}, |
- |
- // For intranet based tests. |
- {"http://intra/one", "Intranet", 2, 2, 80}, |
- {"http://intra/two", "Intranet two", 1, 1, 80}, |
- {"http://intra/three", "Intranet three", 2, 2, 80}, |
- {"http://moo/bar", "Intranet moo", 1, 1, 80}, |
- {"http://typedhost/typedpath", "Intranet typed", 1, 1, 80}, |
- {"http://typedhost/untypedpath", "Intranet untyped", 1, 0, 80}, |
- |
- {"http://x.com/one", "Internet", 2, 2, 80}, |
- {"http://x.com/two", "Internet two", 1, 1, 80}, |
- {"http://x.com/three", "Internet three", 2, 2, 80}, |
- |
- // For punycode tests. |
- {"http://puny.xn--1lq90ic7f1rc.cn/", "Punycode", 2, 2, 5 }, |
- |
- // For experimental HUP scoring test. |
- {"http://7.com/1a", "One", 8, 4, 4}, |
- {"http://7.com/2a", "Two A", 4, 2, 8}, |
- {"http://7.com/2b", "Two B", 4, 1, 8}, |
- {"http://7.com/3a", "Three", 2, 1, 16}, |
- {"http://7.com/4a", "Four A", 1, 1, 32}, |
- {"http://7.com/4b", "Four B", 1, 1, 64}, |
- {"http://7.com/5a", "Five A", 8, 0, 64}, // never typed. |
-}; |
- |
-class HistoryURLProviderTest : public testing::Test, |
- public AutocompleteProviderListener { |
- public: |
- struct UrlAndLegalDefault { |
- std::string url; |
- bool allowed_to_be_default_match; |
- }; |
- |
- HistoryURLProviderTest() |
- : sort_matches_(false) { |
- HistoryQuickProvider::set_disabled(true); |
- } |
- |
- ~HistoryURLProviderTest() override { |
- HistoryQuickProvider::set_disabled(false); |
- } |
- |
- // AutocompleteProviderListener: |
- void OnProviderUpdate(bool updated_matches) override; |
- |
- protected: |
- static scoped_ptr<KeyedService> CreateTemplateURLService( |
- content::BrowserContext* context) { |
- Profile* profile = static_cast<Profile*>(context); |
- return make_scoped_ptr(new TemplateURLService( |
- profile->GetPrefs(), make_scoped_ptr(new SearchTermsData), NULL, |
- scoped_ptr<TemplateURLServiceClient>(new ChromeTemplateURLServiceClient( |
- HistoryServiceFactory::GetForProfile( |
- profile, ServiceAccessType::EXPLICIT_ACCESS))), |
- NULL, NULL, base::Closure())); |
- } |
- |
- // testing::Test |
- void SetUp() override { |
- ASSERT_TRUE(SetUpImpl(false)); |
- } |
- void TearDown() override; |
- |
- // Does the real setup. |
- bool SetUpImpl(bool no_db) WARN_UNUSED_RESULT; |
- |
- // Fills test data into the history system. |
- void FillData(); |
- |
- // Runs an autocomplete query on |text| and checks to see that the returned |
- // results' destination URLs match those provided. Also allows checking |
- // that the input type was identified correctly. |
- void RunTest(const base::string16& text, |
- const std::string& desired_tld, |
- bool prevent_inline_autocomplete, |
- const UrlAndLegalDefault* expected_urls, |
- size_t num_results, |
- metrics::OmniboxInputType::Type* identified_input_type); |
- |
- // A version of the above without the final |type| output parameter. |
- void RunTest(const base::string16& text, |
- const std::string& desired_tld, |
- bool prevent_inline_autocomplete, |
- const UrlAndLegalDefault* expected_urls, |
- size_t num_results) { |
- metrics::OmniboxInputType::Type type; |
- return RunTest(text, desired_tld, prevent_inline_autocomplete, |
- expected_urls, num_results, &type); |
- } |
- |
- content::TestBrowserThreadBundle thread_bundle_; |
- ACMatches matches_; |
- scoped_ptr<TestingProfile> profile_; |
- scoped_ptr<ChromeAutocompleteProviderClient> client_; |
- history::HistoryService* history_service_; |
- scoped_refptr<HistoryURLProvider> autocomplete_; |
- // Should the matches be sorted and duplicates removed? |
- bool sort_matches_; |
-}; |
- |
-class HistoryURLProviderTestNoDB : public HistoryURLProviderTest { |
- protected: |
- void SetUp() override { |
- ASSERT_TRUE(SetUpImpl(true)); |
- } |
-}; |
- |
-class HistoryURLProviderTestNoSearchProvider : public HistoryURLProviderTest { |
- protected: |
- void SetUp() override { |
- DefaultSearchManager::SetFallbackSearchEnginesDisabledForTesting(true); |
- HistoryURLProviderTest::SetUp(); |
- } |
- |
- void TearDown() override { |
- HistoryURLProviderTest::TearDown(); |
- DefaultSearchManager::SetFallbackSearchEnginesDisabledForTesting(false); |
- } |
-}; |
- |
-void HistoryURLProviderTest::OnProviderUpdate(bool updated_matches) { |
- if (autocomplete_->done()) |
- base::MessageLoop::current()->QuitWhenIdle(); |
-} |
- |
-bool HistoryURLProviderTest::SetUpImpl(bool no_db) { |
- profile_.reset(new TestingProfile()); |
- client_.reset(new ChromeAutocompleteProviderClient(profile_.get())); |
- if (!(profile_->CreateHistoryService(true, no_db))) |
- return false; |
- if (!no_db) { |
- profile_->BlockUntilHistoryProcessesPendingRequests(); |
- profile_->BlockUntilHistoryIndexIsRefreshed(); |
- } |
- profile_->GetPrefs()->SetString(prefs::kAcceptLanguages, "en-US,en,ko"); |
- history_service_ = HistoryServiceFactory::GetForProfile( |
- profile_.get(), ServiceAccessType::EXPLICIT_ACCESS); |
- |
- autocomplete_ = new HistoryURLProvider(client_.get(), this); |
- TemplateURLServiceFactory::GetInstance()->SetTestingFactoryAndUse( |
- profile_.get(), &HistoryURLProviderTest::CreateTemplateURLService); |
- FillData(); |
- return true; |
-} |
- |
-void HistoryURLProviderTest::TearDown() { |
- autocomplete_ = NULL; |
-} |
- |
-void HistoryURLProviderTest::FillData() { |
- // Most visits are a long time ago (some tests require this since we do some |
- // special logic for things visited very recently). Note that this time must |
- // be more recent than the "expire history" threshold for the data to be kept |
- // in the main database. |
- // |
- // TODO(brettw) It would be nice if we could test this behavior, in which |
- // case the time would be specifed in the test_db structure. |
- const Time now = Time::Now(); |
- |
- for (size_t i = 0; i < arraysize(test_db); ++i) { |
- const TestURLInfo& cur = test_db[i]; |
- const GURL current_url(cur.url); |
- history_service_->AddPageWithDetails( |
- current_url, base::UTF8ToUTF16(cur.title), cur.visit_count, |
- cur.typed_count, now - TimeDelta::FromDays(cur.age_in_days), false, |
- history::SOURCE_BROWSED); |
- } |
- |
- history_service_->AddPageWithDetails( |
- GURL("http://pa/"), base::UTF8ToUTF16("pa"), 0, 0, |
- Time::Now() - |
- TimeDelta::FromDays(history::kLowQualityMatchAgeLimitInDays - 1), |
- false, history::SOURCE_BROWSED); |
-} |
- |
-void HistoryURLProviderTest::RunTest( |
- const base::string16& text, |
- const std::string& desired_tld, |
- bool prevent_inline_autocomplete, |
- const UrlAndLegalDefault* expected_urls, |
- size_t num_results, |
- metrics::OmniboxInputType::Type* identified_input_type) { |
- AutocompleteInput input(text, base::string16::npos, desired_tld, GURL(), |
- metrics::OmniboxEventProto::INVALID_SPEC, |
- prevent_inline_autocomplete, false, true, true, false, |
- ChromeAutocompleteSchemeClassifier(profile_.get())); |
- *identified_input_type = input.type(); |
- autocomplete_->Start(input, false); |
- if (!autocomplete_->done()) |
- base::MessageLoop::current()->Run(); |
- |
- matches_ = autocomplete_->matches(); |
- if (sort_matches_) { |
- TemplateURLService* service = |
- TemplateURLServiceFactory::GetForProfile(profile_.get()); |
- for (ACMatches::iterator i = matches_.begin(); i != matches_.end(); ++i) { |
- i->ComputeStrippedDestinationURL( |
- input, client_->GetAcceptLanguages(), service); |
- } |
- AutocompleteResult::DedupMatchesByDestination( |
- input.current_page_classification(), false, &matches_); |
- std::sort(matches_.begin(), matches_.end(), |
- &AutocompleteMatch::MoreRelevant); |
- } |
- ASSERT_EQ(num_results, matches_.size()) << "Input text: " << text |
- << "\nTLD: \"" << desired_tld << "\""; |
- for (size_t i = 0; i < num_results; ++i) { |
- EXPECT_EQ(expected_urls[i].url, matches_[i].destination_url.spec()); |
- EXPECT_EQ(expected_urls[i].allowed_to_be_default_match, |
- matches_[i].allowed_to_be_default_match); |
- } |
-} |
- |
-TEST_F(HistoryURLProviderTest, PromoteShorterURLs) { |
- // Test that hosts get synthesized below popular pages. |
- const UrlAndLegalDefault expected_nonsynth[] = { |
- { "http://slashdot.org/favorite_page.html", false }, |
- { "http://slashdot.org/", false } |
- }; |
- RunTest(ASCIIToUTF16("slash"), std::string(), true, expected_nonsynth, |
- arraysize(expected_nonsynth)); |
- |
- // Test that hosts get synthesized above less popular pages. |
- const UrlAndLegalDefault expected_synth[] = { |
- { "http://kerneltrap.org/", false }, |
- { "http://kerneltrap.org/not_very_popular.html", false } |
- }; |
- RunTest(ASCIIToUTF16("kernel"), std::string(), true, expected_synth, |
- arraysize(expected_synth)); |
- |
- // Test that unpopular pages are ignored completely. |
- RunTest(ASCIIToUTF16("fresh"), std::string(), true, NULL, 0); |
- |
- // Test that if we create or promote shorter suggestions that would not |
- // normally be inline autocompletable, we make them inline autocompletable if |
- // the original suggestion (that we replaced as "top") was inline |
- // autocompletable. |
- const UrlAndLegalDefault expected_synthesisa[] = { |
- { "http://synthesisatest.com/", true }, |
- { "http://synthesisatest.com/foo/", true } |
- }; |
- RunTest(ASCIIToUTF16("synthesisa"), std::string(), false, expected_synthesisa, |
- arraysize(expected_synthesisa)); |
- EXPECT_LT(matches_.front().relevance, 1200); |
- const UrlAndLegalDefault expected_synthesisb[] = { |
- { "http://synthesisbtest.com/foo/", true }, |
- { "http://synthesisbtest.com/foo/bar.html", true } |
- }; |
- RunTest(ASCIIToUTF16("synthesisb"), std::string(), false, expected_synthesisb, |
- arraysize(expected_synthesisb)); |
- EXPECT_GE(matches_.front().relevance, 1410); |
- |
- // Test that if we have a synthesized host that matches a suggestion, they |
- // get combined into one. |
- const UrlAndLegalDefault expected_combine[] = { |
- { "http://news.google.com/", false }, |
- { "http://news.google.com/?ned=us&topic=n", false }, |
- }; |
- ASSERT_NO_FATAL_FAILURE(RunTest(ASCIIToUTF16("news"), std::string(), true, |
- expected_combine, |
- arraysize(expected_combine))); |
- // The title should also have gotten set properly on the host for the |
- // synthesized one, since it was also in the results. |
- EXPECT_EQ(ASCIIToUTF16("Google News"), matches_.front().description); |
- |
- // Test that short URL matching works correctly as the user types more |
- // (several tests): |
- // The entry for foo.com is the best of all five foo.com* entries. |
- const UrlAndLegalDefault short_1[] = { |
- { "http://foo.com/", false }, |
- { "http://foo.com/dir/another/again/myfile.html", false }, |
- { "http://foo.com/dir/", false } |
- }; |
- RunTest(ASCIIToUTF16("foo"), std::string(), true, short_1, |
- arraysize(short_1)); |
- |
- // When the user types the whole host, make sure we don't get two results for |
- // it. |
- const UrlAndLegalDefault short_2[] = { |
- { "http://foo.com/", true }, |
- { "http://foo.com/dir/another/again/myfile.html", false }, |
- { "http://foo.com/dir/", false }, |
- { "http://foo.com/dir/another/", false } |
- }; |
- RunTest(ASCIIToUTF16("foo.com"), std::string(), true, short_2, |
- arraysize(short_2)); |
- RunTest(ASCIIToUTF16("foo.com/"), std::string(), true, short_2, |
- arraysize(short_2)); |
- |
- // The filename is the second best of the foo.com* entries, but there is a |
- // shorter URL that's "good enough". The host doesn't match the user input |
- // and so should not appear. |
- const UrlAndLegalDefault short_3[] = { |
- { "http://foo.com/dir/another/", false }, |
- { "http://foo.com/d", true }, |
- { "http://foo.com/dir/another/again/myfile.html", false }, |
- { "http://foo.com/dir/", false } |
- }; |
- RunTest(ASCIIToUTF16("foo.com/d"), std::string(), true, short_3, |
- arraysize(short_3)); |
- // If prevent_inline_autocomplete is false, we won't bother creating the |
- // URL-what-you-typed match because we have promoted inline autocompletions. |
- const UrlAndLegalDefault short_3_allow_inline[] = { |
- { "http://foo.com/dir/another/", true }, |
- { "http://foo.com/dir/another/again/myfile.html", true }, |
- { "http://foo.com/dir/", true } |
- }; |
- RunTest(ASCIIToUTF16("foo.com/d"), std::string(), false, short_3_allow_inline, |
- arraysize(short_3_allow_inline)); |
- |
- // We shouldn't promote shorter URLs than the best if they're not good |
- // enough. |
- const UrlAndLegalDefault short_4[] = { |
- { "http://foo.com/dir/another/again/myfile.html", false }, |
- { "http://foo.com/dir/another/a", true }, |
- { "http://foo.com/dir/another/again/", false } |
- }; |
- RunTest(ASCIIToUTF16("foo.com/dir/another/a"), std::string(), true, short_4, |
- arraysize(short_4)); |
- // If prevent_inline_autocomplete is false, we won't bother creating the |
- // URL-what-you-typed match because we have promoted inline autocompletions. |
- const UrlAndLegalDefault short_4_allow_inline[] = { |
- { "http://foo.com/dir/another/again/myfile.html", true }, |
- { "http://foo.com/dir/another/again/", true } |
- }; |
- RunTest(ASCIIToUTF16("foo.com/dir/another/a"), std::string(), false, |
- short_4_allow_inline, arraysize(short_4_allow_inline)); |
- |
- // Exact matches should always be best no matter how much more another match |
- // has been typed. |
- const UrlAndLegalDefault short_5a[] = { |
- { "http://gooey/", true }, |
- { "http://www.google.com/", true }, |
- { "http://go/", true } |
- }; |
- const UrlAndLegalDefault short_5b[] = { |
- { "http://go/", true }, |
- { "http://gooey/", true }, |
- { "http://www.google.com/", true } |
- }; |
- RunTest(ASCIIToUTF16("g"), std::string(), false, short_5a, |
- arraysize(short_5a)); |
- RunTest(ASCIIToUTF16("go"), std::string(), false, short_5b, |
- arraysize(short_5b)); |
-} |
- |
-TEST_F(HistoryURLProviderTest, CullRedirects) { |
- // URLs we will be using, plus the visit counts they will initially get |
- // (the redirect set below will also increment the visit counts). We want |
- // the results to be in A,B,C order. Note also that our visit counts are |
- // all high enough so that domain synthesizing won't get triggered. |
- struct TestCase { |
- const char* url; |
- int count; |
- } test_cases[] = { |
- {"http://redirects/A", 30}, |
- {"http://redirects/B", 20}, |
- {"http://redirects/C", 10} |
- }; |
- for (size_t i = 0; i < arraysize(test_cases); ++i) { |
- history_service_->AddPageWithDetails(GURL(test_cases[i].url), |
- ASCIIToUTF16("Title"), test_cases[i].count, test_cases[i].count, |
- Time::Now(), false, history::SOURCE_BROWSED); |
- } |
- |
- // Create a B->C->A redirect chain, but set the visit counts such that they |
- // will appear in A,B,C order in the results. The autocomplete query will |
- // search for the most recent visit when looking for redirects, so this will |
- // be found even though the previous visits had no redirects. |
- history::RedirectList redirects_to_a; |
- redirects_to_a.push_back(GURL(test_cases[1].url)); |
- redirects_to_a.push_back(GURL(test_cases[2].url)); |
- redirects_to_a.push_back(GURL(test_cases[0].url)); |
- history_service_->AddPage(GURL(test_cases[0].url), base::Time::Now(), |
- NULL, 0, GURL(), redirects_to_a, ui::PAGE_TRANSITION_TYPED, |
- history::SOURCE_BROWSED, true); |
- |
- // Because all the results are part of a redirect chain with other results, |
- // all but the first one (A) should be culled. We should get the default |
- // "what you typed" result, plus this one. |
- const base::string16 typing(ASCIIToUTF16("http://redirects/")); |
- const UrlAndLegalDefault expected_results[] = { |
- { test_cases[0].url, false }, |
- { base::UTF16ToUTF8(typing), true } |
- }; |
- RunTest(typing, std::string(), true, expected_results, |
- arraysize(expected_results)); |
- |
- // If prevent_inline_autocomplete is false, we won't bother creating the |
- // URL-what-you-typed match because we have promoted inline autocompletions. |
- // The result set should instead consist of a single URL representing the |
- // whole set of redirects. |
- const UrlAndLegalDefault expected_results_allow_inlining[] = { |
- { test_cases[0].url, true } |
- }; |
- RunTest(typing, std::string(), false, expected_results_allow_inlining, |
- arraysize(expected_results_allow_inlining)); |
-} |
- |
-TEST_F(HistoryURLProviderTestNoSearchProvider, WhatYouTypedNoSearchProvider) { |
- // When no search provider is available, make sure we provide WYT matches |
- // for text that could be a URL. |
- |
- 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 } |
- }; |
- 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/", true } |
- }; |
- RunTest(ASCIIToUTF16("http://wytmatch foo bar"), std::string(), false, |
- results_2, arraysize(results_2)); |
- |
- const UrlAndLegalDefault results_3[] = { |
- { "https://wytmatch%20foo%20bar/", true } |
- }; |
- 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/", true } |
- }; |
- RunTest(base::WideToUTF16(L"\uff65@s"), std::string(), false, fixup_crash, |
- arraysize(fixup_crash)); |
- RunTest(base::WideToUTF16(L"\u2015\u2015@ \uff7c"), std::string(), false, |
- NULL, 0); |
- |
- // Fixing up "file:" should result in an inline autocomplete offset of just |
- // after "file:", not just after "file://". |
- const base::string16 input_1(ASCIIToUTF16("file:")); |
- const UrlAndLegalDefault fixup_1[] = { |
- { "file:///C:/foo.txt", true } |
- }; |
- ASSERT_NO_FATAL_FAILURE(RunTest(input_1, std::string(), false, fixup_1, |
- arraysize(fixup_1))); |
- EXPECT_EQ(ASCIIToUTF16("///C:/foo.txt"), |
- matches_.front().inline_autocompletion); |
- |
- // Fixing up "http:/" should result in an inline autocomplete offset of just |
- // after "http:/", not just after "http:". |
- const base::string16 input_2(ASCIIToUTF16("http:/")); |
- const UrlAndLegalDefault fixup_2[] = { |
- { "http://bogussite.com/a", true }, |
- { "http://bogussite.com/b", true }, |
- { "http://bogussite.com/c", true } |
- }; |
- ASSERT_NO_FATAL_FAILURE(RunTest(input_2, std::string(), false, fixup_2, |
- arraysize(fixup_2))); |
- EXPECT_EQ(ASCIIToUTF16("/bogussite.com/a"), |
- matches_.front().inline_autocompletion); |
- |
- // Adding a TLD to a small number like "56" should result in "www.56.com" |
- // rather than "0.0.0.56.com". |
- const UrlAndLegalDefault fixup_3[] = { |
- { "http://www.56.com/", true } |
- }; |
- RunTest(ASCIIToUTF16("56"), "com", true, fixup_3, arraysize(fixup_3)); |
- |
- // An input looks like a IP address like "127.0.0.1" should result in |
- // "http://127.0.0.1/". |
- const UrlAndLegalDefault fixup_4[] = { |
- { "http://127.0.0.1/", true } |
- }; |
- RunTest(ASCIIToUTF16("127.0.0.1"), std::string(), false, fixup_4, |
- arraysize(fixup_4)); |
- |
- // An number "17173" should result in "http://www.17173.com/" in db. |
- const UrlAndLegalDefault fixup_5[] = { |
- { "http://www.17173.com/", true } |
- }; |
- RunTest(ASCIIToUTF16("17173"), std::string(), false, fixup_5, |
- arraysize(fixup_5)); |
-} |
- |
-// Make sure the results for the input 'p' don't change between the first and |
-// second passes. |
-TEST_F(HistoryURLProviderTest, EmptyVisits) { |
- // Wait for history to create the in memory DB. |
- profile_->BlockUntilHistoryProcessesPendingRequests(); |
- |
- AutocompleteInput input( |
- ASCIIToUTF16("pa"), base::string16::npos, std::string(), GURL(), |
- metrics::OmniboxEventProto::INVALID_SPEC, false, false, true, true, false, |
- ChromeAutocompleteSchemeClassifier(profile_.get())); |
- autocomplete_->Start(input, false); |
- // HistoryURLProvider shouldn't be done (waiting on async results). |
- EXPECT_FALSE(autocomplete_->done()); |
- |
- // We should get back an entry for pandora. |
- matches_ = autocomplete_->matches(); |
- ASSERT_GT(matches_.size(), 0u); |
- EXPECT_EQ(GURL("http://pandora.com/"), matches_[0].destination_url); |
- int pandora_relevance = matches_[0].relevance; |
- |
- // Run the message loop. When |autocomplete_| finishes the loop is quit. |
- base::MessageLoop::current()->Run(); |
- EXPECT_TRUE(autocomplete_->done()); |
- matches_ = autocomplete_->matches(); |
- ASSERT_GT(matches_.size(), 0u); |
- EXPECT_EQ(GURL("http://pandora.com/"), matches_[0].destination_url); |
- EXPECT_EQ(pandora_relevance, matches_[0].relevance); |
-} |
- |
-TEST_F(HistoryURLProviderTestNoDB, NavigateWithoutDB) { |
- // Ensure that we will still produce matches for navigation when there is no |
- // database. |
- UrlAndLegalDefault navigation_1[] = { |
- { "http://test.com/", true } |
- }; |
- RunTest(ASCIIToUTF16("test.com"), std::string(), false, navigation_1, |
- arraysize(navigation_1)); |
- |
- UrlAndLegalDefault navigation_2[] = { |
- { "http://slash/", true } |
- }; |
- RunTest(ASCIIToUTF16("slash"), std::string(), false, navigation_2, |
- arraysize(navigation_2)); |
- |
- RunTest(ASCIIToUTF16("this is a query"), std::string(), false, NULL, 0); |
-} |
- |
-TEST_F(HistoryURLProviderTest, DontAutocompleteOnTrailingWhitespace) { |
- AutocompleteInput input( |
- ASCIIToUTF16("slash "), base::string16::npos, std::string(), GURL(), |
- metrics::OmniboxEventProto::INVALID_SPEC, false, false, true, true, false, |
- ChromeAutocompleteSchemeClassifier(profile_.get())); |
- autocomplete_->Start(input, false); |
- if (!autocomplete_->done()) |
- base::MessageLoop::current()->Run(); |
- |
- // None of the matches should attempt to autocomplete. |
- matches_ = autocomplete_->matches(); |
- for (size_t i = 0; i < matches_.size(); ++i) { |
- EXPECT_TRUE(matches_[i].inline_autocompletion.empty()); |
- EXPECT_FALSE(matches_[i].allowed_to_be_default_match); |
- } |
-} |
- |
-TEST_F(HistoryURLProviderTest, TreatEmailsAsSearches) { |
- // Visiting foo.com should not make this string be treated as a navigation. |
- // That means the result should be scored around 1200 ("what you typed") |
- // and not 1400+. |
- const UrlAndLegalDefault expected[] = { |
- { "http://user@foo.com/", true } |
- }; |
- ASSERT_NO_FATAL_FAILURE(RunTest(ASCIIToUTF16("user@foo.com"), std::string(), |
- false, expected, arraysize(expected))); |
- EXPECT_LE(1200, matches_[0].relevance); |
- EXPECT_LT(matches_[0].relevance, 1210); |
-} |
- |
-TEST_F(HistoryURLProviderTest, IntranetURLsWithPaths) { |
- struct TestCase { |
- const char* input; |
- int relevance; |
- } test_cases[] = { |
- { "fooey", 0 }, |
- { "fooey/", 1200 }, // 1200 for URL would still navigate by default. |
- { "fooey/a", 1200 }, // 1200 for UNKNOWN would not. |
- { "fooey/a b", 1200 }, // Also UNKNOWN. |
- { "gooey", 1410 }, |
- { "gooey/", 1410 }, |
- { "gooey/a", 1400 }, |
- { "gooey/a b", 1400 }, |
- }; |
- for (size_t i = 0; i < arraysize(test_cases); ++i) { |
- SCOPED_TRACE(test_cases[i].input); |
- if (test_cases[i].relevance == 0) { |
- RunTest(ASCIIToUTF16(test_cases[i].input), std::string(), false, NULL, 0); |
- } else { |
- const UrlAndLegalDefault output[] = { |
- {url_formatter::FixupURL(test_cases[i].input, std::string()).spec(), |
- true}}; |
- ASSERT_NO_FATAL_FAILURE(RunTest(ASCIIToUTF16(test_cases[i].input), |
- std::string(), false, output, arraysize(output))); |
- // Actual relevance should be at least what test_cases expects and |
- // and no more than 10 more. |
- EXPECT_LE(test_cases[i].relevance, matches_[0].relevance); |
- EXPECT_LT(matches_[0].relevance, test_cases[i].relevance + 10); |
- } |
- } |
-} |
- |
-// Makes sure autocompletion happens for intranet sites that have been |
-// previoulsy visited. |
-TEST_F(HistoryURLProviderTest, IntranetURLCompletion) { |
- sort_matches_ = true; |
- |
- const UrlAndLegalDefault expected1[] = { |
- { "http://intra/three", true }, |
- { "http://intra/two", true } |
- }; |
- ASSERT_NO_FATAL_FAILURE(RunTest(ASCIIToUTF16("intra/t"), std::string(), false, |
- expected1, arraysize(expected1))); |
- EXPECT_LE(1410, matches_[0].relevance); |
- EXPECT_LT(matches_[0].relevance, 1420); |
- // It uses the default scoring. |
- EXPECT_EQ(matches_[1].relevance, 1203); |
- |
- const UrlAndLegalDefault expected2[] = { |
- { "http://moo/b", true }, |
- { "http://moo/bar", true } |
- }; |
- ASSERT_NO_FATAL_FAILURE(RunTest(ASCIIToUTF16("moo/b"), std::string(), false, |
- expected2, arraysize(expected2))); |
- // The url what you typed match should be around 1400, otherwise the |
- // search what you typed match is going to be first. |
- EXPECT_LE(1400, matches_[0].relevance); |
- EXPECT_LT(matches_[0].relevance, 1410); |
- |
- const UrlAndLegalDefault expected3[] = { |
- { "http://intra/one", true }, |
- { "http://intra/three", true }, |
- { "http://intra/two", true } |
- }; |
- RunTest(ASCIIToUTF16("intra"), std::string(), false, expected3, |
- arraysize(expected3)); |
- |
- const UrlAndLegalDefault expected4[] = { |
- { "http://intra/one", true }, |
- { "http://intra/three", true }, |
- { "http://intra/two", true } |
- }; |
- RunTest(ASCIIToUTF16("intra/"), std::string(), false, expected4, |
- arraysize(expected4)); |
- |
- const UrlAndLegalDefault expected5[] = { |
- { "http://intra/one", true } |
- }; |
- ASSERT_NO_FATAL_FAILURE(RunTest(ASCIIToUTF16("intra/o"), std::string(), false, |
- expected5, arraysize(expected5))); |
- EXPECT_LE(1410, matches_[0].relevance); |
- EXPECT_LT(matches_[0].relevance, 1420); |
- |
- const UrlAndLegalDefault expected6[] = { |
- { "http://intra/x", true } |
- }; |
- ASSERT_NO_FATAL_FAILURE(RunTest(ASCIIToUTF16("intra/x"), std::string(), false, |
- expected6, arraysize(expected6))); |
- EXPECT_LE(1400, matches_[0].relevance); |
- EXPECT_LT(matches_[0].relevance, 1410); |
- |
- const UrlAndLegalDefault expected7[] = { |
- { "http://typedhost/untypedpath", true } |
- }; |
- ASSERT_NO_FATAL_FAILURE(RunTest(ASCIIToUTF16("typedhost/untypedpath"), |
- std::string(), false, expected7, |
- arraysize(expected7))); |
- EXPECT_LE(1400, matches_[0].relevance); |
- EXPECT_LT(matches_[0].relevance, 1410); |
-} |
- |
-TEST_F(HistoryURLProviderTest, CrashDueToFixup) { |
- // This test passes if we don't crash. The results don't matter. |
- const char* const test_cases[] = { |
- "//c", |
- "\\@st", |
- "view-source:x", |
- }; |
- for (size_t i = 0; i < arraysize(test_cases); ++i) { |
- AutocompleteInput input( |
- ASCIIToUTF16(test_cases[i]), base::string16::npos, std::string(), |
- GURL(), metrics::OmniboxEventProto::INVALID_SPEC, false, false, true, |
- true, false, ChromeAutocompleteSchemeClassifier(profile_.get())); |
- autocomplete_->Start(input, false); |
- if (!autocomplete_->done()) |
- base::MessageLoop::current()->Run(); |
- } |
-} |
- |
-TEST_F(HistoryURLProviderTest, DoesNotProvideMatchesOnFocus) { |
- AutocompleteInput input( |
- ASCIIToUTF16("foo"), base::string16::npos, std::string(), GURL(), |
- metrics::OmniboxEventProto::INVALID_SPEC, false, false, true, true, true, |
- ChromeAutocompleteSchemeClassifier(profile_.get())); |
- autocomplete_->Start(input, false); |
- EXPECT_TRUE(autocomplete_->matches().empty()); |
-} |
- |
-TEST_F(HistoryURLProviderTest, DoesNotInlinePunycodeMatches) { |
- // A URL that matches due to a match in the punycode URL are allowed to be the |
- // default match if the URL doesn't get rendered as international characters |
- // in the given locale. |
- const UrlAndLegalDefault expected_true[] = { |
- { "http://puny.xn--1lq90ic7f1rc.cn/", true }, |
- }; |
- UrlAndLegalDefault expected_false[] = { |
- { "http://puny.xn--1lq90ic7f1rc.cn/", false }, |
- }; |
- RunTest(ASCIIToUTF16("pun"), std::string(), false, expected_true, |
- arraysize(expected_true)); |
- RunTest(ASCIIToUTF16("puny."), std::string(), false, expected_true, |
- arraysize(expected_true)); |
- RunTest(ASCIIToUTF16("puny.x"), std::string(), false, expected_true, |
- arraysize(expected_true)); |
- RunTest(ASCIIToUTF16("puny.xn"), std::string(), false, expected_true, |
- arraysize(expected_true)); |
- RunTest(ASCIIToUTF16("puny.xn--"), std::string(), false, expected_true, |
- arraysize(expected_true)); |
- RunTest(ASCIIToUTF16("puny.xn--1l"), std::string(), false, expected_true, |
- arraysize(expected_true)); |
- RunTest(ASCIIToUTF16("puny.xn--1lq90ic7f1rc"), std::string(), false, |
- expected_true, arraysize(expected_true)); |
- RunTest(ASCIIToUTF16("puny.xn--1lq90ic7f1rc."), std::string(), false, |
- expected_true, arraysize(expected_true)); |
- // Set the language so the punycode part of the URL is rendered as |
- // international characters. Then this match should not be allowed to be |
- // the default match if the inline autocomplete text starts in the middle |
- // of the international characters. |
- profile_->GetPrefs()->SetString(prefs::kAcceptLanguages, "zh-CN"); |
- RunTest(ASCIIToUTF16("pun"), std::string(), false, expected_true, |
- arraysize(expected_true)); |
- RunTest(ASCIIToUTF16("puny."), std::string(), false, expected_true, |
- arraysize(expected_true)); |
- RunTest(ASCIIToUTF16("puny.x"), std::string(), false, expected_false, |
- arraysize(expected_false)); |
- RunTest(ASCIIToUTF16("puny.xn"), std::string(), false, expected_false, |
- arraysize(expected_false)); |
- RunTest(ASCIIToUTF16("puny.xn--"), std::string(), false, expected_false, |
- arraysize(expected_false)); |
- RunTest(ASCIIToUTF16("puny.xn--1l"), std::string(), false, expected_false, |
- arraysize(expected_false)); |
- RunTest(ASCIIToUTF16("puny.xn--1lq90ic7f1rc"), std::string(), false, |
- expected_true, arraysize(expected_true)); |
- RunTest(ASCIIToUTF16("puny.xn--1lq90ic7f1rc."), std::string(), false, |
- expected_true, arraysize(expected_true)); |
-} |
- |
-TEST_F(HistoryURLProviderTest, CullSearchResults) { |
- // Set up a default search engine. |
- TemplateURLData data; |
- data.SetShortName(ASCIIToUTF16("TestEngine")); |
- data.SetKeyword(ASCIIToUTF16("TestEngine")); |
- data.SetURL("http://testsearch.com/?q={searchTerms}"); |
- TemplateURLService* template_url_service = |
- TemplateURLServiceFactory::GetForProfile(profile_.get()); |
- TemplateURL* template_url = new TemplateURL(data); |
- template_url_service->Add(template_url); |
- template_url_service->SetUserSelectedDefaultSearchProvider(template_url); |
- template_url_service->Load(); |
- |
- // URLs we will be using, plus the visit counts they will initially get |
- // (the redirect set below will also increment the visit counts). We want |
- // the results to be in A,B,C order. Note also that our visit counts are |
- // all high enough so that domain synthesizing won't get triggered. |
- struct TestCase { |
- const char* url; |
- int count; |
- } test_cases[] = { |
- {"https://testsearch.com/", 30}, |
- {"https://testsearch.com/?q=foobar", 20}, |
- {"http://foobar.com/", 10} |
- }; |
- for (size_t i = 0; i < arraysize(test_cases); ++i) { |
- history_service_->AddPageWithDetails(GURL(test_cases[i].url), |
- base::UTF8ToUTF16("Title"), test_cases[i].count, test_cases[i].count, |
- Time::Now(), false, history::SOURCE_BROWSED); |
- } |
- |
- // We should not see search URLs when typing a previously used query. |
- const UrlAndLegalDefault expected_when_searching_query[] = { |
- { test_cases[2].url, false } |
- }; |
- RunTest(ASCIIToUTF16("foobar"), std::string(), true, |
- expected_when_searching_query, arraysize(expected_when_searching_query)); |
- |
- // We should not see search URLs when typing the search engine name. |
- const UrlAndLegalDefault expected_when_searching_site[] = { |
- { test_cases[0].url, false } |
- }; |
- RunTest(ASCIIToUTF16("testsearch"), std::string(), true, |
- expected_when_searching_site, arraysize(expected_when_searching_site)); |
-} |
- |
-TEST_F(HistoryURLProviderTest, SuggestExactInput) { |
- const size_t npos = std::string::npos; |
- struct TestCase { |
- // Inputs: |
- const char* input; |
- bool trim_http; |
- // Expected Outputs: |
- const char* contents; |
- // Offsets of the ACMatchClassifications, terminated by npos. |
- size_t offsets[3]; |
- // The index of the ACMatchClassification that should have the MATCH bit |
- // set, npos if no ACMatchClassification should have the MATCH bit set. |
- size_t match_classification_index; |
- } test_cases[] = { |
- { "http://www.somesite.com", false, |
- "http://www.somesite.com", {0, npos, npos}, 0 }, |
- { "www.somesite.com", true, |
- "www.somesite.com", {0, npos, npos}, 0 }, |
- { "www.somesite.com", false, |
- "http://www.somesite.com", {0, 7, npos}, 1 }, |
- { "somesite.com", true, |
- "somesite.com", {0, npos, npos}, 0 }, |
- { "somesite.com", false, |
- "http://somesite.com", {0, 7, npos}, 1 }, |
- { "w", true, |
- "w", {0, npos, npos}, 0 }, |
- { "w", false, |
- "http://w", {0, 7, npos}, 1 }, |
- { "w.com", true, |
- "w.com", {0, npos, npos}, 0 }, |
- { "w.com", false, |
- "http://w.com", {0, 7, npos}, 1 }, |
- { "www.w.com", true, |
- "www.w.com", {0, npos, npos}, 0 }, |
- { "www.w.com", false, |
- "http://www.w.com", {0, 7, npos}, 1 }, |
- { "view-source:w", true, |
- "view-source:w", {0, npos, npos}, 0 }, |
- { "view-source:www.w.com/", true, |
- "view-source:www.w.com", {0, npos, npos}, npos }, |
- { "view-source:www.w.com/", false, |
- "view-source:http://www.w.com", {0, npos, npos}, npos }, |
- { "view-source:http://www.w.com/", false, |
- "view-source:http://www.w.com", {0, npos, npos}, 0 }, |
- { " view-source:", true, |
- "view-source:", {0, npos, npos}, 0 }, |
- { "http:////////w.com", false, |
- "http://w.com", {0, npos, npos}, npos }, |
- { " http:////////www.w.com", false, |
- "http://www.w.com", {0, npos, npos}, npos }, |
- { "http:a///www.w.com", false, |
- "http://a///www.w.com", {0, npos, npos}, npos }, |
- { "mailto://a@b.com", true, |
- "mailto://a@b.com", {0, npos, npos}, 0 }, |
- { "mailto://a@b.com", false, |
- "mailto://a@b.com", {0, npos, npos}, 0 }, |
- }; |
- for (size_t i = 0; i < arraysize(test_cases); ++i) { |
- SCOPED_TRACE(testing::Message() << "Index " << i << " input: " |
- << test_cases[i].input << ", trim_http: " |
- << test_cases[i].trim_http); |
- |
- AutocompleteInput input( |
- ASCIIToUTF16(test_cases[i].input), base::string16::npos, std::string(), |
- GURL("about:blank"), metrics::OmniboxEventProto::INVALID_SPEC, false, |
- false, true, true, false, |
- ChromeAutocompleteSchemeClassifier(profile_.get())); |
- AutocompleteMatch match(autocomplete_->SuggestExactInput( |
- input, input.canonicalized_url(), test_cases[i].trim_http)); |
- EXPECT_EQ(ASCIIToUTF16(test_cases[i].contents), match.contents); |
- for (size_t match_index = 0; match_index < match.contents_class.size(); |
- ++match_index) { |
- EXPECT_EQ(test_cases[i].offsets[match_index], |
- match.contents_class[match_index].offset); |
- EXPECT_EQ(ACMatchClassification::URL | |
- (match_index == test_cases[i].match_classification_index ? |
- ACMatchClassification::MATCH : 0), |
- match.contents_class[match_index].style); |
- } |
- EXPECT_EQ(npos, test_cases[i].offsets[match.contents_class.size()]); |
- } |
-} |
- |
-TEST_F(HistoryURLProviderTest, HUPScoringExperiment) { |
- HUPScoringParams max_2000_no_time_decay; |
- max_2000_no_time_decay.typed_count_buckets.buckets().push_back( |
- std::make_pair(0.0, 2000)); |
- HUPScoringParams max_1250_no_time_decay; |
- max_1250_no_time_decay.typed_count_buckets.buckets().push_back( |
- std::make_pair(0.0, 1250)); |
- HUPScoringParams max_1000_no_time_decay; |
- max_1000_no_time_decay.typed_count_buckets.buckets().push_back( |
- std::make_pair(0.0, 1000)); |
- |
- HUPScoringParams max_1100_with_time_decay_and_max_cap; |
- max_1100_with_time_decay_and_max_cap.typed_count_buckets. |
- set_relevance_cap(1400); |
- max_1100_with_time_decay_and_max_cap.typed_count_buckets. |
- set_half_life_days(16); |
- max_1100_with_time_decay_and_max_cap.typed_count_buckets.buckets().push_back( |
- std::make_pair(0.5, 1100)); |
- max_1100_with_time_decay_and_max_cap.typed_count_buckets.buckets().push_back( |
- std::make_pair(0.24, 200)); |
- max_1100_with_time_decay_and_max_cap.typed_count_buckets.buckets().push_back( |
- std::make_pair(0.0, 100)); |
- |
- HUPScoringParams max_1100_visit_typed_decays; |
- max_1100_visit_typed_decays.typed_count_buckets.set_half_life_days(16); |
- max_1100_visit_typed_decays.typed_count_buckets.buckets().push_back( |
- std::make_pair(0.5, 1100)); |
- max_1100_visit_typed_decays.typed_count_buckets.buckets().push_back( |
- std::make_pair(0.0, 100)); |
- max_1100_visit_typed_decays.visited_count_buckets.set_half_life_days(16); |
- max_1100_visit_typed_decays.visited_count_buckets.buckets().push_back( |
- std::make_pair(0.5, 550)); |
- max_1100_visit_typed_decays.visited_count_buckets.buckets().push_back( |
- std::make_pair(0.0, 50)); |
- |
- const int kMaxMatches = 3; |
- struct TestCase { |
- const char* input; |
- HUPScoringParams scoring_params; |
- struct ExpectedMatch { |
- const char* url; |
- int control_relevance; |
- int experiment_relevance; |
- }; |
- ExpectedMatch matches[kMaxMatches]; |
- } test_cases[] = { |
- // Max score 2000 -> no demotion. |
- { "7.com/1", max_2000_no_time_decay, |
- {{"7.com/1a", 1413, 1413}, {NULL, 0, 0}, {NULL, 0, 0}} }, |
- |
- // Limit score to 1250/1000 and make sure that the top match is unchanged. |
- { "7.com/1", max_1250_no_time_decay, |
- {{"7.com/1a", 1413, 1413}, {NULL, 0, 0}, {NULL, 0, 0}} }, |
- { "7.com/2", max_1250_no_time_decay, |
- {{"7.com/2a", 1413, 1413}, {"7.com/2b", 1412, 1250}, {NULL, 0, 0}} }, |
- { "7.com/4", max_1000_no_time_decay, |
- {{"7.com/4", 1203, 1203}, {"7.com/4a", 1202, 1000}, |
- {"7.com/4b", 1201, 999}} }, |
- |
- // Max relevance cap is 1400 and half-life is 16 days. |
- { "7.com/1", max_1100_with_time_decay_and_max_cap, |
- {{"7.com/1a", 1413, 1413}, {NULL, 0, 0}, {NULL, 0, 0}} }, |
- { "7.com/4", max_1100_with_time_decay_and_max_cap, |
- {{"7.com/4", 1203, 1203}, {"7.com/4a", 1202, 200}, |
- {"7.com/4b", 1201, 100}} }, |
- |
- // Max relevance cap is 1400 and half-life is 16 days for both visit/typed. |
- { "7.com/5", max_1100_visit_typed_decays, |
- {{"7.com/5", 1203, 1203}, {"7.com/5a", 1202, 50}, {NULL, 0, 0}} }, |
- }; |
- for (size_t i = 0; i < arraysize(test_cases); ++i) { |
- SCOPED_TRACE(test_cases[i].input); |
- UrlAndLegalDefault output[kMaxMatches]; |
- int max_matches; |
- for (max_matches = 0; max_matches < kMaxMatches; ++max_matches) { |
- if (test_cases[i].matches[max_matches].url == NULL) |
- break; |
- output[max_matches].url = |
- url_formatter::FixupURL(test_cases[i].matches[max_matches].url, |
- std::string()) |
- .spec(); |
- output[max_matches].allowed_to_be_default_match = true; |
- } |
- autocomplete_->scoring_params_ = test_cases[i].scoring_params; |
- |
- // Test the experiment (scoring enabled). When scoring is disabled, it uses |
- // 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); |
- } |
- } |
-} |