[FileAPI] Add file open ID and close callback

webkit/fileapi/file_system_operation.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 WEBKIT_FILEAPI_FILE_SYSTEM_OPERATION_H_
 #define WEBKIT_FILEAPI_FILE_SYSTEM_OPERATION_H_
 
 #include <vector>
 
 #include "base/files/file_util_proxy.h"
@@ skipping 52 matching lines @@
   // of the file.
   typedef base::Callback<
       void(base::PlatformFileError result,
            const base::PlatformFileInfo& file_info,
            const base::FilePath& platform_path)> GetMetadataCallback;
 
   // Used for OpenFile(). |result| is the return code of the operation.
   typedef base::Callback<
       void(base::PlatformFileError result,
            base::PlatformFile file,
+           const base::Closure& close_callback,

[kinuko, 2013/04/25 05:55:52]

documentation?

Also close_callback makes me feel that the implementor needs to pass
ClosePlatformFile callback or something like that.

Maybe onclose_callback?

[tzik, 2013/04/25 06:43:46]

Done.

            base::ProcessHandle peer_handle)> OpenFileCallback;
 
   // Used for ReadDirectoryCallback.
   typedef std::vector<base::FileUtilProxy::Entry> FileEntryList;
 
   // Used for ReadDirectory(). |result| is the return code of the operation,
   // |file_list| is the list of files read, and |has_more| is true if some files
   // are yet to be read.
   typedef base::Callback<
       void(base::PlatformFileError result,
@@ skipping 188 matching lines @@
     kOperationOpenFile,
     kOperationCloseFile,
     kOperationGetLocalPath,
     kOperationCancel,
   };
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_FILEAPI_FILE_SYSTEM_OPERATION_H_