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

Unified Diff: components/omnibox/browser/builtin_provider.cc

Issue 2481923002: [WIP] make GURL::path() return a StringPiece (Closed)
Patch Set: thanks asan Created 4 years, 1 month 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: components/omnibox/browser/builtin_provider.cc
diff --git a/components/omnibox/browser/builtin_provider.cc b/components/omnibox/browser/builtin_provider.cc
index 35fb3a8225156557c15796640f358dd59afb8b6a..704d37496b2e8574c454589ef377bf57d2bbc838 100644
--- a/components/omnibox/browser/builtin_provider.cc
+++ b/components/omnibox/browser/builtin_provider.cc
@@ -91,7 +91,8 @@ void BuiltinProvider::Start(const AutocompleteInput& input,
}
// Include the path for sub-pages (e.g. "chrome://settings/browser").
- base::string16 host_and_path = base::UTF8ToUTF16(url.host() + url.path());
+ base::string16 host_and_path =
+ base::UTF8ToUTF16(url.host() + url.path().as_string());
base::TrimString(host_and_path, base::ASCIIToUTF16("/"), &host_and_path);
size_t match_length = embedderAbout.length() + host_and_path.length();
for (Builtins::const_iterator i(builtins_.begin());

Powered by Google App Engine
This is Rietveld 408576698