Pepper quota support

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 ebfda188af936f58ec03bddf34f870390aabf040..ce75031493f78fb3f52768d9359341f4b78df6de 100644
--- a/webkit/plugins/ppapi/plugin_delegate.h
+++ b/webkit/plugins/ppapi/plugin_delegate.h
@@ -24,6 +24,7 @@
 #include "ui/gfx/size.h"
 #include "webkit/fileapi/file_system_types.h"
 #include "webkit/plugins/ppapi/dir_contents.h"
+#include "webkit/quota/quota_client.h"
 
 class AudioMessageFilter;
 class GURL;
@@ -294,8 +295,9 @@ class PluginDelegate {
       WebKit::WebFileChooserCompletion* chooser_completion) = 0;
 
   // Sends an async IPC to open a file.
-  typedef Callback2<base::PlatformFileError, base::PassPlatformFile
-                    >::Type AsyncOpenFileCallback;
+  typedef Callback3<base::PlatformFileError,
+                    base::PassPlatformFile,
+                    int64_t /* file_size */>::Type AsyncOpenFileCallback;

[michaeln, 2011/07/20 21:28:33]

How is this size used? There's no guarantee that it's accurate assuming that
others may be writing/truncating it.

[kinuko, 2011/07/21 14:34:08]

No, but I don't think we can strictly guarantee accurate counting in the current
model.  For WillWrite/WillSetLength it's almost impossible to perform
GetFileSize and Write/Truncate in a single task, and for untrusted writes by API
we still cannot avoid a stale size situation unless we flush after each Write
call, since we keep file descriptors opened for FileIO.

Anyway I dropped this size parameter from the OpenFile callback and modified the
code to perform GetFileInfo and Write as closely as possible.

   virtual bool AsyncOpenFile(const FilePath& path,
                              int flags,
                              AsyncOpenFileCallback* callback) = 0;
@@ -327,6 +329,17 @@ class PluginDelegate {
       const GURL& directory_path,
       fileapi::FileSystemCallbackDispatcher* dispatcher) = 0;
 
+  // For quota handlings for FileIO API.
+  typedef Callback1<int64>::Type AvailableSpaceCallback;
+  virtual bool QueryAvailableSpace(

[michaeln, 2011/07/20 21:28:33]

Does the return value mean anything? I don't see the callsites checking it,
maybe there shouldn't be a return value.

[kinuko, 2011/07/21 14:34:08]

Done.

+      const GURL& origin,
+      quota::StorageType type,
+      AvailableSpaceCallback* callback) = 0;
+  virtual void NotifyStorageModified(quota::QuotaClient::ID client_id,
+                                     const GURL& origin,
+                                     quota::StorageType type,
+                                     int64 delta) = 0;
+
   virtual base::PlatformFileError OpenFile(const PepperFilePath& path,
                                            int flags,
                                            base::PlatformFile* file) = 0;