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

Unified Diff: chrome/browser/ui/search_engines/edit_search_engine_controller.cc

Issue 7558014: Add a URL param to indicate group selection in Instant field trial. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Thread safe handling of Profiles Created 9 years, 4 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/ui/search_engines/edit_search_engine_controller.cc
diff --git a/chrome/browser/ui/search_engines/edit_search_engine_controller.cc b/chrome/browser/ui/search_engines/edit_search_engine_controller.cc
index a0c9b52bf07fb1de02009b939c8413dc0215ac51..a85981a8db57ae5d5fbea2f7b044125323a50429 100644
--- a/chrome/browser/ui/search_engines/edit_search_engine_controller.cc
+++ b/chrome/browser/ui/search_engines/edit_search_engine_controller.cc
@@ -53,7 +53,8 @@ bool EditSearchEngineController::IsURLValid(
// If the url has a search term, replace it with a random string and make
// sure the resulting URL is valid. We don't check the validity of the url
// with the search term as that is not necessarily valid.
- return GURL(template_ref.ReplaceSearchTerms(TemplateURL(), ASCIIToUTF16("a"),
+ return GURL(template_ref.ReplaceSearchTerms(profile_,
Peter Kasting 2011/08/10 20:54:06 You can pass NULL here, it won't matter to the val
+ TemplateURL(), ASCIIToUTF16("a"),
TemplateURLRef::NO_SUGGESTIONS_AVAILABLE, string16())).is_valid();
}
@@ -137,7 +138,8 @@ std::string EditSearchEngineController::GetFixedUpURL(
TemplateURL t_url;
t_url.SetURL(url, 0, 0);
std::string expanded_url =
- t_url.url()->ReplaceSearchTerms(t_url, ASCIIToUTF16("x"), 0, string16());
+ t_url.url()->ReplaceSearchTerms(profile_, t_url, ASCIIToUTF16("x"), 0,
Peter Kasting 2011/08/10 20:54:06 You can pass NULL here as you don't read the query
+ string16());
url_parse::Parsed parts;
std::string scheme(
URLFixerUpper::SegmentURL(expanded_url, &parts));
@@ -148,4 +150,3 @@ std::string EditSearchEngineController::GetFixedUpURL(
return url;
}
-

Powered by Google App Engine
This is Rietveld 408576698