Make FileSystemOperation NOT self-destruct

webkit/browser/fileapi/syncable/syncable_file_system_operation.h

 // Copyright 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_SYNCABLE_SYNCABLE_FILE_SYSTEM_OPERATION_H_
 #define WEBKIT_BROWSER_FILEAPI_SYNCABLE_SYNCABLE_FILE_SYSTEM_OPERATION_H_
 
 #include <vector>
 
 #include "base/callback.h"
@@ skipping 10 matching lines @@
 class SandboxMountPointProvider;
 }
 
 namespace sync_file_system {
 
 class SyncableFileOperationRunner;
 
 // A wrapper class of LocalFileSystemOperation for syncable file system.
 class WEBKIT_STORAGE_EXPORT SyncableFileSystemOperation
     : public fileapi::LocalFileSystemOperation,
+      public base::SupportsWeakPtr<SyncableFileSystemOperation>,
       public base::NonThreadSafe {
  public:
   virtual ~SyncableFileSystemOperation();
 
   // fileapi::FileSystemOperation overrides.
   virtual void CreateFile(const fileapi::FileSystemURL& url,
                           bool exclusive,
                           const StatusCallback& callback) OVERRIDE;
   virtual void CreateDirectory(const fileapi::FileSystemURL& url,
                                bool exclusive,
@@ skipping 33 matching lines @@
   virtual void Cancel(const StatusCallback& cancel_callback) OVERRIDE;
   virtual void CreateSnapshotFile(
       const fileapi::FileSystemURL& path,
       const SnapshotFileCallback& callback) OVERRIDE;
 
   // LocalFileSystemOperation overrides.
   virtual void CopyInForeignFile(const base::FilePath& src_local_disk_path,
                                  const fileapi::FileSystemURL& dest_url,
                                  const StatusCallback& callback) OVERRIDE;
 
+  using base::SupportsWeakPtr<SyncableFileSystemOperation>::AsWeakPtr;
+
  private:
   typedef SyncableFileSystemOperation self;
   class QueueableTask;
 
   // Only MountPointProviders can create a new operation directly.
   friend class fileapi::SandboxMountPointProvider;
   friend class SandboxMountPointProvider;
   SyncableFileSystemOperation(
       const fileapi::FileSystemURL& url,
       fileapi::FileSystemContext* file_system_context,
       scoped_ptr<fileapi::FileSystemOperationContext> operation_context);
   fileapi::LocalFileSystemOperation* NewOperation();
 
   void DidFinish(base::PlatformFileError status);
   void DidWrite(const WriteCallback& callback,
                 base::PlatformFileError result,
                 int64 bytes,
                 bool complete);
 
   void OnCancelled();
-  void AbortOperation(const StatusCallback& callback,
-                      base::PlatformFileError error);
 
   const fileapi::FileSystemURL url_;
 
   base::WeakPtr<SyncableFileOperationRunner> operation_runner_;
-  fileapi::LocalFileSystemOperation* inflight_operation_;
+  scoped_ptr<fileapi::LocalFileSystemOperation> inflight_operation_;
   std::vector<fileapi::FileSystemURL> target_paths_;
 
   StatusCallback completion_callback_;
 
   bool is_directory_operation_enabled_;
 
   DISALLOW_COPY_AND_ASSIGN(SyncableFileSystemOperation);
 };
 
 }  // namespace sync_file_system
 
 #endif  // WEBKIT_BROWSER_FILEAPI_SYNCABLE_SYNCABLE_FILE_SYSTEM_OPERATION_H_