Cleanup: Deprecate FileSystemCallbackDispatcher

content/common/fileapi/webfilewriter_impl.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 "content/common/fileapi/webfilewriter_impl.h"
 
+#include "base/bind.h"
 #include "content/common/child_thread.h"
 #include "content/common/fileapi/file_system_dispatcher.h"
 
 namespace content {
 namespace {
 
 inline FileSystemDispatcher* GetFileSystemDispatcher() {
   return ChildThread::current()->file_system_dispatcher();
 }
 }
 
-class WebFileWriterImpl::CallbackDispatcher
-    : public fileapi::FileSystemCallbackDispatcher {
+class WebFileWriterImpl::CallbackDispatcher {
  public:
+  ~CallbackDispatcher() {}
+
+  static FileSystemDispatcher::StatusCallback GetStatusCallback(
+      const base::WeakPtr<WebFileWriterImpl>& writer) {
+    CallbackDispatcher* dispatcher = new CallbackDispatcher(writer);
+    return base::Bind(&CallbackDispatcher::DidFinish, base::Owned(dispatcher));
+  }
+
+  static FileSystemDispatcher::WriteCallback GetWriteCallback(
+      const base::WeakPtr<WebFileWriterImpl>& writer) {
+    CallbackDispatcher* dispatcher = new CallbackDispatcher(writer);
+    return base::Bind(&CallbackDispatcher::DidWrite, base::Owned(dispatcher));
+  }
+
+  void DidFinish(base::PlatformFileError error_code) {
+    if (!writer_)
+      return;
+    if (error_code == base::PLATFORM_FILE_OK)
+      writer_->DidSucceed();
+    else
+      writer_->DidFail(error_code);
+  }
+
+  void DidWrite(base::PlatformFileError error_code,
+                int64 bytes, bool complete) {
+    if (!writer_)
+      return;
+    if (error_code == base::PLATFORM_FILE_OK)
+      writer_->DidWrite(bytes, complete);
+    else
+      writer_->DidFail(error_code);
+  }
+
+ private:
   explicit CallbackDispatcher(
       const base::WeakPtr<WebFileWriterImpl>& writer) : writer_(writer) {
   }
-  virtual ~CallbackDispatcher() {
-  }
-  virtual void DidReadMetadata(
-      const base::PlatformFileInfo&,
-      const base::FilePath&) OVERRIDE {
-    NOTREACHED();
-  }
-  virtual void DidCreateSnapshotFile(
-      const base::PlatformFileInfo&,
-      const base::FilePath&) OVERRIDE {
-    NOTREACHED();
-  }
-  virtual void DidReadDirectory(
-      const std::vector<base::FileUtilProxy::Entry>& entries,
-      bool has_more) OVERRIDE {
-    NOTREACHED();
-  }
-  virtual void DidOpenFileSystem(const std::string& name,
-                                 const GURL& root) OVERRIDE {
-    NOTREACHED();
-  }
-  virtual void DidSucceed() OVERRIDE {
-    if (writer_)
-      writer_->DidSucceed();
-  }
-  virtual void DidFail(base::PlatformFileError error_code) OVERRIDE {
-    if (writer_)
-      writer_->DidFail(error_code);
-  }
-  virtual void DidWrite(int64 bytes, bool complete) OVERRIDE {
-    if (writer_)
-      writer_->DidWrite(bytes, complete);
-  }
 
- private:
   base::WeakPtr<WebFileWriterImpl> writer_;
 };
 
 WebFileWriterImpl::WebFileWriterImpl(
      const GURL& path, WebKit::WebFileWriterClient* client)
   : WebFileWriterBase(path, client),
     request_id_(0) {
 }
 
 WebFileWriterImpl::~WebFileWriterImpl() {
 }
 
 void WebFileWriterImpl::DoTruncate(const GURL& path, int64 offset) {
   // The FileSystemDispatcher takes ownership of the CallbackDispatcher.
-  GetFileSystemDispatcher()->Truncate(path, offset, &request_id_,
-                                      new CallbackDispatcher(AsWeakPtr()));
+  GetFileSystemDispatcher()->Truncate(
+      path, offset, &request_id_,
+      CallbackDispatcher::GetStatusCallback(AsWeakPtr()));
 }
 
 void WebFileWriterImpl::DoWrite(
     const GURL& path, const GURL& blob_url, int64 offset) {
   GetFileSystemDispatcher()->Write(
       path, blob_url, offset, &request_id_,
-      new CallbackDispatcher(AsWeakPtr()));
+      CallbackDispatcher::GetWriteCallback(AsWeakPtr()));
 }
 
 void WebFileWriterImpl::DoCancel() {
-  GetFileSystemDispatcher()->Cancel(request_id_,
-                                    new CallbackDispatcher(AsWeakPtr()));
+  GetFileSystemDispatcher()->Cancel(
+      request_id_,
+      CallbackDispatcher::GetStatusCallback(AsWeakPtr()));
 }
 
 }  // namespace content