Fixed file/directory url resolution for external mount point provider.

webkit/fileapi/file_system_context.cc

Index: webkit/fileapi/file_system_context.cc
===================================================================
--- webkit/fileapi/file_system_context.cc	(revision 82028)
+++ webkit/fileapi/file_system_context.cc	(working copy)
@@ -33,6 +33,25 @@
           file_message_loop, profile_path, is_incognito)) {
 }
 
+FileSystemContext::FileSystemContext(
+    scoped_refptr<base::MessageLoopProxy> file_message_loop,
+    scoped_refptr<base::MessageLoopProxy> io_message_loop,
+    scoped_refptr<quota::SpecialStoragePolicy> special_storage_policy,
+    const FilePath& profile_path,
+    bool is_incognito,
+    bool allow_file_access,
+    bool unlimited_quota,
+    FileSystemPathManager* path_manager)
+    : file_message_loop_(file_message_loop),
+      io_message_loop_(io_message_loop),
+      special_storage_policy_(special_storage_policy),
+      allow_file_access_from_files_(allow_file_access),
+      unlimited_quota_(unlimited_quota),
+      path_manager_(path_manager),
+      usage_tracker_(new FileSystemUsageTracker(
+          file_message_loop, profile_path, is_incognito)) {

[ericu, 2011/04/19 20:30:49]

How about passing in an path_manager that may be null, and creating one if it's
not, rather than having another constructor?  We do that in some other
filesystem classes.

[zel, 2011/04/19 23:14:00]

Done.

+}
+
 FileSystemContext::~FileSystemContext() {
 }
 
@@ -67,6 +86,7 @@
   delete this;
 }
 

[ericu, 2011/04/19 20:30:49]

Extra line.

[zel, 2011/04/19 23:14:00]

Done.

+
 SandboxMountPointProvider* FileSystemContext::sandbox_provider() const {
   return path_manager_->sandbox_provider();
 }