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

webkit/fileapi/local_file_system_file_util.h

 // Copyright (c) 2011 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.
 
 #ifndef WEBKIT_FILEAPI_LOCAL_FILE_SYSTEM_FILE_UTIL_H_
 #define WEBKIT_FILEAPI_LOCAL_FILE_SYSTEM_FILE_UTIL_H_
 
 #include <vector>
 
 #include "base/callback.h"
@@ skipping 31 matching lines @@
       FileSystemOperationContext* context,
       const FilePath& file_path,
       int file_flags,
       PlatformFile* file_handle,
       bool* created);
 
   virtual PlatformFileError EnsureFileExists(
       FileSystemOperationContext* context,
       const FilePath& file_path, bool* created);
 
+  virtual PlatformFileError GetLocalFilePath(
+      FileSystemOperationContext* context,
+      const FilePath& virtual_file,
+      FilePath* local_path);
+
   virtual PlatformFileError GetFileInfo(
       FileSystemOperationContext* context,
-      const FilePath& file_,
+      const FilePath& file,
       base::PlatformFileInfo* file_info,
       FilePath* platform_file);
 
+  virtual FilePath GetLocalPath(

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

Why is this now virtual and public?  I think external calls are all going into
GetLocalFilePath.

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

Done.

+      FileSystemOperationContext* context,
+      const GURL& origin_url,
+      FileSystemType type,
+      const FilePath& virtual_path);
+
   virtual PlatformFileError ReadDirectory(
       FileSystemOperationContext* context,
       const FilePath& file_path,
       std::vector<base::FileUtilProxy::Entry>* entries);
 
   virtual PlatformFileError CreateDirectory(
       FileSystemOperationContext* context,
       const FilePath& file_path,
       bool exclusive,
       bool recursive);
@@ skipping 24 matching lines @@
       const FilePath& path,
       int64 length);
 
  protected:
   LocalFileSystemFileUtil() { }
 
   friend struct DefaultSingletonTraits<LocalFileSystemFileUtil>;
   DISALLOW_COPY_AND_ASSIGN(LocalFileSystemFileUtil);
 
  private:
-  // Given the filesystem's root URL and a virtual path, produces a real, full
-  // local path.
-  FilePath GetLocalPath(
-      FileSystemOperationContext* context,
-      const GURL& origin_url,
-      FileSystemType type,
-      const FilePath& virtual_path);
-
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_FILEAPI_LOCAL_FILE_SYSTEM_FILE_UTIL_H_