Make FileSystemOperation's lifetime more explicit.

webkit/fileapi/sandboxed_file_system_operation.cc

 // Copyright (c) 2010 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/fileapi/sandboxed_file_system_operation.h"
 
 #include "net/url_request/url_request_context.h"
 #include "webkit/fileapi/file_system_callback_dispatcher.h"
 #include "webkit/fileapi/file_system_path_manager.h"
 #include "webkit/fileapi/file_system_quota_manager.h"
@@ skipping 108 matching lines @@
                 const base::Time& last_access_time,
                 const base::Time& last_modified_time) {
   if (!VerifyFileSystemPathForWrite(path, true /* create */, 0))
     return;
   FileSystemOperation::TouchFile(path, last_access_time, last_modified_time);
 }
 
 void SandboxedFileSystemOperation::DidGetRootPath(
     bool success, const FilePath& path, const std::string& name) {
   DCHECK(success || path.empty());
-  dispatcher()->DidOpenFileSystem(name, path);
+  destructive_dispatcher()->DidOpenFileSystem(name, path);
 }
 
 bool SandboxedFileSystemOperation::VerifyFileSystemPathForRead(
     const FilePath& path) {
   // We may want do more checks, but for now it just checks if the given
   // |path| is under the valid FileSystem root path for this host context.
   if (!file_system_context_->path_manager()->CrackFileSystemPath(
           path, NULL, NULL, NULL)) {
-    dispatcher()->DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
+    destructive_dispatcher()->DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
     return false;
   }
   return true;
 }
 
 bool SandboxedFileSystemOperation::VerifyFileSystemPathForWrite(
     const FilePath& path, bool create, int64 growth) {
   GURL origin_url;
   FilePath virtual_path;
   if (!file_system_context_->path_manager()->CrackFileSystemPath(
           path, &origin_url, NULL, &virtual_path)) {
-    dispatcher()->DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
+    destructive_dispatcher()->DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
     return false;
   }
   // Any write access is disallowed on the root path.
   if (virtual_path.value().length() == 0 ||
       virtual_path.DirName().value() == virtual_path.value()) {
-    dispatcher()->DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
+    destructive_dispatcher()->DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
     return false;
   }
   if (create && file_system_context_->path_manager()->IsRestrictedFileName(
           path.BaseName())) {
-    dispatcher()->DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
+    destructive_dispatcher()->DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
     return false;
   }
   // TODO(kinuko): For operations with kUnknownSize we'll eventually
   // need to resolve what amount of size it's going to write.
   if (!file_system_context_->quota_manager()->CheckOriginQuota(
           origin_url, growth)) {
-    dispatcher()->DidFail(base::PLATFORM_FILE_ERROR_NO_SPACE);
+    destructive_dispatcher()->DidFail(base::PLATFORM_FILE_ERROR_NO_SPACE);
     return false;
   }
   return true;
 }
 
 bool SandboxedFileSystemOperation::CheckIfFilePathIsSafe(
     const FilePath& path) {
   if (file_system_context_->path_manager()->IsRestrictedFileName(
           path.BaseName())) {
-    dispatcher()->DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
+    destructive_dispatcher()->DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
     return false;
   }
   return true;
 }
 
 }  // namespace fileapi