Enables internal filesystem types via Isolated filesystems

webkit/fileapi/isolated_file_util_unittest.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 <map>
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/file_util.h"
@@ skipping 32 matching lines @@
 };
 
 FilePath GetTopLevelPath(const FilePath& path) {
   std::vector<FilePath::StringType> components;
   path.GetComponents(&components);
   return FilePath(components[0]);
 }
 
 }  // namespace
 
+// TODO(kinuko): we should have separate tests for DraggedFileUtil and
+// IsolatedFileUtil.
 class IsolatedFileUtilTest : public testing::Test {
  public:
   IsolatedFileUtilTest() {}
 
   void SetUp() {
     ASSERT_TRUE(data_dir_.CreateUniqueTempDir());
-    file_util_.reset(new IsolatedFileUtil());
+    file_util_.reset(new DraggedFileUtil());
 
     // Register the files/directories of RegularTestCases (with random
     // root paths) as dropped files.
     SimulateDropFiles();
 
     file_system_context_ = new FileSystemContext(
         base::MessageLoopProxy::current(),
         base::MessageLoopProxy::current(),
         make_scoped_refptr(new quota::MockSpecialStoragePolicy()),
         NULL /* quota_manager */,
@@ skipping 129 matching lines @@
       const test::TestCaseRecord& test_case = test::kRegularTestCases[i];
       FilePath path(test_case.path);
       FilePath toplevel = GetTopLevelPath(path);
 
       // We create the test case files under one of the kRootPaths
       // to simulate a drop with multiple directories.
       if (toplevel_root_map_.find(toplevel) == toplevel_root_map_.end()) {
         FilePath root = root_path().Append(
             kRootPaths[(root_path_index++) % arraysize(kRootPaths)]);
         toplevel_root_map_[toplevel] = root;
-        toplevels.AddPath(root.Append(path));
+        toplevels.AddPath(root.Append(path), NULL);
       }
 
       test::SetUpOneTestCase(toplevel_root_map_[toplevel], test_case);
     }
 
     // Register the toplevel entries.
-    filesystem_id_ = isolated_context()->RegisterFileSystem(toplevels);
+    filesystem_id_ = isolated_context()->RegisterDraggedFileSystem(toplevels);
   }
 
   ScopedTempDir data_dir_;
   MessageLoop message_loop_;
   std::string filesystem_id_;
   scoped_refptr<FileSystemContext> file_system_context_;
   std::map<FilePath, FilePath> toplevel_root_map_;
   scoped_ptr<IsolatedFileUtil> file_util_;
   scoped_ptr<LocalFileUtil> other_file_util_;
   FileSystemTestOriginHelper other_file_util_helper_;
@@ skipping 160 matching lines @@
                   false /* exclusive */,
                   true /* recursive */));
 
     context.reset(new FileSystemOperationContext(file_system_context()));
     ASSERT_EQ(base::PLATFORM_FILE_OK,
               FileUtilHelper::Copy(
                   context.get(),
                   file_util(), other_file_util(),
                   src_url, dest_url));
 
-    // The other way (copy-in) should not work.
-    context.reset(new FileSystemOperationContext(file_system_context()));
-    ASSERT_EQ(base::PLATFORM_FILE_ERROR_SECURITY,
-              FileUtilHelper::Copy(
-                  context.get(),
-                  other_file_util(), file_util(),
-                  dest_url, src_url));
-
     VerifyFilesHaveSameContent(file_util(), other_file_util(),
                                src_url, dest_url);
   }
 }
 
 TEST_F(IsolatedFileUtilTest, CopyOutDirectoryTest) {
   scoped_ptr<FileSystemOperationContext> context(
       new FileSystemOperationContext(file_system_context()));
   FileSystemURL root_url = GetFileSystemURL(FilePath());
   scoped_ptr<FileSystemFileUtil::AbstractFileEnumerator> file_enum(
@@ skipping 20 matching lines @@
                   false /* exclusive */,
                   true /* recursive */));
 
     context.reset(new FileSystemOperationContext(file_system_context()));
     ASSERT_EQ(base::PLATFORM_FILE_OK,
               FileUtilHelper::Copy(
                   context.get(),
                   file_util(), other_file_util(),
                   src_url, dest_url));
 
-    // The other way (copy-in) should not work for two reasons:
-    // write is prohibited in the isolated filesystem, and copying directory
-    // to non-empty directory shouldn't work.
-    context.reset(new FileSystemOperationContext(file_system_context()));
-    base::PlatformFileError result = FileUtilHelper::Copy(
-        context.get(), other_file_util(), file_util(), dest_url, src_url);
-    ASSERT_TRUE(result == base::PLATFORM_FILE_ERROR_FAILED ||
-                result == base::PLATFORM_FILE_ERROR_NOT_EMPTY);
-
     VerifyDirectoriesHaveSameContent(file_util(), other_file_util(),
                                      src_url, dest_url);
   }
 }
 
 TEST_F(IsolatedFileUtilTest, TouchTest) {
   for (size_t i = 0; i < test::kRegularTestCaseSize; ++i) {
     const test::TestCaseRecord& test_case = test::kRegularTestCases[i];
     if (test_case.is_directory)
       continue;
@@ skipping 42 matching lines @@
     EXPECT_EQ(base::PLATFORM_FILE_OK,
               file_util()->Truncate(GetOperationContext().get(), url, 999));
     ASSERT_EQ(base::PLATFORM_FILE_OK,
               file_util()->GetFileInfo(GetOperationContext().get(), url,
                                        &info, &platform_path));
     EXPECT_EQ(999, info.size);
   }
 }
 
 }  // namespace fileapi