Stop depending on TemplateURLService from DefaultSearchManager

chrome/browser/search_engines/default_search_manager.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_SEARCH_ENGINES_DEFAULT_SEARCH_MANAGER_H_
 #define CHROME_BROWSER_SEARCH_ENGINES_DEFAULT_SEARCH_MANAGER_H_
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 86 matching lines @@
   // Clear the extension-provided default search engine. Does not explicitly
   // disable Default Search. The new current default search engine will be
   // defined by policy, extensions, or pre-populated data.
   void ClearExtensionControlledDefaultSearchEngine();
 
   // Clear the user's default search provider choice from |pref_service_|. Does
   // not explicitly disable Default Search. The new default search
   // 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
   // NotifyObserver() if the effective DSE might have changed.
   void OnDefaultSearchPrefChanged();
 
   // Handles changes to kSearchProviderOverrides pref. Calls
   // LoadPrepopulatedDefaultSearch() and NotifyObserver() if the effective DSE
   // might have changed.
   void OnOverridesPrefChanged();
@@ skipping 33 matching lines @@
   // default search engines.
   scoped_ptr<TemplateURLData> prefs_default_search_;
 
   // True if the default search is currently enforced by policy.
   bool default_search_controlled_by_policy_;
 
   DISALLOW_COPY_AND_ASSIGN(DefaultSearchManager);
 };
 
 #endif  // CHROME_BROWSER_SEARCH_ENGINES_DEFAULT_SEARCH_MANAGER_H_