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

Unified Diff: chrome/browser/autocomplete/builtin_provider.cc

Issue 6995096: Update BuiltinProvider to use chrome:// URLs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Update tests after kChromeUICryptohomeHost was added. Created 9 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/autocomplete/builtin_provider.cc
diff --git a/chrome/browser/autocomplete/builtin_provider.cc b/chrome/browser/autocomplete/builtin_provider.cc
index 507d9ed775113e31a50a4a7c8ce55050018e04ca..7fb5159d646e5d949afbedd2041ca08c87348e86 100644
--- a/chrome/browser/autocomplete/builtin_provider.cc
+++ b/chrome/browser/autocomplete/builtin_provider.cc
@@ -6,9 +6,35 @@
#include "base/string_util.h"
#include "base/utf_string_conversions.h"
-#include "chrome/browser/autocomplete/autocomplete_match.h"
#include "chrome/browser/browser_about_handler.h"
#include "chrome/browser/net/url_fixer_upper.h"
+#include "chrome/common/url_constants.h"
+
+namespace {
+
+// This list should be kept in sync with chrome/common/url_constants.h.
+const char* kChromeSettingsSubPages[] = {
+ chrome::kAdvancedOptionsSubPage,
+ chrome::kAutofillSubPage,
+ chrome::kBrowserOptionsSubPage,
+ chrome::kClearBrowserDataSubPage,
+ chrome::kContentSettingsSubPage,
+ chrome::kContentSettingsExceptionsSubPage,
+ chrome::kImportDataSubPage,
+ chrome::kInstantConfirmPage,
+ chrome::kLanguageOptionsSubPage,
+ chrome::kPersonalOptionsSubPage,
+ chrome::kPasswordManagerSubPage,
+ chrome::kSearchEnginesSubPage,
+ chrome::kSyncSetupSubPage,
+#if defined(OS_CHROMEOS)
+ chrome::kAboutOptionsSubPage,
+ chrome::kInternetOptionsSubPage,
+ chrome::kSystemOptionsSubPage,
+#endif
+};
+
+} // namespace
const int BuiltinProvider::kRelevance = 575;
@@ -17,8 +43,12 @@ BuiltinProvider::BuiltinProvider(ACProviderListener* listener,
: AutocompleteProvider(listener, profile, "Builtin") {
std::vector<std::string> builtins(ChromePaths());
for (std::vector<std::string>::iterator i(builtins.begin());
- i != builtins.end(); ++i)
- builtins_.push_back(ASCIIToUTF16("about:") + ASCIIToUTF16(*i));
+ i != builtins.end(); ++i)
+ builtins_.push_back(ASCIIToUTF16(*i));
+ string16 settings(ASCIIToUTF16(chrome::kChromeUISettingsHost) +
+ ASCIIToUTF16("/"));
+ for (size_t i = 0; i < arraysize(kChromeSettingsSubPages); i++)
+ builtins_.push_back(settings + ASCIIToUTF16(kChromeSettingsSubPages[i]));
}
BuiltinProvider::~BuiltinProvider() {}
@@ -31,24 +61,65 @@ void BuiltinProvider::Start(const AutocompleteInput& input,
(input.type() == AutocompleteInput::QUERY) ||
(input.matches_requested() == AutocompleteInput::BEST_MATCH))
return;
- for (Builtins::const_iterator i(builtins_.begin());
- (i != builtins_.end()) && (matches_.size() < kMaxMatches); ++i) {
- if (StartsWith(*i, input.text(), false)) {
- AutocompleteMatch match(this, kRelevance, false,
- AutocompleteMatch::NAVSUGGEST);
- match.fill_into_edit = *i;
- match.destination_url = GURL(*i);
- match.contents = match.fill_into_edit;
- match.contents_class.push_back(ACMatchClassification(0,
- ACMatchClassification::MATCH | ACMatchClassification::URL));
- if (match.contents.length() > input.text().length()) {
- match.contents_class.push_back(
- ACMatchClassification(input.text().length(),
- ACMatchClassification::URL));
+
+ static const string16 kAbout = ASCIIToUTF16(chrome::kAboutScheme) +
+ ASCIIToUTF16(chrome::kStandardSchemeSeparator);
+ static const string16 kChrome = ASCIIToUTF16(chrome::kChromeUIScheme) +
+ ASCIIToUTF16(chrome::kStandardSchemeSeparator);
+
+ static const int kUrl = ACMatchClassification::URL;
+ static const int kMatch = kUrl | ACMatchClassification::MATCH;
+
+ string16 text = input.text();
+ bool starting_chrome = StartsWith(kChrome, text, false);
+ if (starting_chrome || StartsWith(kAbout, text, false)) {
+ ACMatchClassifications styles;
+ // Highlight the input portion matching "chrome://"; or if the user has
+ // input "about:" (with optional slashes), highlight the whole "chrome://".
+ static const size_t kAboutSchemeLength = strlen(chrome::kAboutScheme);
+ bool highlight = starting_chrome || text.length() > kAboutSchemeLength;
+ styles.push_back(ACMatchClassification(0, highlight ? kMatch : kUrl));
+ size_t offset = starting_chrome ? text.length() : kChrome.length();
+ if (highlight)
+ styles.push_back(ACMatchClassification(offset, kUrl));
+ // Include some common builtin chrome URLs as the user types the scheme.
+ AddMatch(ASCIIToUTF16(chrome::kChromeUIChromeURLsURL), styles);
+ AddMatch(ASCIIToUTF16(chrome::kChromeUISettingsURL), styles);
+ AddMatch(ASCIIToUTF16(chrome::kChromeUIVersionURL), styles);
+ } else {
+ // Match input about: or chrome: URL input against builtin chrome URLs.
+ GURL url = URLFixerUpper::FixupURL(UTF16ToUTF8(text), std::string());
+ if (url.SchemeIs(chrome::kChromeUIScheme) && url.has_host()) {
+ // Include the path for sub-pages (e.g. "chrome://settings/browser").
+ string16 host_and_path = UTF8ToUTF16(url.host() + url.path());
+ TrimString(host_and_path, ASCIIToUTF16("/").c_str(), &host_and_path);
+ size_t match_length = kChrome.length() + host_and_path.length();
+ for (Builtins::const_iterator i(builtins_.begin());
+ (i != builtins_.end()) && (matches_.size() < kMaxMatches); ++i) {
+ if (StartsWith(*i, host_and_path, false)) {
+ ACMatchClassifications styles;
+ // Highlight the "chrome://" scheme, even for input "about:foo".
+ styles.push_back(ACMatchClassification(0, kMatch));
+ string16 match_string = kChrome + *i;
+ if (match_string.length() > match_length)
+ styles.push_back(ACMatchClassification(match_length, kUrl));
+ AddMatch(match_string, styles);
+ }
}
- matches_.push_back(match);
}
}
+
for (size_t i = 0; i < matches_.size(); ++i)
matches_[i].relevance = kRelevance + matches_.size() - (i + 1);
}
+
+void BuiltinProvider::AddMatch(const string16& match_string,
+ const ACMatchClassifications& styles) {
+ AutocompleteMatch match(this, kRelevance, false,
+ AutocompleteMatch::NAVSUGGEST);
+ match.fill_into_edit = match_string;
+ match.destination_url = GURL(match_string);
+ match.contents = match_string;
+ match.contents_class = styles;
+ matches_.push_back(match);
+}
« no previous file with comments | « chrome/browser/autocomplete/builtin_provider.h ('k') | chrome/browser/autocomplete/builtin_provider_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698