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

webkit/fileapi/file_system_callback_dispatcher.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_FILE_SYSTEM_CALLBACK_DISPATCHER_H_
 #define WEBKIT_FILEAPI_FILE_SYSTEM_CALLBACK_DISPATCHER_H_
 
 #include <vector>
 
 #include "base/file_util_proxy.h"
 #include "base/platform_file.h"
 #include "base/process.h"
 
 class GURL;
 
 namespace fileapi {
 
 // This class mirrors the callbacks in
 // third_party/WebKit/Source/WebKit/chromium/public/WebFileSystemCallbacks.h,
 // but uses chromium types.
 class FileSystemCallbackDispatcher {
  public:
   virtual ~FileSystemCallbackDispatcher();
 
   // Callback for various operations that don't require return values.
   virtual void DidSucceed() = 0;
 
   // Callback to report information for a file.
+  virtual void DidGetLocalPath(const FilePath& local_path) = 0;

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

Move this down below DidOpenFile, as it's not one of those that mirrors
WebFileSystemCallbacks [as mentioned in the above comment].  Given how specific
it is in purpose, I think it makes sense to give it a default implementation
here with a NOTREACHED, just like DidOpenFile.

Also, for the comment, please indicate that it's only to be used when absolutely
unavoidable, and won't work in some cases.  And "report information" is a bit
generic; how about "look up the real local platform path of the file, where
possible".

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

Done.

+
+  // Callback to report information for a file.
   virtual void DidReadMetadata(
       const base::PlatformFileInfo& file_info,
       const FilePath& platform_path) = 0;
 
   // Callback to report the contents of a directory. If the contents of
   // the given directory are reported in one batch, then |entries| will have
   // the list of all files/directories in the given directory, |has_more| will
   // be false, and this callback will be called only once. If the contents of
   // the given directory are reported in multiple chunks, then this callback
   // will be called multiple times, |entries| will have only a subset of
@@ skipping 17 matching lines @@
   // Callback for OpenFile.  This isn't in WebFileSystemCallbacks, as it's just
   // for Pepper.
   virtual void DidOpenFile(
       base::PlatformFile file,
       base::ProcessHandle peer_handle);
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_FILEAPI_FILE_SYSTEM_CALLBACK_DISPATCHER_H_