Enable cross-filesystem moves and copies.

webkit/fileapi/file_system_operation.cc

 // Copyright (c) 2011 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/file_system_operation.h"
 
 #include "base/time.h"
 #include "base/utf_string_conversions.h"
 #include "net/base/escape.h"
 #include "net/url_request/url_request_context.h"
@@ skipping 132 matching lines @@
   FileSystemFileUtil* dest_file_system_file_util;
 
   if (!VerifyFileSystemPathForRead(src_path, &src_origin_url, &src_type,
         &virtual_path_0, &src_file_system_file_util) ||
       !VerifyFileSystemPathForWrite(dest_path, true /* create */,
           &dest_origin_url, &dest_type, &virtual_path_1,
           &dest_file_system_file_util)) {
     delete this;
     return;
   }
-  if (src_origin_url.GetOrigin() != dest_origin_url.GetOrigin()) {
-    // TODO(ericu): We don't yet support copying across filesystem types, from
-    // extension to sandbox, etc.  From temporary to persistent works, though.
-    // Since the sandbox code isn't in yet, I'm not sure exactly what check
-    // belongs here, but there's also no danger yet.
-    delete this;
-    return;
-  }
-  if (src_file_system_file_util != dest_file_system_file_util) {
-    // TODO(ericu): implement this.
-    delete this;
-    return;
-  }
   file_system_operation_context_.set_src_origin_url(src_origin_url);
   file_system_operation_context_.set_dest_origin_url(dest_origin_url);
   file_system_operation_context_.set_src_type(src_type);
   file_system_operation_context_.set_dest_type(dest_type);
   file_system_operation_context_.set_src_virtual_path(virtual_path_0);
   file_system_operation_context_.set_dest_virtual_path(virtual_path_1);
   if (!file_system_operation_context_.src_file_system_file_util())
     file_system_operation_context_.set_src_file_system_file_util(
         src_file_system_file_util);
   if (!file_system_operation_context_.dest_file_system_file_util())
@@ skipping 39 matching lines @@
   FileSystemFileUtil* dest_file_system_file_util;
 
   if (!VerifyFileSystemPathForWrite(src_path, false, &src_origin_url, &src_type,
         &virtual_path_0, &src_file_system_file_util) ||
       !VerifyFileSystemPathForWrite(dest_path, true /* create */,
           &dest_origin_url, &dest_type, &virtual_path_1,
           &dest_file_system_file_util)) {
     delete this;
     return;
   }
-  if (src_origin_url.GetOrigin() != dest_origin_url.GetOrigin()) {
-    // TODO(ericu): We don't yet support moving across filesystem types, from
-    // extension to sandbox, etc.  From temporary to persistent works, though.
-    delete this;
-    return;
-  }
-  if (src_file_system_file_util != dest_file_system_file_util) {
-    // TODO(ericu): implement this.
-    delete this;
-    return;
-  }
   file_system_operation_context_.set_src_origin_url(src_origin_url);
   file_system_operation_context_.set_dest_origin_url(dest_origin_url);
   file_system_operation_context_.set_src_type(src_type);
   file_system_operation_context_.set_dest_type(dest_type);
   file_system_operation_context_.set_src_virtual_path(virtual_path_0);
   file_system_operation_context_.set_dest_virtual_path(virtual_path_1);
   if (!file_system_operation_context_.src_file_system_file_util())
     file_system_operation_context_.set_src_file_system_file_util(
         src_file_system_file_util);
   if (!file_system_operation_context_.dest_file_system_file_util())
@@ skipping 622 matching lines @@
   }
   DCHECK(file_system_file_util);
   *file_system_file_util =
       file_system_context()->path_manager()->GetFileSystemFileUtil(*type);
   DCHECK(*file_system_file_util);
 
   return true;
 }
 
 }  // namespace fileapi