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

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

Issue 10909130: autocomplete: Add AutocompleteProvider::Type enum. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: update comments Created 8 years, 3 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 61d9cc4fefe78de142f069e5f526ea4909de1fff..9ffe613e895a669e2b5986fa05d99d25bc771395 100644
--- a/chrome/browser/autocomplete/autocomplete_provider_unittest.cc
+++ b/chrome/browser/autocomplete/autocomplete_provider_unittest.cc
@@ -45,7 +45,7 @@ class TestProvider : public AutocompleteProvider {
TestProvider(int relevance, const string16& prefix,
Profile* profile,
const string16 match_keyword)
- : AutocompleteProvider(NULL, profile, ""),
+ : AutocompleteProvider(NULL, profile, AutocompleteProvider::TYPE_SEARCH),
Daniel Erat 2012/09/07 21:45:55 I'm using TYPE_SEARCH pretty arbitrarily here. Le
Peter Kasting 2012/09/07 23:23:02 This is fine.
relevance_(relevance),
prefix_(prefix),
match_keyword_(match_keyword) {
@@ -564,4 +564,3 @@ TEST_F(AutocompleteProviderTest, UpdateAssistedQueryStats) {
RunAssistedQueryStatsTest(test_data, ARRAYSIZE_UNSAFE(test_data));
}
}
-

Powered by Google App Engine
This is Rietveld 408576698