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

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

Issue 343523003: Remove AutocompleteInput Type and PageClassification. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 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/autocomplete_provider_unittest.cc
diff --git a/chrome/browser/autocomplete/autocomplete_provider_unittest.cc b/chrome/browser/autocomplete/autocomplete_provider_unittest.cc
index 1b381fc73f1db1a60e0b4eb8dde3e606afd46bb3..aab3b3fa7a253352189674ccfa48e7ff778f6fe4 100644
--- a/chrome/browser/autocomplete/autocomplete_provider_unittest.cc
+++ b/chrome/browser/autocomplete/autocomplete_provider_unittest.cc
@@ -25,6 +25,7 @@
#include "chrome/common/chrome_switches.h"
#include "chrome/test/base/testing_browser_process.h"
#include "chrome/test/base/testing_profile.h"
+#include "components/metrics/proto/omnibox_event.pb.h"
#include "content/public/browser/notification_observer.h"
#include "content/public/browser/notification_registrar.h"
#include "content/public/browser/notification_source.h"
@@ -404,7 +405,7 @@ void AutocompleteProviderTest::RunQuery(const base::string16 query) {
result_.Reset();
controller_->Start(AutocompleteInput(
query, base::string16::npos, base::string16(), GURL(),
- AutocompleteInput::INVALID_SPEC, true, false, true, true));
+ metrics::OmniboxEventProto::INVALID_SPEC, true, false, true, true));
if (!controller_->done())
// The message loop will terminate when all autocomplete input has been
@@ -422,7 +423,7 @@ void AutocompleteProviderTest::RunExactKeymatchTest(
// keyword or not.)
controller_->Start(AutocompleteInput(
base::ASCIIToUTF16("k test"), base::string16::npos, base::string16(),
- GURL(), AutocompleteInput::INVALID_SPEC, true, false,
+ GURL(), metrics::OmniboxEventProto::INVALID_SPEC, true, false,
allow_exact_keyword_match, false));
EXPECT_TRUE(controller_->done());
EXPECT_EQ(AutocompleteProvider::TYPE_SEARCH,
@@ -648,7 +649,8 @@ TEST_F(AutocompleteProviderTest, GetDestinationURL) {
EXPECT_EQ("//aqs=chrome.0.69i57j69i58j5l2j0l3j69i59.2456j1j0&", url.path());
// Test page classification set.
- controller_->input_.current_page_classification_ = AutocompleteInput::OTHER;
+ controller_->input_.current_page_classification_ =
+ metrics::OmniboxEventProto::OTHER;
controller_->search_provider_->field_trial_triggered_in_session_ = false;
EXPECT_FALSE(
controller_->search_provider_->field_trial_triggered_in_session());
« no previous file with comments | « chrome/browser/autocomplete/autocomplete_provider.cc ('k') | chrome/browser/autocomplete/autocomplete_result.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698