Send OnModifyFile Notification when File Write Finishes

webkit/chromeos/fileapi/remote_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_CHROMEOS_FILEAPI_REMOTE_FILE_SYSTEM_OPERATION_H_
 #define WEBKIT_CHROMEOS_FILEAPI_REMOTE_FILE_SYSTEM_OPERATION_H_
 
 #include "webkit/chromeos/fileapi/remote_file_system_proxy.h"
 #include "webkit/fileapi/file_system_operation.h"
+#include "webkit/fileapi/file_writer_delegate.h"
 
 namespace base {
 class Value;
 }
 
 namespace fileapi {
 class FileWriterDelegate;
 class LocalFileSystemOperation;
 }
 
@@ skipping 75 matching lines @@
                       base::PlatformFileError rv,
                       const base::PlatformFileInfo& file_info,
                       const FilePath& platform_path);
   void DidReadDirectory(
       const ReadDirectoryCallback& callback,
       base::PlatformFileError rv,
       const std::vector<base::FileUtilProxy::Entry>& entries,
       bool has_more);
   void DidWrite(base::PlatformFileError result,
                 int64 bytes,
-                bool complete);
+                fileapi::FileWriterDelegate::WriteCompletionStatus write_status
+                );

[kinuko, 2012/09/25 14:00:49]

style-nit: can you format this function like the one above so that we don't need
to put ');' on a separate line

void DidWrite(
    base::PlatformFileError result,
    int64 bytes,
    ...);

[calvinlo, 2012/09/26 05:23:07]

Originally on its own new line because the previous line was 81 chars long.
Added typedef to shorten the type so it's not as long and moved the ); back up
one line.

   void DidFinishFileOperation(const StatusCallback& callback,
                               base::PlatformFileError rv);
   void DidCreateSnapshotFile(
       const SnapshotFileCallback& callback,
       base::PlatformFileError result,
       const base::PlatformFileInfo& file_info,
       const FilePath& platform_path,
       const scoped_refptr<webkit_blob::ShareableFileReference>& file_ref);
   void DidOpenFile(
       const OpenFileCallback& callback,
       base::PlatformFileError result,
       base::PlatformFile file,
       base::ProcessHandle peer_handle);
 
 
   scoped_refptr<fileapi::RemoteFileSystemProxyInterface> remote_proxy_;
   // A flag to make sure we call operation only once per instance.
   OperationType pending_operation_;
   scoped_ptr<fileapi::FileWriterDelegate> file_writer_delegate_;
 
   WriteCallback write_callback_;
   StatusCallback cancel_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(RemoteFileSystemOperation);
 };
 
 }  // namespace chromeos
 
 #endif  // WEBKIT_CHROMEOS_FILEAPI_REMOTE_FILE_SYSTEM_OPERATION_H_