Move DirectoryReader::ReadEntries to FileRef::ReadDirectoryEntries

webkit/plugins/ppapi/file_callbacks.h

Index: webkit/plugins/ppapi/file_callbacks.h
diff --git a/webkit/plugins/ppapi/file_callbacks.h b/webkit/plugins/ppapi/file_callbacks.h
index c45932ab885a44532ab692201798f4b27544bbac..84f56c419189fa7abe572f67a1062226de966516 100644
--- a/webkit/plugins/ppapi/file_callbacks.h
+++ b/webkit/plugins/ppapi/file_callbacks.h
@@ -12,6 +12,7 @@
 #include "base/memory/weak_ptr.h"
 #include "base/platform_file.h"
 #include "googleurl/src/gurl.h"
+#include "ppapi/c/pp_array_output.h"
 #include "ppapi/c/pp_completion_callback.h"
 #include "ppapi/c/pp_file_info.h"
 #include "ppapi/c/pp_resource.h"
@@ -26,14 +27,24 @@ class FilePath;
 namespace ppapi {
 class Resource;
 class TrackedCallback;
+struct PPB_FileRef_CreateInfo;
 }
 
 namespace webkit {
 namespace ppapi {
 
+class PPB_FileRef_Impl;
+
 // Instances of this class are deleted by FileSystemDispatcher.
 class FileCallbacks : public fileapi::FileSystemCallbackDispatcher {
  public:
+  // Doesn't take the ownership of members.
+  struct ReadEntriesParams {
+    PPB_FileRef_Impl* dir_ref;
+    std::vector< ::ppapi::PPB_FileRef_CreateInfo>* files;

[palmer, 2013/05/03 00:07:24]

extra space

[hamaji, 2013/05/03 01:10:34]

I think we don't use <:: because it's a trigraph :(

+    std::vector<PP_FileType>* file_types;
+  };
+
   FileCallbacks(::ppapi::Resource* resource,
                 scoped_refptr< ::ppapi::TrackedCallback> callback,
                 PP_FileInfo* info);
@@ -41,6 +52,9 @@ class FileCallbacks : public fileapi::FileSystemCallbackDispatcher {
                 scoped_refptr< ::ppapi::TrackedCallback> callback,
                 PP_FileInfo* info,
                 PP_FileSystemType file_system_type);
+  FileCallbacks(::ppapi::Resource* resource,
+                scoped_refptr< ::ppapi::TrackedCallback> callback,

[palmer, 2013/05/03 00:07:24]

space

[hamaji, 2013/05/03 01:10:34]

ditto

+                const ReadEntriesParams& params);
   virtual ~FileCallbacks();
 
   // FileSystemCallbackDispatcher implementation.
@@ -66,6 +80,9 @@ class FileCallbacks : public fileapi::FileSystemCallbackDispatcher {
   scoped_refptr< ::ppapi::TrackedCallback> callback_;
   PP_FileInfo* info_;
   PP_FileSystemType file_system_type_;
+  PPB_FileRef_Impl* read_entries_dir_ref_;
+  std::vector< ::ppapi::PPB_FileRef_CreateInfo>* read_entries_files_;

[palmer, 2013/05/03 00:07:24]

space

[hamaji, 2013/05/03 01:10:34]

ditto

+  std::vector<PP_FileType>* read_entries_file_types_;

[dmichael (off chromium), 2013/05/02 20:39:13]

As I noted elsewhere, I think these may need to be linked_ptr to avoid
use-after-free.

(This is kind of ugly, but probably good enough for the interim time before
FileRef is refactored)

[hamaji, 2013/05/02 22:45:14]

Done. As we've chatted, I also modified Query as well. Query was actually more
tricky than ReadEntries because it also supports in-process mode as well. I
think I managed to solve the problem by introducing a hack called
maybe_linked_ptr. Please let me know if there is a better solution for this.

[dmichael (off chromium), 2013/05/02 23:20:43]

Nobody actually uses FileRef::Query in-process; we should just kill it off now
and not add maybe_linked_ptr.

[hamaji, 2013/05/02 23:36:33]

A good news! Done

 };
 
 }  // namespace ppapi