Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(81)

Unified Diff: webkit/fileapi/file_system_operation.cc

Issue 7066033: Enable cross-filesystem moves and copies. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rolled in Michael's feedback; removed unnecessary unlimited quota flag. Created 9 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webkit/fileapi/file_system_operation.cc
diff --git a/webkit/fileapi/file_system_operation.cc b/webkit/fileapi/file_system_operation.cc
index 9692cb3bd471c9aec4360b4387b4eb18af7c24d6..372b3113cce553cf59e90e56bb363ce635b14be8 100644
--- a/webkit/fileapi/file_system_operation.cc
+++ b/webkit/fileapi/file_system_operation.cc
@@ -150,19 +150,6 @@ void FileSystemOperation::Copy(const GURL& src_path,
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);
@@ -222,17 +209,6 @@ void FileSystemOperation::Move(const GURL& src_path,
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);

Powered by Google App Engine
This is Rietveld 408576698