Notify CloseFile from Pepper to FileSystem.

webkit/plugins/ppapi/plugin_delegate.h

Index: webkit/plugins/ppapi/plugin_delegate.h
diff --git a/webkit/plugins/ppapi/plugin_delegate.h b/webkit/plugins/ppapi/plugin_delegate.h
index 6f851907bdded5f64c17342ce3b35da16abab203..003e75691bebed3524d3cb55502132aab7455e33 100644
--- a/webkit/plugins/ppapi/plugin_delegate.h
+++ b/webkit/plugins/ppapi/plugin_delegate.h
@@ -424,6 +424,10 @@ class PluginDelegate {
       int flags,
       const AsyncOpenFileCallback& callback) = 0;
 
+  // Retrieves a callback for doing clean-up task for closing filesystem URL.
+  virtual base::Callback<void (base::PlatformFileError)>

[yzshen1, 2012/06/28 20:14:17]

It seems more intuitive to have such an interface:
- Don't have a separate GetCloseFileSystemURLCallback method;
AsyncOpenFileSystemURL will take a new type of callback,
AsyncOpenFile*SystemURL*Callback.

- AsyncOpenFileSystemURLCallback has one more input parameter, which is a close
callback. This callback has remembered the corresponding plugin delegate
instance, as well as the file system URL.
If the open succeeds, the user is responsible for closing the file handle and
then call the close callback. They can easily do this by passing both of them to
base::FileUtilProxy::Close.

(Maybe it is just more intuitive to me? :))

[kinaba, 2012/06/29 09:28:33]

No reason to object. Done.

+      GetCloseFileSystemURLCallback(const GURL& path) = 0;
+
   virtual bool OpenFileSystem(
       const GURL& url,
       fileapi::FileSystemType type,