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

Unified Diff: components/url_formatter/url_formatter.cc

Issue 2478573004: Convert GURL::{host,path} to GURL::{host_piece,path_piece} for ==. (Closed)
Patch Set: rebase to #431874 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/url_formatter/url_formatter.cc
diff --git a/components/url_formatter/url_formatter.cc b/components/url_formatter/url_formatter.cc
index 59b072033454744a22f4e662f6ecb3ea5324bfbd..43b53412202bea9b87c8d46a6dccd4a85ceb626d 100644
--- a/components/url_formatter/url_formatter.cc
+++ b/components/url_formatter/url_formatter.cc
@@ -778,7 +778,7 @@ bool CanStripTrailingSlash(const GURL& url) {
// Omit the path only for standard, non-file URLs with nothing but "/" after
// the hostname.
return url.IsStandard() && !url.SchemeIsFile() && !url.SchemeIsFileSystem() &&
- !url.has_query() && !url.has_ref() && url.path() == "/";
+ !url.has_query() && !url.has_ref() && url.path_piece() == "/";
}
void AppendFormattedHost(const GURL& url, base::string16* output) {
« no previous file with comments | « components/sessions/content/content_serialized_navigation_driver.cc ('k') | content/browser/appcache/appcache_quota_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698