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

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

Issue 10909130: autocomplete: Add AutocompleteProvider::Type enum. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: add !! for windows 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
« no previous file with comments | « chrome/browser/autocomplete/search_provider.cc ('k') | chrome/browser/autocomplete/shortcuts_provider.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 e072b7a3db7b6087a2d05300781e4d6831a724a9..ffe1fd54da1e2c01966a53e45e001c38ebe5aa6a 100644
--- a/chrome/browser/autocomplete/search_provider_unittest.cc
+++ b/chrome/browser/autocomplete/search_provider_unittest.cc
@@ -669,9 +669,12 @@ TEST_F(SearchProviderTest, UpdateKeywordDescriptions) {
ACProviders providers;
SearchProvider* provider = provider_.release();
providers.push_back(provider);
- AutocompleteController controller(providers, &profile_);
- controller.set_search_provider(provider);
+
+ AutocompleteController controller(&profile_, NULL, 0);
+ controller.providers_.swap(providers);
+ controller.search_provider_ = provider;
provider->set_listener(&controller);
+
controller.Start(ASCIIToUTF16("k t"), string16(), false, false, true,
AutocompleteInput::ALL_MATCHES);
const AutocompleteResult& result = controller.result();
« no previous file with comments | « chrome/browser/autocomplete/search_provider.cc ('k') | chrome/browser/autocomplete/shortcuts_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698