diff options
Diffstat (limited to 'chrome/browser/autocomplete')
| -rw-r--r-- | chrome/browser/autocomplete/autocomplete_controller.cc | 5 | ||||
| -rw-r--r-- | chrome/browser/autocomplete/autocomplete_provider_unittest.cc | 25 |
2 files changed, 26 insertions, 4 deletions
diff --git a/chrome/browser/autocomplete/autocomplete_controller.cc b/chrome/browser/autocomplete/autocomplete_controller.cc index ee39845..492533a 100644 --- a/chrome/browser/autocomplete/autocomplete_controller.cc +++ b/chrome/browser/autocomplete/autocomplete_controller.cc @@ -543,10 +543,11 @@ GURL AutocompleteController::GetDestinationURL( !match.search_terms_args->assisted_query_stats.empty()) { TemplateURLRef::SearchTermsArgs search_terms_args(*match.search_terms_args); search_terms_args.assisted_query_stats += base::StringPrintf( - ".%" PRId64 "j%d", + ".%" PRId64 "j%dj%d", query_formulation_time.InMilliseconds(), search_provider_ && - search_provider_->field_trial_triggered_in_session()); + search_provider_->field_trial_triggered_in_session(), + input_.current_page_classification()); destination_url = GURL(template_url->url_ref(). ReplaceSearchTerms(search_terms_args)); } diff --git a/chrome/browser/autocomplete/autocomplete_provider_unittest.cc b/chrome/browser/autocomplete/autocomplete_provider_unittest.cc index 8b56cbb..7946fd6 100644 --- a/chrome/browser/autocomplete/autocomplete_provider_unittest.cc +++ b/chrome/browser/autocomplete/autocomplete_provider_unittest.cc @@ -628,7 +628,7 @@ TEST_F(AutocompleteProviderTest, GetDestinationURL) { "chrome.0.69i57j69i58j5l2j0l3j69i59"; url = controller_->GetDestinationURL(match, base::TimeDelta::FromMilliseconds(2456)); - EXPECT_EQ("//aqs=chrome.0.69i57j69i58j5l2j0l3j69i59.2456j0&", url.path()); + EXPECT_EQ("//aqs=chrome.0.69i57j69i58j5l2j0l3j69i59.2456j0j0&", url.path()); // Test field trial triggered bit set. controller_->search_provider_->field_trial_triggered_in_session_ = true; @@ -636,5 +636,26 @@ TEST_F(AutocompleteProviderTest, GetDestinationURL) { controller_->search_provider_->field_trial_triggered_in_session()); url = controller_->GetDestinationURL(match, base::TimeDelta::FromMilliseconds(2456)); - EXPECT_EQ("//aqs=chrome.0.69i57j69i58j5l2j0l3j69i59.2456j1&", url.path()); + EXPECT_EQ("//aqs=chrome.0.69i57j69i58j5l2j0l3j69i59.2456j1j0&", url.path()); + + // Test page classification set. + controller_->Start(AutocompleteInput( + string16(), string16::npos, string16(), GURL(), + AutocompleteInput::OTHER, true, false, true, + AutocompleteInput::ALL_MATCHES)); + + controller_->search_provider_->field_trial_triggered_in_session_ = false; + EXPECT_FALSE( + controller_->search_provider_->field_trial_triggered_in_session()); + url = controller_->GetDestinationURL(match, + base::TimeDelta::FromMilliseconds(2456)); + EXPECT_EQ("//aqs=chrome.0.69i57j69i58j5l2j0l3j69i59.2456j0j4&", url.path()); + + // Test page classification and field trial triggered set. + controller_->search_provider_->field_trial_triggered_in_session_ = true; + EXPECT_TRUE( + controller_->search_provider_->field_trial_triggered_in_session()); + url = controller_->GetDestinationURL(match, + base::TimeDelta::FromMilliseconds(2456)); + EXPECT_EQ("//aqs=chrome.0.69i57j69i58j5l2j0l3j69i59.2456j1j4&", url.path()); } |