[FileAPI] Remove Flush() on write completion on SandboxFileSystems

webkit/browser/fileapi/file_system_context.cc

 // 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.
 
 #include "webkit/browser/fileapi/file_system_context.h"
 
 #include "base/bind.h"
 #include "base/single_thread_task_runner.h"
 #include "base/stl_util.h"
 #include "base/task_runner_util.h"
@@ skipping 461 matching lines @@
     return false;
 #if defined(OS_CHROMEOS)
   if (url.type() == kFileSystemTypeTemporary &&
       sandbox_backend_->enable_temporary_file_system_in_incognito()) {
     return true;
   }
 #endif
   return !is_incognito_ || !FileSystemContext::IsSandboxFileSystem(url.type());
 }
 
+bool FileSystemContext::ShouldFlushOnWriteCompletion(
+    FileSystemType type) const {
+  return !IsSandboxFileSystem(type);

[kinuko, 2014/05/02 05:26:16]

This'd look hacky or cryptic, can you comment some context here?

[tzik, 2014/05/02 06:55:10]

Added some comment.

+}
+
 void FileSystemContext::OpenPluginPrivateFileSystem(
     const GURL& origin_url,
     FileSystemType type,
     const std::string& filesystem_id,
     const std::string& plugin_id,
     OpenFileSystemMode mode,
     const StatusCallback& callback) {
   DCHECK(plugin_private_backend_);
   plugin_private_backend_->OpenPrivateFileSystem(
       origin_url, type, filesystem_id, plugin_id, mode, callback);
@@ skipping 115 matching lines @@
   } else if (parent != child) {
     bool result = parent.AppendRelativePath(child, &path);
     DCHECK(result);
   }
 
   operation_runner()->GetMetadata(
       url, base::Bind(&DidGetMetadataForResolveURL, path, callback, info));
 }
 
 }  // namespace fileapi