Deprecate MountPointProvider::IsAccessAllowed in favor of GetPermissionPolicy

webkit/fileapi/file_system_mount_point_provider_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/basictypes.h"
 #include "base/bind.h"
 #include "base/file_util.h"
@@ skipping 83 matching lines @@
   { FILE_PATH_LITERAL("//tmp/foo.txt"), false, },
   { FILE_PATH_LITERAL("//etc/hosts"), false, },
   { FILE_PATH_LITERAL("foo.txt"), true, },
   { FILE_PATH_LITERAL("a/b/c"), true, },
   // Any paths that includes parent references are considered invalid.
   { FILE_PATH_LITERAL(".."), false, },
   { FILE_PATH_LITERAL("tmp/.."), false, },
   { FILE_PATH_LITERAL("a/b/../c/.."), false, },
 };
 
-const struct IsRestrictedNameTest {
-  base::FilePath::StringType name;
-  bool expected_dangerous;
-} kIsRestrictedNameTestCases[] = {
-  // Names that contain strings that used to be restricted, but are now allowed.
-  { FILE_PATH_LITERAL("con"), false, },
-  { FILE_PATH_LITERAL("Con.txt"), false, },
-  { FILE_PATH_LITERAL("Prn.png"), false, },
-  { FILE_PATH_LITERAL("AUX"), false, },
-  { FILE_PATH_LITERAL("nUl."), false, },
-  { FILE_PATH_LITERAL("coM1"), false, },
-  { FILE_PATH_LITERAL("COM3.com"), false, },
-  { FILE_PATH_LITERAL("cOM7"), false, },
-  { FILE_PATH_LITERAL("com9"), false, },
-  { FILE_PATH_LITERAL("lpT1"), false, },
-  { FILE_PATH_LITERAL("LPT4.com"), false, },
-  { FILE_PATH_LITERAL("lPT8"), false, },
-  { FILE_PATH_LITERAL("lPT9"), false, },
-  { FILE_PATH_LITERAL("com1."), false, },
-
-  // Similar cases that have always been allowed.
-  { FILE_PATH_LITERAL("con3"), false, },
-  { FILE_PATH_LITERAL("PrnImage.png"), false, },
-  { FILE_PATH_LITERAL("AUXX"), false, },
-  { FILE_PATH_LITERAL("NULL"), false, },
-  { FILE_PATH_LITERAL("coM0"), false, },
-  { FILE_PATH_LITERAL("COM.com"), false, },
-  { FILE_PATH_LITERAL("lpT0"), false, },
-  { FILE_PATH_LITERAL("LPT.com"), false, },
-
-  // Ends with period or whitespace--used to be banned, now OK.
-  { FILE_PATH_LITERAL("b "), false, },
-  { FILE_PATH_LITERAL("b\t"), false, },
-  { FILE_PATH_LITERAL("b\n"), false, },
-  { FILE_PATH_LITERAL("b\r\n"), false, },
-  { FILE_PATH_LITERAL("b."), false, },
-  { FILE_PATH_LITERAL("b.."), false, },
-
-  // Similar cases that have always been allowed.
-  { FILE_PATH_LITERAL("b c"), false, },
-  { FILE_PATH_LITERAL("b\tc"), false, },
-  { FILE_PATH_LITERAL("b\nc"), false, },
-  { FILE_PATH_LITERAL("b\r\nc"), false, },
-  { FILE_PATH_LITERAL("b c d e f"), false, },
-  { FILE_PATH_LITERAL("b.c"), false, },
-  { FILE_PATH_LITERAL("b..c"), false, },
-
-  // Name that has restricted chars in it.
-  { FILE_PATH_LITERAL("\\"), true, },
-  { FILE_PATH_LITERAL("/"), true, },
-  { FILE_PATH_LITERAL("a\\b"), true, },
-  { FILE_PATH_LITERAL("a/b"), true, },
-  { FILE_PATH_LITERAL("ab\\"), true, },
-  { FILE_PATH_LITERAL("ab/"), true, },
-  { FILE_PATH_LITERAL("\\ab"), true, },
-  { FILE_PATH_LITERAL("/ab"), true, },
-  { FILE_PATH_LITERAL("ab/.txt"), true, },
-  { FILE_PATH_LITERAL("ab\\.txt"), true, },
-
-  // Names that contain chars that were formerly restricted, now OK.
-  { FILE_PATH_LITERAL("a<b"), false, },
-  { FILE_PATH_LITERAL("a>b"), false, },
-  { FILE_PATH_LITERAL("a:b"), false, },
-  { FILE_PATH_LITERAL("a?b"), false, },
-  { FILE_PATH_LITERAL("a|b"), false, },
-  { FILE_PATH_LITERAL("ab<.txt"), false, },
-  { FILE_PATH_LITERAL("ab>.txt"), false, },
-  { FILE_PATH_LITERAL("ab:.txt"), false, },
-  { FILE_PATH_LITERAL("ab?.txt"), false, },
-  { FILE_PATH_LITERAL("ab|.txt"), false, },
-  { FILE_PATH_LITERAL("<ab"), false, },
-  { FILE_PATH_LITERAL(">ab"), false, },
-  { FILE_PATH_LITERAL(":ab"), false, },
-  { FILE_PATH_LITERAL("?ab"), false, },
-  { FILE_PATH_LITERAL("|ab"), false, },
-
-  // Names that are restricted still.
-  { FILE_PATH_LITERAL(".."), true, },
-  { FILE_PATH_LITERAL("."), true, },
-
-  // Similar but safe cases.
-  { FILE_PATH_LITERAL(" ."), false, },
-  { FILE_PATH_LITERAL(". "), false, },
-  { FILE_PATH_LITERAL(" . "), false, },
-  { FILE_PATH_LITERAL(" .."), false, },
-  { FILE_PATH_LITERAL(".. "), false, },
-  { FILE_PATH_LITERAL(" .. "), false, },
-  { FILE_PATH_LITERAL("b."), false, },
-  { FILE_PATH_LITERAL(".b"), false, },
-};
-
 // For External filesystem.
 const base::FilePath::CharType kMountPoint[] = FILE_PATH_LITERAL("/tmp/testing");
 const base::FilePath::CharType kRootPath[] = FILE_PATH_LITERAL("/tmp");
 const base::FilePath::CharType kVirtualPath[] = FILE_PATH_LITERAL("testing");
 
 }  // namespace
 
 class FileSystemMountPointProviderTest : public testing::Test {
  public:
   FileSystemMountPointProviderTest()
@@ skipping 182 matching lines @@
     EXPECT_TRUE(GetRootPath(GURL(kRootPathFileURITestCases[i].origin_url),
                             kRootPathFileURITestCases[i].type,
                             true /* create */, &root_path));
     base::FilePath expected = file_system_path().AppendASCII(
         kRootPathFileURITestCases[i].expected_path);
     EXPECT_EQ(expected.value(), root_path.value());
     EXPECT_TRUE(file_util::DirectoryExists(root_path));
   }
 }
 
-TEST_F(FileSystemMountPointProviderTest, IsRestrictedName) {
-  SetupNewContext(CreateDisallowFileAccessOptions());
-  for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kIsRestrictedNameTestCases); ++i) {
-    SCOPED_TRACE(testing::Message() << "IsRestrictedName #" << i << " "
-                 << kIsRestrictedNameTestCases[i].name);
-    base::FilePath name(kIsRestrictedNameTestCases[i].name);
-    EXPECT_EQ(kIsRestrictedNameTestCases[i].expected_dangerous,
-              provider(kFileSystemTypeTemporary)->IsRestrictedFileName(name));
-  }
-}
-
 }  // namespace fileapi