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

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

Issue 338363004: Remove Profile* from TemplateURL (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments Created 6 years, 6 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 1712fa0a6bd3e8690f3cefbf429c560fb6bdd0c2..dc059c05ba0aa9a225f4326d3fb82894e542c78b 100644
--- a/chrome/browser/search_engines/template_url_service.h
+++ b/chrome/browser/search_engines/template_url_service.h
@@ -124,17 +124,8 @@ class TemplateURLService : public WebDataServiceConsumer,
// Returns the search url for t_url. Returns an empty GURL if t_url has no
// url().
- // NOTE: |t_url| is non-const in this version because of the need to access
- // t_url->profile().
- static GURL GenerateSearchURL(TemplateURL* t_url);
-
- // Just like GenerateSearchURL except that it takes SearchTermsData to supply
- // the data for some search terms, e.g. so this can be used on threads other
- // than the UI thread. See the various TemplateURLRef::XXXUsingTermsData()
- // functions.
- static GURL GenerateSearchURLUsingTermsData(
- const TemplateURL* t_url,
- const SearchTermsData& search_terms_data);
+ static GURL GenerateSearchURL(const TemplateURL* t_url,
+ const SearchTermsData& search_terms_data);
// Saves enough of url to |prefs| so that it can be loaded from preferences on
// start up.
@@ -181,7 +172,6 @@ class TemplateURLService : public WebDataServiceConsumer,
// Like Add(), but overwrites the |template_url|'s values with the provided
// ones.
- void AddAndSetProfile(TemplateURL* template_url, Profile* profile);
void AddWithOverrides(TemplateURL* template_url,
const base::string16& short_name,
const base::string16& keyword,

Powered by Google App Engine
This is Rietveld 408576698