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

Unified Diff: components/search_engines/template_url_service.h

Issue 2479113002: Make extensions DSE persistent in browser prefs (Closed)
Patch Set: Tests updated(rewritten) after review Created 4 years 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: components/search_engines/template_url_service.h
diff --git a/components/search_engines/template_url_service.h b/components/search_engines/template_url_service.h
index 4a8a5af3f6d10e7d484ffa77b6b7216b78a262f8..7a18677782880d87207a3b25db002462c2656700 100644
--- a/components/search_engines/template_url_service.h
+++ b/components/search_engines/template_url_service.h
@@ -424,6 +424,7 @@ class TemplateURLService : public WebDataServiceConsumer,
friend class InstantUnitTestBase;
friend class TemplateURLServiceTestUtil;
+ friend class TemplateURLServiceTest;
vasilii 2016/12/06 19:16:42 What method is the reason for this?
Alexander Yashkin 2016/12/09 08:19:52 Removed, it seems its left from previous tests att
using GUIDToTURL = std::map<std::string, TemplateURL*>;
@@ -699,13 +700,6 @@ class TemplateURLService : public WebDataServiceConsumer,
TemplateURL* FindMatchingExtensionTemplateURL(const TemplateURLData& data,
TemplateURL::Type type);
- // Finds the most recently-installed NORMAL_CONTROLLED_BY_EXTENSION engine
- // that supports replacement and wants to be default, if any. Notifies the
- // DefaultSearchManager, which might change the effective default search
- // engine.
- void UpdateExtensionDefaultSearchEngine();
-
-
// ---------- Browser state related members ---------------------------------
PrefService* prefs_;

Powered by Google App Engine
This is Rietveld 408576698