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

Unified Diff: webkit/fileapi/file_system_operation_unittest.cc

Issue 5357009: Revert 67385 - On windows filepaths need \\?\ prefix to allow extended file p... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years, 1 month 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: webkit/fileapi/file_system_operation_unittest.cc
===================================================================
--- webkit/fileapi/file_system_operation_unittest.cc (revision 67702)
+++ webkit/fileapi/file_system_operation_unittest.cc (working copy)
@@ -463,34 +463,6 @@
EXPECT_EQ(request_id_, mock_dispatcher_->request_id());
}
-TEST_F(FileSystemOperationTest, TestCreateVeryLongName) {
- ScopedTempDir dir;
- ASSERT_TRUE(dir.CreateUniqueTempDir());
-
-#if defined(OS_WIN)
- FilePath dir_path(FILE_PATH_LITERAL("\\\\?\\") + dir.path().value());
-#else
- FilePath dir_path = dir.path();
-#endif
-
- // TODO(kkanetkar): Once each platform's limitations have been enforced
- // consider that in this test. Currently this test is for
- // windows primarily.
- FilePath dir1 = dir_path.AppendASCII(
- "012345678901234567890123456789012345678901234567890123456789"
- "012345678901234567890123456789012345678901234567890123456789"
- "0123456789012345678901234567890123456789");
- FilePath file = dir1.AppendASCII(
- "012345678901234567890123456789012345678901234567890123456789"
- "012345678901234567890123456789012345678901234567890123456789"
- "0123456789012345678901234567890123456789");
- operation()->CreateDirectory(dir1, false, true);
- MessageLoop::current()->RunAllPending();
- operation()->CreateFile(file, true);
- MessageLoop::current()->RunAllPending();
- EXPECT_TRUE(file_util::PathExists(file));
-}
-
TEST_F(FileSystemOperationTest, TestCreateFileSuccessFileExists) {
// Already existing file and exclusive false.
ScopedTempDir dir;
« no previous file with comments | « chrome/browser/renderer_host/browser_render_process_host.cc ('k') | webkit/fileapi/file_system_path_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698