Change webkit/{fileapi,quota} code to use TaskRunner.

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 <string>
 #include <vector>
 
 #include "base/file_path.h"
 #include "base/file_util_proxy.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
-#include "base/message_loop_proxy.h"
 #include "base/platform_file.h"
 #include "base/process.h"
 #include "googleurl/src/gurl.h"
 #include "webkit/fileapi/file_system_operation_context.h"
 #include "webkit/fileapi/file_system_operation_interface.h"
 #include "webkit/fileapi/file_system_types.h"
 #include "webkit/quota/quota_manager.h"
 
 namespace base {
 class Time;
+class SequencedTaskRunner;
 }
 
 namespace chromeos {
 class CrosMountPointProvider;
 }
 
 namespace net {
 class URLRequest;
 class URLRequestContext;
 }  // namespace net
@@ skipping 86 matching lines @@
   friend class IsolatedMountPointProvider;
   friend class SandboxMountPointProvider;
   friend class chromeos::CrosMountPointProvider;
 
   friend class FileSystemOperationTest;
   friend class FileSystemOperationWriteTest;
   friend class FileWriterDelegateTest;
   friend class FileSystemTestOriginHelper;
   friend class FileSystemQuotaTest;
 
-  FileSystemOperation(scoped_refptr<base::MessageLoopProxy> proxy,
+  FileSystemOperation(base::SequencedTaskRunner* task_runner,

[michaeln, 2012/04/27 22:31:08]

similarly, maybe the 'runner' could be picked up from the context

[kinuko, 2012/05/04 19:05:35]

Done.

                       FileSystemContext* file_system_context);
 
   FileSystemContext* file_system_context() const {
     return operation_context_.file_system_context();
   }
 
   FileSystemOperationContext* file_system_operation_context() {
     return &operation_context_;
   }
 
@@ skipping 85 matching lines @@
   base::PlatformFileError SetUpFileSystemPath(
       const GURL& path_url,
       FileSystemPath* file_system_path,
       FileSystemFileUtil** file_util,
       SetUpPathMode mode);
 
   // Used only for internal assertions.
   // Returns false if there's another inflight pending operation.
   bool SetPendingOperationType(OperationType type);
 
-  // Proxy for calling file_util_proxy methods.
-  scoped_refptr<base::MessageLoopProxy> proxy_;
+  // Task runner for making file operations.
+  scoped_refptr<base::SequencedTaskRunner> task_runner_;
 
   FileSystemOperationContext operation_context_;
   FileSystemPath src_path_;
   FileSystemPath dest_path_;
   FileSystemFileUtil* src_util_;  // Not owned.
   FileSystemFileUtil* dest_util_;  // Not owned.
 
   // This is set before any write operations.  The destructor of
   // ScopedQuotaNotifier sends notification to the QuotaManager
   // to tell the update is done; so that we can make sure notify
@@ skipping 21 matching lines @@
 
   // A flag to make sure we call operation only once per instance.
   OperationType pending_operation_;
 
   DISALLOW_COPY_AND_ASSIGN(FileSystemOperation);
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_FILEAPI_FILE_SYSTEM_OPERATION_H_