Cosmetic change for disk flush interval in Files.app file copying.

webkit/browser/fileapi/copy_or_move_operation_delegate.h

 // 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.
 
 #ifndef WEBKIT_BROWSER_FILEAPI_COPY_OR_MOVE_OPERATION_DELEGATE_H_
 #define WEBKIT_BROWSER_FILEAPI_COPY_OR_MOVE_OPERATION_DELEGATE_H_
 
 #include <set>
 #include <stack>
 
@@ skipping 55 matching lines @@
     void Read(const StatusCallback& callback);
     void DidRead(const StatusCallback& callback, int result);
 
     // Writes the content in |buffer| to |writer_|.
     void Write(const StatusCallback& callback,
                scoped_refptr<net::DrainableIOBuffer> buffer);
     void DidWrite(const StatusCallback& callback,
                   scoped_refptr<net::DrainableIOBuffer> buffer, int result);
 
     // Flushes the written content in |writer_|.
-    void Flush(const StatusCallback& callback);
-    void DidFlush(const StatusCallback& callback, int result);
+    void Flush(const StatusCallback& callback, bool is_eof);
+    void DidFlush(const StatusCallback& callback, bool is_eof, int result);
 
     scoped_ptr<webkit_blob::FileStreamReader> reader_;
     scoped_ptr<FileStreamWriter> writer_;
     const bool need_flush_;
     FileSystemOperation::CopyFileProgressCallback file_progress_callback_;
     scoped_refptr<net::IOBufferWithSize> io_buffer_;
     int64 num_copied_bytes_;
+    int64 previous_flush_;

[kinuko, 2013/12/13 10:42:01]

nit: previous_flush_ -> previous_flush_offset_ or something? (I first thought
this might be related to time)

[kinaba, 2013/12/14 05:14:19]

sg. done.

     base::Time last_progress_callback_invocation_time_;
     base::TimeDelta min_progress_callback_invocation_span_;
     bool cancel_requested_;
     base::WeakPtrFactory<StreamCopyHelper> weak_factory_;
     DISALLOW_COPY_AND_ASSIGN(StreamCopyHelper);
   };
 
   CopyOrMoveOperationDelegate(
       FileSystemContext* file_system_context,
       const FileSystemURL& src_root,
@@ skipping 57 matching lines @@
 
   std::set<CopyOrMoveImpl*> running_copy_set_;
   base::WeakPtrFactory<CopyOrMoveOperationDelegate> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(CopyOrMoveOperationDelegate);
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_BROWSER_FILEAPI_COPY_OR_MOVE_OPERATION_DELEGATE_H_