PPAPI: Add PluginPrivateFileSystem

ppapi/proxy/file_system_resource.cc

 // Copyright (c) 2013 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.
 
 #include "ppapi/proxy/file_system_resource.h"
 
 #include "base/bind.h"
 #include "ipc/ipc_message.h"
 #include "ppapi/c/pp_errors.h"
 #include "ppapi/proxy/ppapi_messages.h"
 #include "ppapi/shared_impl/tracked_callback.h"
 
 using ppapi::thunk::PPB_FileSystem_API;
 
 namespace ppapi {
 namespace proxy {
 
 FileSystemResource::FileSystemResource(Connection connection,
                                        PP_Instance instance,
                                        PP_FileSystemType type)
     : PluginResource(connection, instance),
       type_(type),
       called_open_(false),
-      callback_count_(0) {
+      callback_count_(0),
+      callback_result_(PP_OK) {
   DCHECK(type_ != PP_FILESYSTEMTYPE_INVALID);
   SendCreate(RENDERER, PpapiHostMsg_FileSystem_Create(type_));
   SendCreate(BROWSER, PpapiHostMsg_FileSystem_Create(type_));
 }
 
 FileSystemResource::FileSystemResource(Connection connection,
                                        PP_Instance instance,
                                        int pending_renderer_id,
                                        int pending_browser_id,
                                        PP_FileSystemType type)
     : PluginResource(connection, instance),
       type_(type),
-      called_open_(true) {
+      called_open_(true),
+      callback_count_(0),
+      callback_result_(PP_OK) {
   DCHECK(type_ != PP_FILESYSTEMTYPE_INVALID);
   AttachToPendingHost(RENDERER, pending_renderer_id);
   AttachToPendingHost(BROWSER, pending_browser_id);
 }
 
 FileSystemResource::~FileSystemResource() {
 }
 
 PPB_FileSystem_API* FileSystemResource::AsPPB_FileSystem_API() {
   return this;
@@ skipping 44 matching lines @@
       base::Bind(&FileSystemResource::InitIsolatedFileSystemComplete,
       this,
       callback));
   return PP_OK_COMPLETIONPENDING;
 }
 
 void FileSystemResource::OpenComplete(
     scoped_refptr<TrackedCallback> callback,
     const ResourceMessageReplyParams& params) {
   ++callback_count_;
+  if (params.result() != PP_OK)
+    callback_result_ = params.result();

[dmichael (off chromium), 2013/11/14 18:09:31]

Please comment on why you need to save the callback_result_.

[nhiroki, 2013/11/15 12:24:01]

Done.

   // Received callback from browser and renderer.
   if (callback_count_ == 2)
-    callback->Run(params.result());
+    callback->Run(callback_result_);
 }
 
 void FileSystemResource::InitIsolatedFileSystemComplete(
     const base::Callback<void(int32_t)>& callback,
     const ResourceMessageReplyParams& params) {
   ++callback_count_;
+  if (params.result() != PP_OK)
+    callback_result_ = params.result();
   // Received callback from browser and renderer.
   if (callback_count_ == 2)
-    callback.Run(params.result());
+    callback.Run(callback_result_);
 }
 
 }  // namespace proxy
 }  // namespace ppapi