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

Unified Diff: chrome/browser/autocomplete/autocomplete_match.h

Issue 11198074: Initial implementation of dedupping search provider's URLs. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Fix a minor compilation error. Created 8 years, 2 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/autocomplete/autocomplete_match.h
diff --git a/chrome/browser/autocomplete/autocomplete_match.h b/chrome/browser/autocomplete/autocomplete_match.h
index f459578bb027f3f012342998b33a6066be4c94c5..551b9ef1f72788281edbc06331a5aa2c84ef1714 100644
--- a/chrome/browser/autocomplete/autocomplete_match.h
+++ b/chrome/browser/autocomplete/autocomplete_match.h
@@ -187,7 +187,11 @@ struct AutocompleteMatch {
// remove likely duplicates; these URLs are not used as actual
// destination URLs. This method is invoked internally by the
// AutocompleteResult and does not normally need to be invoked.
- void ComputeStrippedDestinationURL();
+ // If |search_provider_url| is not NULL, it is used to normalize
+ // URLs that might have been generated by the default search provider.
Peter Kasting 2012/10/19 22:49:33 We should not pass this in. If the match has an a
+ // This is in particular important for all URLs with time/position sensitive
+ // CGI params that need to be stripped off.
+ void ComputeStrippedDestinationURL(TemplateURL* search_provider_url);
// Gets data relevant to whether there should be any special keyword-related
// UI shown for this match. If this match represents a selected keyword, i.e.

Powered by Google App Engine
This is Rietveld 408576698