Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(346)

Unified Diff: webkit/chromeos/fileapi/remote_file_system_operation.h

Issue 10956064: Send OnModifyFile Notification when File Write Finishes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added write status inside FileWriteDelegate so onFileModify notifications are only sent when a file… Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webkit/chromeos/fileapi/remote_file_system_operation.h
diff --git a/webkit/chromeos/fileapi/remote_file_system_operation.h b/webkit/chromeos/fileapi/remote_file_system_operation.h
index b912755b65bc5ff0b0e7b0dd3a51144c9cbb9066..e0e8289adabfd22925e3dbfa5340755b4c5ee6ef 100644
--- a/webkit/chromeos/fileapi/remote_file_system_operation.h
+++ b/webkit/chromeos/fileapi/remote_file_system_operation.h
@@ -7,6 +7,7 @@
#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;
@@ -102,7 +103,8 @@ class RemoteFileSystemOperation : public fileapi::FileSystemOperation {
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 o
calvinlo 2012/09/26 05:23:07 Originally on its own new line because the previou
void DidFinishFileOperation(const StatusCallback& callback,
base::PlatformFileError rv);
void DidCreateSnapshotFile(
« no previous file with comments | « no previous file | webkit/chromeos/fileapi/remote_file_system_operation.cc » ('j') | webkit/fileapi/file_writer_delegate.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698