Add full support for filesystem URLs.

net/base/net_util.cc

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 change is about now:

You have introduced SchemeIsFileSystem() into GURL. However existing code may
not expect that, and may make assumption on stuff like what (url.IsStandard() &&
!url.SchemeIsFile())  entails.

Did you create this changelist by doing a search over Chrome codebase for all
occurrences of things like "IsStandard", "SchemeIsFile"?

Hmm that is tough. There could be other harder to find callers testing the
scheme string directly for instance.

[ericu, 2012/02/16 01:42:56]

Having looked through again, I found two files that are probably OK as they are,
but would be better with a bit of filesystem-specific code.  I'll either add
them to the patch next week [I'm out until Monday] or log bugs for later.

+      && url.path() == "/";
 }
 
 GURL SimplifyUrlForRequest(const GURL& url) {