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

Unified Diff: chrome/browser/search_engines/template_url_service.h

Issue 12084076: Ensure post-sync TemplateURL of prepopulated engines use built-in version. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 months 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: 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 084460ac463fa081306e328e6f9db18d601c9841..7867a0de985b0b575cff7a1c97a6c9c619317461 100644
--- a/chrome/browser/search_engines/template_url_service.h
+++ b/chrome/browser/search_engines/template_url_service.h
@@ -446,6 +446,12 @@ class TemplateURLService : public WebDataServiceConsumer,
const TemplateURL& new_values,
const SearchTermsData& old_search_terms_data);
+ // If the TemplateURL comes from a prepopulated URL, update all its fields
+ // save for the keyword, short name and id so that they match the internal
+ // prepopulated URL. TemplateURL not coming from a prepopulated URL are not
Peter Kasting 2013/01/30 22:32:33 Nit: TemplateURLs
beaudoin 2013/02/01 21:12:38 Done.
+ // modified.
+ void UpdatePrepopulatedTemplateURL(TemplateURL* existing_turl);
Peter Kasting 2013/01/30 22:32:33 Nit: Seems like the function could be named better
beaudoin 2013/02/01 21:12:38 Done.
+
// Returns the preferences we use.
PrefService* GetPrefs();

Powered by Google App Engine
This is Rietveld 408576698