Implement media path filter

webkit/fileapi/media/native_media_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 <set>
+#include <string>
+
+#include "base/bind.h"
+#include "base/message_loop.h"
+#include "base/scoped_temp_dir.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "webkit/fileapi/file_system_context.h"
+#include "webkit/fileapi/file_system_operation_interface.h"
+#include "webkit/fileapi/isolated_context.h"
+#include "webkit/fileapi/media/native_media_file_util.h"
+#include "webkit/fileapi/mock_file_system_options.h"
+#include "webkit/fileapi/native_file_util.h"
+#include "webkit/quota/mock_special_storage_policy.h"
+
+#define FPL(x) FILE_PATH_LITERAL(x)
+
+using namespace fileapi;
+
+namespace {
+
+typedef FileSystemOperationInterface::FileEntryList FileEntryList;
+
+struct FilteringTestCase {
+  const FilePath::CharType* path;
+  bool is_directory;
+  bool visible;
+};
+
+void ExpectEqHelper(base::PlatformFileError expected,
+                    base::PlatformFileError actual) {
+  EXPECT_EQ(expected, actual);
+}
+
+void DidReadDirectory(std::set<FilePath::StringType>* content,
+                      bool* completed,
+                      base::PlatformFileError error,
+                      const FileEntryList& file_list,
+                      bool has_more) {
+  EXPECT_TRUE(!*completed);
+  *completed = !has_more;
+  for (FileEntryList::const_iterator itr = file_list.begin();
+       itr != file_list.end(); ++itr)
+    EXPECT_TRUE(content->insert(itr->name).second);
+}
+
+}  // namespace
+
+class NativeMediaFileUtilTest : public testing::Test {
+ public:
+  NativeMediaFileUtilTest()
+      : file_util_(NULL) {
+  }
+
+  void SetUp() {
+    ASSERT_TRUE(data_dir_.CreateUniqueTempDir());
+
+    scoped_refptr<quota::SpecialStoragePolicy> storage_policy =
+        new quota::MockSpecialStoragePolicy();
+
+    file_system_context_ =
+        new FileSystemContext(
+            base::MessageLoopProxy::current(),
+            base::MessageLoopProxy::current(),
+            storage_policy,
+            NULL,
+            data_dir_.path(),
+            CreateAllowFileAccessOptions());
+
+    file_util_ = file_system_context_->GetFileUtil(kFileSystemTypeNativeMedia);
+
+    filesystem_id_ = isolated_context()->RegisterFileSystemForPath(
+        kFileSystemTypeNativeMedia, root_path(), NULL);
+
+    isolated_context()->AddReference(filesystem_id_);
+
+    virtual_root_path_ =
+        isolated_context()->CreateVirtualRootPath(filesystem_id_);
+  }
+
+  void TearDown() {
+    isolated_context()->RemoveReference(filesystem_id_);
+    file_system_context_ = NULL;
+  }
+
+ protected:
+  FileSystemContext* file_system_context() {
+    return file_system_context_.get();
+  }
+
+  IsolatedContext* isolated_context() {
+    return IsolatedContext::GetInstance();
+  }
+
+  FilePath root_path() {
+    return data_dir_.path().Append(FPL("Media Directory"));
+  }
+
+  FileSystemFileUtil* file_util() {
+    return file_util_;
+  }
+
+  GURL origin() {
+    return GURL("http://example.com");
+  }
+
+  FileSystemType type() {
+    return kFileSystemTypeNativeMedia;
+  }
+
+  FileSystemOperationInterface* NewOperation(const FileSystemURL& url) {
+    return file_system_context_->CreateFileSystemOperation(url);
+  }
+
+ private:
+  MessageLoop message_loop_;
+
+  ScopedTempDir data_dir_;
+  scoped_refptr<FileSystemContext> file_system_context_;
+
+  FileSystemFileUtil* file_util_;
+  std::string filesystem_id_;
+  FilePath virtual_root_path_;
+
+  DISALLOW_COPY_AND_ASSIGN(NativeMediaFileUtilTest);
+};
+
+TEST_F(NativeMediaFileUtilTest, FilteringTest) {

[kinuko, 2012/07/31 01:26:35]

For easier diagnosis could we split this test into two or three (e.g. one for
FooExists and the other for ReadDirectory)?  The setup part could be in a
separate method or in SetUp().

[tzik, 2012/07/31 22:31:23]

Done.

+  const FilteringTestCase kTestCases[] = {
+    { FPL("hoge"), true, true },
+    // Directory should always be visible.
+    { FPL("hoge/fuga"), true, true },
+    { FPL("hoge/piyo.jpg"), true, true },
+    { FPL("hoge/moga.txt"), true, true },
+
+    // File should be visible if it's a supported media file.
+    { FPL("hoge/foo"), false, false },  // File without extension.
+    { FPL("hoge/bar.jpg"), false, true },  // Supported media file.
+    { FPL("hoge/baz.txt"), false, false },  // Non-media file.
+    { FPL("hoge/foobar.cod"), false, false },  // Unsupported media file.
+  };
+
+  for (size_t i = 0; i < arraysize(kTestCases); ++i) {
+    FilePath path = root_path().Append(kTestCases[i].path);
+    if (kTestCases[i].is_directory) {
+      ASSERT_TRUE(file_util::CreateDirectory(path));
+    } else {
+      bool created = false;
+      ASSERT_EQ(base::PLATFORM_FILE_OK,
+                NativeFileUtil::EnsureFileExists(path, &created));
+      ASSERT_TRUE(created);
+    }
+  }
+
+  for (size_t i = 0; i < arraysize(kTestCases); ++i) {
+    FilePath path = root_path().Append(kTestCases[i].path);
+    FileSystemURL url(origin(), type(), path);
+    FileSystemOperationInterface* operation = NewOperation(url);
+
+    base::PlatformFileError expectation =
+        kTestCases[i].visible ?
+        base::PLATFORM_FILE_OK :
+        base::PLATFORM_FILE_ERROR_NOT_FOUND;
+
+    if (kTestCases[i].is_directory)
+      operation->DirectoryExists(url, base::Bind(&ExpectEqHelper, expectation));
+    else
+      operation->FileExists(url, base::Bind(&ExpectEqHelper, expectation));
+    MessageLoop::current()->RunAllPending();
+  }
+
+  std::set<FilePath::StringType> content;
+  FileSystemURL url(origin(), type(),
+                    root_path().Append(kTestCases[0].path));
+  bool completed = false;
+  NewOperation(url)->ReadDirectory(
+      url, base::Bind(&DidReadDirectory, &content, &completed));
+  MessageLoop::current()->RunAllPending();
+  EXPECT_TRUE(completed);
+  EXPECT_EQ(4u, content.size());
+
+  for (size_t i = 1; i < arraysize(kTestCases); ++i) {
+    FilePath::StringType name = FilePath(kTestCases[i].path).BaseName().value();
+    std::set<FilePath::StringType>::const_iterator found = content.find(name);
+    EXPECT_EQ(kTestCases[i].visible, found != content.end());
+  }
+}