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

Unified Diff: net/base/net_util.cc

Issue 7811006: Add full support for filesystem URLs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Test updates Created 8 years, 10 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: net/base/net_util.cc
diff --git a/net/base/net_util.cc b/net/base/net_util.cc
index 5ecf74042bf250a4b164cb1aeeafa50b6d534aaa..7d5e0b8069992a71ac2b493c3f9cadfbfc5a10cf 100644
--- a/net/base/net_util.cc
+++ b/net/base/net_util.cc
@@ -1898,8 +1898,9 @@ string16 FormatUrl(const GURL& url,
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.has_query() &&
- !url.has_ref() && url.path() == "/";
+ return url.IsStandard() && !url.SchemeIsFile() &&
+ !url.SchemeIsFileSystem() && !url.has_query() && !url.has_ref()
eroman 2012/02/15 06:07:13 I see, I think I understand a bit better what this
ericu 2012/02/16 01:42:56 Having looked through again, I found two files tha
+ && url.path() == "/";
}
GURL SimplifyUrlForRequest(const GURL& url) {

Powered by Google App Engine
This is Rietveld 408576698