Store default search provider data in dictionary pref.

chrome/browser/search_engines/template_url_service.h

Index: chrome/browser/search_engines/template_url_service.h
diff --git a/chrome/browser/search_engines/template_url_service.h b/chrome/browser/search_engines/template_url_service.h
index e2dc86a5d87fda9a557120938123713964ff3f7a..f581ca156e8872d04f25a3ee39e77632598282b2 100644
--- a/chrome/browser/search_engines/template_url_service.h
+++ b/chrome/browser/search_engines/template_url_service.h
@@ -16,6 +16,7 @@
 #include "base/memory/scoped_ptr.h"
 #include "base/observer_list.h"
 #include "base/prefs/pref_change_registrar.h"
+#include "chrome/browser/search_engines/default_search_manager.h"
 #include "chrome/browser/search_engines/template_url_id.h"
 #include "chrome/browser/webdata/web_data_service.h"
 #include "components/keyed_service/core/keyed_service.h"
@@ -776,6 +777,8 @@ class TemplateURLService : public WebDataServiceConsumer,
   // Stores a list of callbacks to be run after TemplateURLService has loaded.
   base::CallbackList<void(void)> on_loaded_callbacks_;
 
+  scoped_ptr<DefaultSearchManager> default_search_manager_;

[Peter Kasting, 2014/04/23 20:41:06]

Why is this a scoped_ptr<> instead of a straight data member?

[Cait (Slow), 2014/04/23 23:26:55]

TemplateURLService may be initialized with a NULL profile (and thus a NULL
PrefService), so the options are either to have DefaultSearchManager handle null
PrefServices, or use a scoped_ptr for default_search_manager_, which gets set to
NULL when we have no PrefService. Of the 2, I think I prefer the latter, since
at this point in time, DefaultSearchManager is useless w/o a PrefService.

+
   DISALLOW_COPY_AND_ASSIGN(TemplateURLService);
 };