Cleanup: Deprecate FileSystemCallbackDispatcher

content/common/fileapi/webfilesystem_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/webfilesystem_impl.h"
 
+#include "base/bind.h"
 #include "content/common/child_thread.h"
 #include "content/common/fileapi/file_system_dispatcher.h"
 #include "content/common/fileapi/webfilesystem_callback_dispatcher.h"
 #include "content/common/fileapi/webfilewriter_impl.h"
 #include "third_party/WebKit/Source/Platform/chromium/public/WebFileInfo.h"
 #include "third_party/WebKit/Source/Platform/chromium/public/WebString.h"
 #include "third_party/WebKit/Source/Platform/chromium/public/WebURL.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFileSystemCallbacks.h"
 #include "webkit/glue/webkit_glue.h"
 
 using WebKit::WebFileInfo;
 using WebKit::WebFileSystemCallbacks;
 using WebKit::WebFileSystemEntry;
 using WebKit::WebString;
 using WebKit::WebURL;
 using WebKit::WebVector;
 
 namespace content {
 
+typedef WebFileSystemCallbackDispatcher CallbackDispatcher;
+
 WebFileSystemImpl::WebFileSystemImpl() {
 }
 
 void WebFileSystemImpl::move(const WebURL& src_path,
                              const WebURL& dest_path,
                              WebFileSystemCallbacks* callbacks) {
   FileSystemDispatcher* dispatcher =
       ChildThread::current()->file_system_dispatcher();
   dispatcher->Move(GURL(src_path),
                    GURL(dest_path),
-                   new WebFileSystemCallbackDispatcher(callbacks));
+                   base::Bind(&CallbackDispatcher::DidFinish,
+                              base::Owned(new CallbackDispatcher(callbacks))));
 }
 
 void WebFileSystemImpl::copy(const WebURL& src_path,
                              const WebURL& dest_path,
                              WebFileSystemCallbacks* callbacks) {
   FileSystemDispatcher* dispatcher =
       ChildThread::current()->file_system_dispatcher();
   dispatcher->Copy(GURL(src_path),
                    GURL(dest_path),
-                   new WebFileSystemCallbackDispatcher(callbacks));
+                   base::Bind(&CallbackDispatcher::DidFinish,
+                              base::Owned(new CallbackDispatcher(callbacks))));
 }
 
 void WebFileSystemImpl::remove(const WebURL& path,
                                WebFileSystemCallbacks* callbacks) {
   FileSystemDispatcher* dispatcher =
       ChildThread::current()->file_system_dispatcher();
-  dispatcher->Remove(GURL(path),
-                     false /* recursive */,
-                     new WebFileSystemCallbackDispatcher(callbacks));
+  dispatcher->Remove(
+      GURL(path),
+      false /* recursive */,
+      base::Bind(&CallbackDispatcher::DidFinish,
+                 base::Owned(new CallbackDispatcher(callbacks))));
 }
 
 void WebFileSystemImpl::removeRecursively(const WebURL& path,
                                           WebFileSystemCallbacks* callbacks) {
   FileSystemDispatcher* dispatcher =
       ChildThread::current()->file_system_dispatcher();
-  dispatcher->Remove(GURL(path),
-                     true /* recursive */,
-                     new WebFileSystemCallbackDispatcher(callbacks));
+  dispatcher->Remove(
+      GURL(path),
+      true /* recursive */,
+      base::Bind(&CallbackDispatcher::DidFinish,
+                 base::Owned(new CallbackDispatcher(callbacks))));
 }
 
 void WebFileSystemImpl::readMetadata(const WebURL& path,
                                      WebFileSystemCallbacks* callbacks) {
   FileSystemDispatcher* dispatcher =
       ChildThread::current()->file_system_dispatcher();
-  dispatcher->ReadMetadata(GURL(path),
-                           new WebFileSystemCallbackDispatcher(callbacks));
+  dispatcher->ReadMetadata(
+      GURL(path),
+      base::Bind(&CallbackDispatcher::DidReadMetadata,
+                  base::Owned(new CallbackDispatcher(callbacks))));
 }
 
 void WebFileSystemImpl::createFile(const WebURL& path,
                                    bool exclusive,
                                    WebFileSystemCallbacks* callbacks) {
   FileSystemDispatcher* dispatcher =
       ChildThread::current()->file_system_dispatcher();
-  dispatcher->Create(GURL(path), exclusive, false,
-                     false, new WebFileSystemCallbackDispatcher(callbacks));
+  dispatcher->Create(
+      GURL(path), exclusive, false /* directory */, false /* recursive */,
+      base::Bind(&CallbackDispatcher::DidFinish,
+                 base::Owned(new CallbackDispatcher(callbacks))));
 }
 
 void WebFileSystemImpl::createDirectory(const WebURL& path,
                                         bool exclusive,
                                         WebFileSystemCallbacks* callbacks) {
   FileSystemDispatcher* dispatcher =
       ChildThread::current()->file_system_dispatcher();
-  dispatcher->Create(GURL(path), exclusive, true,
-                     false, new WebFileSystemCallbackDispatcher(callbacks));
+  dispatcher->Create(
+      GURL(path), exclusive, true /* directory */, false /* recursive */,
+      base::Bind(&CallbackDispatcher::DidFinish,
+                 base::Owned(new CallbackDispatcher(callbacks))));
 }
 
 void WebFileSystemImpl::fileExists(const WebURL& path,
                                    WebFileSystemCallbacks* callbacks) {
   FileSystemDispatcher* dispatcher =
       ChildThread::current()->file_system_dispatcher();
-  dispatcher->Exists(GURL(path), false,
-                     new WebFileSystemCallbackDispatcher(callbacks));
+  dispatcher->Exists(
+      GURL(path), false /* directory */,
+      base::Bind(&CallbackDispatcher::DidFinish,
+                 base::Owned(new CallbackDispatcher(callbacks))));
 }
 
 void WebFileSystemImpl::directoryExists(const WebURL& path,
                                         WebFileSystemCallbacks* callbacks) {
   FileSystemDispatcher* dispatcher =
       ChildThread::current()->file_system_dispatcher();
-  dispatcher->Exists(GURL(path), true,
-                     new WebFileSystemCallbackDispatcher(callbacks));
+  dispatcher->Exists(
+      GURL(path), true /* directory */,
+      base::Bind(&CallbackDispatcher::DidFinish,
+                 base::Owned(new CallbackDispatcher(callbacks))));
 }
 
 void WebFileSystemImpl::readDirectory(const WebURL& path,
                                       WebFileSystemCallbacks* callbacks) {
   FileSystemDispatcher* dispatcher =
       ChildThread::current()->file_system_dispatcher();
-  dispatcher->ReadDirectory(GURL(path),
-                            new WebFileSystemCallbackDispatcher(callbacks));
+  dispatcher->ReadDirectory(
+      GURL(path),
+      base::Bind(&CallbackDispatcher::DidReadDirectory,
+                 base::Owned(new CallbackDispatcher(callbacks))));
 }
 
 WebKit::WebFileWriter* WebFileSystemImpl::createFileWriter(
     const WebURL& path, WebKit::WebFileWriterClient* client) {
   return new WebFileWriterImpl(GURL(path), client);
 }
 
 void WebFileSystemImpl::createSnapshotFileAndReadMetadata(
     const WebKit::WebURL& path,
     WebKit::WebFileSystemCallbacks* callbacks) {
   FileSystemDispatcher* dispatcher =
       ChildThread::current()->file_system_dispatcher();
   dispatcher->CreateSnapshotFile(
-      GURL(path), new WebFileSystemCallbackDispatcher(callbacks));
+      GURL(path),
+      base::Bind(&CallbackDispatcher::DidCreateSnapshotFile,
+                 base::Owned(new CallbackDispatcher(callbacks))));
 }
 
 }  // namespace content