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

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

Issue 10274023: Omnibox SearchProvider Experiment Client Implementation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Make BestURLPrefix perform case-insensitive comparison. Created 8 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
« no previous file with comments | « chrome/browser/autocomplete/search_provider_unittest.cc ('k') | chrome/browser/autocomplete/url_prefix.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autocomplete/url_prefix.h
diff --git a/chrome/browser/autocomplete/url_prefix.h b/chrome/browser/autocomplete/url_prefix.h
index 29ca6211b29311a2fdd265e9862754d488cbbc4d..196f24e76d977f0213cc267a54591ef395d9a5fe 100644
--- a/chrome/browser/autocomplete/url_prefix.h
+++ b/chrome/browser/autocomplete/url_prefix.h
@@ -18,7 +18,7 @@ typedef std::vector<URLPrefix> URLPrefixes;
struct URLPrefix {
URLPrefix(const string16& prefix, size_t num_components);
- // Returns a vector of valid URL prefixes.
+ // Returns a vector of URL prefixes sorted by descending number of components.
static const URLPrefixes& GetURLPrefixes();
// Returns if the argument is a valid URL prefix.
@@ -27,6 +27,7 @@ struct URLPrefix {
// Returns the prefix with the most components that begins |text|, or NULL.
// |prefix_suffix| (which may be empty) is appended to every attempted prefix,
// which is useful for finding the innermost match of user input in a URL.
+ // Performs case insensitive string comparison.
static const URLPrefix* BestURLPrefix(const string16& text,
const string16& prefix_suffix);
« no previous file with comments | « chrome/browser/autocomplete/search_provider_unittest.cc ('k') | chrome/browser/autocomplete/url_prefix.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698