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

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

Issue 10909130: autocomplete: Add AutocompleteProvider::Type enum. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: more indentation changes 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_classifier.cc
diff --git a/chrome/browser/autocomplete/autocomplete_classifier.cc b/chrome/browser/autocomplete/autocomplete_classifier.cc
index 9d2648564f950cccdf488b53ff9c2e3248f6cce8..35497a727bd5850d1c4f4fec37b88bcafcd8d7c5 100644
--- a/chrome/browser/autocomplete/autocomplete_classifier.cc
+++ b/chrome/browser/autocomplete/autocomplete_classifier.cc
@@ -8,10 +8,24 @@
#include "chrome/browser/autocomplete/autocomplete_controller.h"
#include "chrome/browser/autocomplete/autocomplete_input.h"
#include "chrome/browser/autocomplete/autocomplete_match.h"
+#include "chrome/browser/autocomplete/autocomplete_provider.h"
#include "googleurl/src/gurl.h"
+// static
+const int AutocompleteClassifier::kDefaultOmniboxProviders =
+ AutocompleteProvider::TYPE_BUILTIN |
+ AutocompleteProvider::TYPE_EXTENSION_APP |
+ AutocompleteProvider::TYPE_HISTORY_CONTENTS |
+ AutocompleteProvider::TYPE_HISTORY_QUICK |
+ AutocompleteProvider::TYPE_HISTORY_URL |
+ AutocompleteProvider::TYPE_KEYWORD |
+ AutocompleteProvider::TYPE_SEARCH |
+ AutocompleteProvider::TYPE_SHORTCUTS |
+ AutocompleteProvider::TYPE_ZERO_SUGGEST;
+
AutocompleteClassifier::AutocompleteClassifier(Profile* profile)
- : controller_(new AutocompleteController(profile, NULL)),
+ : controller_(new AutocompleteController(profile, NULL,
+ kDefaultOmniboxProviders)),
Peter Kasting 2012/09/11 00:05:39 Nit: You have room to indent this even with "profi
Daniel Erat 2012/09/11 00:42:19 Done.
inside_classify_(false) {
}

Powered by Google App Engine
This is Rietveld 408576698