Stop depending on TemplateURLService from DefaultSearchManager

chrome/browser/search_engines/default_search_manager.h

Index: chrome/browser/search_engines/default_search_manager.h
diff --git a/chrome/browser/search_engines/default_search_manager.h b/chrome/browser/search_engines/default_search_manager.h
index 95a5bfbf4f37a24463f026161415dd794483185a..b8f7fcb9558f3cc428d52324faf6b36888deb572 100644
--- a/chrome/browser/search_engines/default_search_manager.h
+++ b/chrome/browser/search_engines/default_search_manager.h
@@ -104,6 +104,9 @@ class DefaultSearchManager {
   // engine will be defined by policy, extensions, or pre-populated data.
   void ClearUserSelectedDefaultSearchEngine();
 
+  // Disables the pre-populated TemplateURLs for testing purposes.

[Peter Kasting, 2014/06/18 00:01:04]

Nit: This comment doesn't seem very accurate or helpful; how about

Testing code can call this with |disabled| set to true to cause
GetDefaultSearchEngine() to return NULL instead of |fallback_default_search_| in
cases where the DSE source is FROM_FALLBACK.

[hashimoto, 2014/06/18 00:08:38]

Done.

+  static void SetFallbackSearchEnginesDisabledForTesting(bool disabled);

[Peter Kasting, 2014/06/18 00:01:04]

Nit: I would put this up with the other static methods.

[hashimoto, 2014/06/18 00:08:38]

Done.

+
  private:
   // Handles changes to kDefaultSearchProviderData pref. This includes sync and
   // policy changes. Calls LoadDefaultSearchEngineFromPrefs() and