Add full support for filesystem URLs.

webkit/fileapi/sandbox_mount_point_provider.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "webkit/fileapi/sandbox_mount_point_provider.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 617 matching lines @@
     return FilePath();
 
   return root;
 }
 
 bool SandboxMountPointProvider::IsAllowedScheme(const GURL& url) const {
   // Basically we only accept http or https. We allow file:// URLs
   // only if --allow-file-access-from-files flag is given.
   if (url.SchemeIs("http") || url.SchemeIs("https"))
     return true;
+  if (url.SchemeIsFileSystem())
+    return url.inner_url() && IsAllowedScheme(*url.inner_url());

[abarth-chromium, 2012/02/15 08:43:20]

I wonder if places like this should be able to check more generally for the
presence of an inner URL rather than assuming that we should consult the inner
URL iff the scheme if FileSystem.

[ericu, 2012/02/15 22:48:09]

Eventually, probably...once we decide what all we might use inner_url for.  It's
a clear possibility for view-source, but we probably don't want to implement
full generality without some idea of if we'll ever use it or not, since it will
complicate a bunch of code.

+
   for (size_t i = 0;
        i < file_system_options_.additional_allowed_schemes().size();
        ++i) {
     if (url.SchemeIs(
             file_system_options_.additional_allowed_schemes()[i].c_str()))
       return true;
   }
   return false;
 }
 
 }  // namespace fileapi