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

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

Issue 2779893005: Continue to clean c_str() calls. (Closed)
Patch Set: Revert changes in font_service_app.cc Created 3 years, 9 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: components/omnibox/browser/builtin_provider.cc
diff --git a/components/omnibox/browser/builtin_provider.cc b/components/omnibox/browser/builtin_provider.cc
index 35fb3a8225156557c15796640f358dd59afb8b6a..d272e19556ad9436006d43572ad83aff2f5ac219 100644
--- a/components/omnibox/browser/builtin_provider.cc
+++ b/components/omnibox/browser/builtin_provider.cc
@@ -68,8 +68,7 @@ void BuiltinProvider::Start(const AutocompleteInput& input,
base::CompareCase::SENSITIVE);
// BuiltinProvider doesn't know how to suggest valid ?query or #fragment
// extensions to builtin URLs.
- if (url.SchemeIs(
- client_->GetEmbedderRepresentationOfAboutScheme().c_str()) &&
+ if (url.SchemeIs(client_->GetEmbedderRepresentationOfAboutScheme()) &&
url.has_host() && !url.has_query() && !url.has_ref()) {
// Suggest about:blank for substrings, taking URL fixup into account.
// Chrome does not support trailing slashes or paths for about:blank.
« no previous file with comments | « components/nacl/zygote/nacl_fork_delegate_linux.cc ('k') | components/omnibox/browser/omnibox_edit_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698