Grant file access permissions for cached file paths to file browsers/handlers.

File browser extensions are given access permissions for whole pinned and tmp cache dirs.

File handlers are given permissions during runtime for files for wich their task is being executed.
To simplify things they are given permissions for all possible file's cache paths.

Since we check permissions for cache paths only when performing entry.file(), read-only permissions
should be sufficient.

BUG=chromium-os:27926
TEST=manual. browser_tests:*FileSystemExtension*

Review URL: https://chromiumcodereview.appspot.com/9808023

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@128417 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 0 insertions, 237 deletions

diff --git a/chrome/browser/extensions/extension_local_filesystem_apitest.cc b/chrome/browser/extensions/extension_local_filesystem_apitest.cc
deleted file mode 100644
index 1e1dcdd..0000000
--- a/chrome/browser/extensions/extension_local_filesystem_apitest.cc
+++ /dev/null
@@ -1,237 +0,0 @@
-// 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 "base/file_util.h"
-#include "base/platform_file.h"
-#include "base/stringprintf.h"
-#include "chrome/browser/extensions/extension_apitest.h"
-#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/ui/browser.h"
-#include "testing/gmock/include/gmock/gmock.h"
-#include "webkit/fileapi/file_system_context.h"
-#include "webkit/fileapi/file_system_mount_point_provider.h"
-#if defined(OS_CHROMEOS)
-#include "chrome/browser/chromeos/gdata/gdata_util.h"
-#include "webkit/chromeos/fileapi/remote_file_system_proxy.h"
-#endif
-
-using ::testing::_;
-using content::BrowserContext;
-
-#if defined(OS_CHROMEOS)
-
-// These should match the counterparts in remote.js.
-const char kTestFileName[] = "hello.txt";
-const char kTestFileContents[] = "hello, world";
-
-namespace {
-
-// The ID of the file browser extension.
-const char kFileBrowserExtensionId[] = "ddammdhioacbehjngdmkjcjbnfginlla";
-
-// Returns the expected URL for the given path.
-GURL GetExpectedURL(const std::string& path) {
-  return GURL(
-      base::StringPrintf(
-          "filesystem:chrome-extension://%s/external/%s",
-          kFileBrowserExtensionId,
-          path.c_str()));
-}
-
-// Action used to set mock expectations for CreateDirectory().
-ACTION_P(MockCreateDirectory, status) {
-  arg3.Run(status);
-}
-
-// Action used to set mock expectations for GetFileInfo().
-ACTION_P3(MockGetFileInfo, status, file_info, path) {
-  arg1.Run(status, file_info, path);
-}
-
-// Action used to set mock expectations for CreateSnapshotFile().
-ACTION_P4(MockCreateSnapshotFile, status, file_info, path, file_ref) {
-  arg1.Run(status, file_info, path, file_ref);
-}
-
-// The mock is used to add a remote mount point, and write tests for it.
-class MockRemoteFileSystemProxy :
-      public fileapi::RemoteFileSystemProxyInterface {
- public:
-  MockRemoteFileSystemProxy() {}
-  virtual ~MockRemoteFileSystemProxy() {}
-
-  MOCK_METHOD2(
-      GetFileInfo,
-      void(const GURL& path,
-           const fileapi::FileSystemOperationInterface::GetMetadataCallback&
-           callback));
-  MOCK_METHOD3(
-      Copy,
-      void(const GURL& src_path,
-           const GURL& dest_path,
-           const fileapi::FileSystemOperationInterface::StatusCallback&
-           callback));
-  MOCK_METHOD3(
-      Move,
-      void(const GURL& src_path,
-           const GURL& dest_path,
-           const fileapi::FileSystemOperationInterface::StatusCallback&
-           callback));
-  MOCK_METHOD2(
-      ReadDirectory,
-      void(const GURL& path,
-           const fileapi::FileSystemOperationInterface::ReadDirectoryCallback&
-           callback));
-  MOCK_METHOD3(
-      Remove,
-      void(const GURL& path,
-           bool recursive,
-           const fileapi::FileSystemOperationInterface::StatusCallback&
-           callback));
-  MOCK_METHOD4(
-      CreateDirectory,
-      void(const GURL& file_url,
-           bool exclusive,
-           bool recursive,
-           const fileapi::FileSystemOperationInterface::StatusCallback&
-           callback));
-  MOCK_METHOD2(
-      CreateSnapshotFile,
-      void(const GURL& path,
-           const fileapi::FileSystemOperationInterface::SnapshotFileCallback&
-           callback));
-};
-
-const char kExpectedWriteError[] =
-    "Got unexpected error: File handler error: SECURITY_ERR";
-}
-
-class FileSystemExtensionApiTest : public ExtensionApiTest {
- public:
-  FileSystemExtensionApiTest() : test_mount_point_("/tmp") {
-  }
-
-  virtual ~FileSystemExtensionApiTest() {}
-
-  // Adds a local mount point at at mount point /tmp.
-  void AddTmpMountPoint() {
-    fileapi::ExternalFileSystemMountPointProvider* provider =
-        BrowserContext::GetFileSystemContext(browser()->profile())->
-            external_provider();
-    provider->AddLocalMountPoint(test_mount_point_);
-  }
-
- private:
-  FilePath test_mount_point_;
-};
-
-class RemoteFileSystemExtensionApiTest : public ExtensionApiTest {
- public:
-  RemoteFileSystemExtensionApiTest()
-      : test_mount_point_("/tmp"),
-        mock_remote_file_system_proxy_(NULL) {
-  }
-
-  virtual ~RemoteFileSystemExtensionApiTest() {}
-
-  virtual void SetUp() OVERRIDE {
-    file_util::CreateTemporaryFile(&test_file_path_);
-    file_util::WriteFile(test_file_path_,
-                         kTestFileContents,
-                         sizeof(kTestFileContents) - 1);
-    file_util::GetFileInfo(test_file_path_, &test_file_info_);
-
-    // ExtensionApiTest::SetUp() should be called at the end. For some
-    // reason, ExtensionApiTest::SetUp() starts running tests, so any
-    // setup has to be done before calling this.
-    ExtensionApiTest::SetUp();
-  }
-
-  virtual void TearDown() OVERRIDE {
-    file_util::Delete(test_file_path_, false);
-
-    ExtensionApiTest::TearDown();
-  }
-
-  // Adds a remote mount point at at mount point /tmp.
-  void AddTmpMountPoint() {
-    fileapi::ExternalFileSystemMountPointProvider* provider =
-        BrowserContext::GetFileSystemContext(browser()->profile())->
-            external_provider();
-    mock_remote_file_system_proxy_ = new MockRemoteFileSystemProxy;
-    // Take the ownership of mock_remote_file_system_proxy_.
-    provider->AddRemoteMountPoint(test_mount_point_,
-                                  mock_remote_file_system_proxy_);
-  }
-
- protected:
-  base::PlatformFileInfo test_file_info_;
-  FilePath test_file_path_;
-  FilePath test_mount_point_;
-  MockRemoteFileSystemProxy* mock_remote_file_system_proxy_;
-};
-
-IN_PROC_BROWSER_TEST_F(FileSystemExtensionApiTest, LocalFileSystem) {
-  AddTmpMountPoint();
-  ASSERT_TRUE(RunComponentExtensionTest("local_filesystem")) << message_;
-}
-
-IN_PROC_BROWSER_TEST_F(FileSystemExtensionApiTest, FileBrowserTest) {
-  AddTmpMountPoint();
-  ASSERT_TRUE(RunExtensionTest("filesystem_handler")) << message_;
-  ASSERT_TRUE(RunComponentExtensionSubtest("filebrowser_component",
-                                           "read.html")) << message_;
-}
-
-IN_PROC_BROWSER_TEST_F(FileSystemExtensionApiTest, FileBrowserTestWrite) {
-  AddTmpMountPoint();
-  ASSERT_TRUE(RunExtensionTest("filesystem_handler_write")) << message_;
-  ASSERT_TRUE(RunComponentExtensionSubtest("filebrowser_component",
-                                            "write.html")) << message_;
-}
-
-IN_PROC_BROWSER_TEST_F(FileSystemExtensionApiTest,
-                       FileBrowserTestWriteComponent) {
-  AddTmpMountPoint();
-  ASSERT_TRUE(RunComponentExtensionTest("filesystem_handler_write"))
-      << message_;
-  ASSERT_TRUE(RunComponentExtensionSubtest("filebrowser_component",
-                                           "write.html")) << message_;
-}
-
-IN_PROC_BROWSER_TEST_F(RemoteFileSystemExtensionApiTest, RemoteMountPoint) {
-  AddTmpMountPoint();
-
-  // The test directory is created first.
-  const GURL test_dir_url = GetExpectedURL("tmp/test_dir");
-  EXPECT_CALL(*mock_remote_file_system_proxy_,
-              CreateDirectory(test_dir_url, false, false, _))
-      .WillOnce(MockCreateDirectory(base::PLATFORM_FILE_OK));
-
-  // Then GetFileInfo() is called over "tmp/test_dir/hello.txt".
-  const std::string expected_path =
-      std::string("tmp/test_dir/") + kTestFileName;
-  GURL expected_url = GetExpectedURL(expected_path);
-  EXPECT_CALL(*mock_remote_file_system_proxy_,
-              GetFileInfo(expected_url, _))
-      .WillOnce(MockGetFileInfo(
-          base::PLATFORM_FILE_OK,
-          test_file_info_,
-          FilePath::FromUTF8Unsafe(expected_path)));
-
-  // Then CreateSnapshotFile() is called over "tmp/test_dir/hello.txt".
-  EXPECT_CALL(*mock_remote_file_system_proxy_,
-              CreateSnapshotFile(expected_url, _))
-      .WillOnce(MockCreateSnapshotFile(
-          base::PLATFORM_FILE_OK,
-          test_file_info_,
-          // Returns the path to the temporary file on the local drive.
-          test_file_path_,
-          scoped_refptr<webkit_blob::ShareableFileReference>(NULL)));
-
-  ASSERT_TRUE(RunComponentExtensionSubtest("filebrowser_component",
-                                           "remote.html")) << message_;
-}
-
-#endif