Do PPB_FileIO Query and Read in the plugin process.

ppapi/proxy/file_io_resource.h

 // Copyright (c) 2012 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 PPAPI_PROXY_FILE_IO_RESOURCE_H_
 #define PPAPI_PROXY_FILE_IO_RESOURCE_H_
 
 #include <string>
 
 #include "ppapi/c/private/pp_file_handle.h"
@@ skipping 53 matching lines @@
                             scoped_refptr<TrackedCallback> callback) OVERRIDE;
   virtual int32_t WillSetLength(
       int64_t length,
       scoped_refptr<TrackedCallback> callback) OVERRIDE;
 
  private:
   int32_t ReadValidated(int64_t offset,
                         int32_t bytes_to_read,
                         const PP_ArrayOutput& array_output,
                         scoped_refptr<TrackedCallback> callback);
+  void CloseFileHandle();
 
-  // Handlers of reply messages. Note that all of them have a callback
-  // parameters bound when call to the host.
+  // Reply message handlers for operations that are done in the host.
   void OnPluginMsgGeneralComplete(scoped_refptr<TrackedCallback> callback,
                                   const ResourceMessageReplyParams& params);
   void OnPluginMsgOpenFileComplete(scoped_refptr<TrackedCallback> callback,
                                    const ResourceMessageReplyParams& params);
-  void OnPluginMsgQueryComplete(scoped_refptr<TrackedCallback> callback,
-                                PP_FileInfo* output_info_,
-                                const ResourceMessageReplyParams& params,
-                                const PP_FileInfo& info);
-  void OnPluginMsgReadComplete(scoped_refptr<TrackedCallback> callback,
-                               PP_ArrayOutput array_output,
-                               const ResourceMessageReplyParams& params,
-                               const std::string& data);
   void OnPluginMsgRequestOSFileHandleComplete(
       scoped_refptr<TrackedCallback> callback,
       PP_FileHandle* output_handle,
       const ResourceMessageReplyParams& params);
 
+  // When we pass a callback to FileUtilProxy code, we must keep the resource
+  // alive until it's called. However, we can't directly pass a reference to the
+  // resource, since that can cause our destructor to run when when the callback
+  // is destroyed, when we don't hold the proxy lock. Instead, pass a scoped_ptr
+  // to a scoped_refptr. This keeps the resource alive and allows us to take the
+  // reference so the destructor only runs while we hold the lock.
+  typedef scoped_ptr<scoped_refptr<FileIOResource> > PassedRef;
+
+  // These callbacks can be safely passed to FileUtilProxy. They take the lock,
+  // then take the resource reference and run the reply message handler. If
+  // this is the last reference, the destructor will run while we hold the lock.
+  static void QueryCallback(PassedRef file_io_ref,
+                            scoped_refptr<TrackedCallback> callback,
+                            PP_FileInfo* output_info,
+                            base::PlatformFileError error_code,
+                            const base::PlatformFileInfo& file_info);
+
+  static void ReadCallback(PassedRef file_io_ref,
+                           scoped_refptr<TrackedCallback> callback,
+                           PP_ArrayOutput array_output,
+                           base::PlatformFileError error_code,
+                           const char* data, int bytes_read);
+
+  // Reply message handlers for operations that are done in the plugin process.
+  // These should not be called directly.
+  void OnQueryComplete(scoped_refptr<TrackedCallback> callback,
+                       PP_FileInfo* output_info,
+                       base::PlatformFileError error_code,
+                       const base::PlatformFileInfo& file_info);
+  void OnReadComplete(scoped_refptr<TrackedCallback> callback,
+                      PP_ArrayOutput array_output,
+                      base::PlatformFileError error_code,
+                      const char* data, int bytes_read);
+
+  PP_FileHandle file_handle_;
+  PP_FileSystemType file_system_type_;
   FileIOStateManager state_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(FileIOResource);
 };
 
 }  // namespace proxy
 }  // namespace ppapi
 
 #endif  // PPAPI_PROXY_FILE_IO_RESOURCE_H_