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

Unified Diff: chrome/browser/chromeos/fileapi/file_access_permissions_unittest.cc

Issue 17644006: Move files under webkit/browser/fileapi/... to chrome/browser/chromeos/fileapi/... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address review comments. Created 7 years, 6 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: chrome/browser/chromeos/fileapi/file_access_permissions_unittest.cc
diff --git a/webkit/browser/chromeos/fileapi/file_access_permissions_unittest.cc b/chrome/browser/chromeos/fileapi/file_access_permissions_unittest.cc
similarity index 84%
rename from webkit/browser/chromeos/fileapi/file_access_permissions_unittest.cc
rename to chrome/browser/chromeos/fileapi/file_access_permissions_unittest.cc
index 3b6875d84bb7f580abe49a736a3af32814378a4e..9e5c04e212573a9b37c53cf8c2029178782f8c8e 100644
--- a/webkit/browser/chromeos/fileapi/file_access_permissions_unittest.cc
+++ b/chrome/browser/chromeos/fileapi/file_access_permissions_unittest.cc
@@ -2,29 +2,22 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "testing/gtest/include/gtest/gtest.h"
-#include "webkit/browser/chromeos/fileapi/file_access_permissions.h"
+#include "chrome/browser/chromeos/fileapi/file_access_permissions.h"
+#include "testing/gtest/include/gtest/gtest.h"
-class FileAccessPermissionsTest : public testing::Test {
-};
+namespace chromeos {
-TEST_F(FileAccessPermissionsTest, FileAccessChecks) {
-#if defined(OS_WIN)
- base::FilePath good_dir(FILE_PATH_LITERAL("c:\\root\\dir"));
- base::FilePath bad_dir(FILE_PATH_LITERAL("c:\\root"));
- base::FilePath good_file(FILE_PATH_LITERAL("c:\\root\\dir\\good_file.txt"));
- base::FilePath bad_file(FILE_PATH_LITERAL("c:\\root\\dir\\bad_file.txt"));
-#elif defined(OS_POSIX)
+TEST(FileAccessPermissionsTest, FileAccessChecks) {
base::FilePath good_dir(FILE_PATH_LITERAL("/root/dir"));
base::FilePath bad_dir(FILE_PATH_LITERAL("/root"));
base::FilePath good_file(FILE_PATH_LITERAL("/root/dir/good_file.txt"));
base::FilePath bad_file(FILE_PATH_LITERAL("/root/dir/bad_file.txt"));
-#endif
+
std::string extension1("ddammdhioacbehjngdmkjcjbnfginlla");
std::string extension2("jkhdjkhkhsdkfhsdkhrterwmtermeter");
- chromeos::FileAccessPermissions permissions;
+ FileAccessPermissions permissions;
// By default extension have no access to any local file.
EXPECT_FALSE(permissions.HasAccessPermission(extension1, good_dir));
EXPECT_FALSE(permissions.HasAccessPermission(extension1, good_file));
@@ -65,3 +58,5 @@ TEST_F(FileAccessPermissionsTest, FileAccessChecks) {
EXPECT_FALSE(permissions.HasAccessPermission(extension2, good_file));
EXPECT_FALSE(permissions.HasAccessPermission(extension2, bad_file));
}
+
+} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698