autocomplete: Add AutocompleteProvider::Type enum.

chrome/browser/autocomplete/autocomplete_controller.h

Index: chrome/browser/autocomplete/autocomplete_controller.h
diff --git a/chrome/browser/autocomplete/autocomplete_controller.h b/chrome/browser/autocomplete/autocomplete_controller.h
index c66d024402b98bfdeb0c4485b448c26fcaeb1d77..9551c626eb583c67d9537bd6c5ed079bd113f508 100644
--- a/chrome/browser/autocomplete/autocomplete_controller.h
+++ b/chrome/browser/autocomplete/autocomplete_controller.h
@@ -14,11 +14,11 @@
 #include "chrome/browser/autocomplete/autocomplete_provider.h"
 #include "chrome/browser/autocomplete/autocomplete_provider_listener.h"
 #include "chrome/browser/autocomplete/autocomplete_result.h"
+#include "chrome/browser/autocomplete/keyword_provider.h"
+#include "chrome/browser/autocomplete/search_provider.h"
 
 class AutocompleteControllerDelegate;
-class KeywordProvider;
 class Profile;
-class SearchProvider;
 class ZeroSuggestProvider;
 
 // The AutocompleteController is the center of the autocomplete system.  A
@@ -50,17 +50,31 @@ class AutocompleteController : public AutocompleteProviderListener {
   // second to set the providers to some known testing providers.  The default
   // providers will be overridden and the controller will take ownership of the
   // providers, Release()ing them on destruction.
+  //
+  // |provider_types| is a bitmap containing AutocompleteProvider::Type values
+  // that will (potentially, depending on platform, flags, etc.) be
+  // instantiated.
   AutocompleteController(Profile* profile,
-                         AutocompleteControllerDelegate* delegate);
+                         AutocompleteControllerDelegate* delegate,
+                         int provider_types);
 #ifdef UNIT_TEST
   AutocompleteController(const ACProviders& providers, Profile* profile)
       : delegate_(NULL),
         providers_(providers),
         keyword_provider_(NULL),
         search_provider_(NULL),
+        zero_suggest_provider_(NULL),
         done_(true),
         in_start_(false),
+        in_zero_suggest_(false),
         profile_(profile) {
+    for (ACProviders::iterator it = providers_.begin();

[Daniel Erat, 2012/09/11 01:33:08]

It looks like I can remove this loop now, too.

[Peter Kasting, 2012/09/11 01:38:42]

Yep, but I think you can kill this constructor entirely -- see
autocompelte_provider_unittest.cc.

[Daniel Erat, 2012/09/11 02:09:22]

Done.

+         it != providers_.end(); ++it) {
+      if ((*it)->type() == AutocompleteProvider::TYPE_SEARCH)
+        search_provider_ = static_cast<SearchProvider*>(*it);
+      else if ((*it)->type() == AutocompleteProvider::TYPE_KEYWORD)
+        keyword_provider_ = static_cast<KeywordProvider*>(*it);
+    }
   }
 #endif
   ~AutocompleteController();
@@ -128,14 +142,6 @@ class AutocompleteController : public AutocompleteProviderListener {
   // the popup to ensure it's not showing an out-of-date query.
   void ExpireCopiedEntries();
 
-#ifdef UNIT_TEST
-  void set_search_provider(SearchProvider* provider) {
-    search_provider_ = provider;
-  }
-  void set_keyword_provider(KeywordProvider* provider) {
-    keyword_provider_ = provider;
-  }
-#endif
   SearchProvider* search_provider() const { return search_provider_; }
   KeywordProvider* keyword_provider() const { return keyword_provider_; }