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

Unified Diff: webkit/fileapi/file_system_operation_unittest.cc

Issue 5259003: On windows filepaths need \\?\ prefix to allow extended file path names. Fix ... (Closed) Base URL: svn://chrome-svn/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 67172)
+++ webkit/fileapi/file_system_operation_unittest.cc (working copy)
@@ -463,6 +463,26 @@
EXPECT_EQ(request_id_, mock_dispatcher_->request_id());
}
+TEST_F(FileSystemOperationTest, TestCreateVeryLongName) {
+ ScopedTempDir dir;
+ ASSERT_TRUE(dir.CreateUniqueTempDir());
+
+ FilePath dir_path(FILE_PATH_LITERAL("\\\\?\\") + dir.path().value());
ericu 2010/11/25 04:18:35 I don't see anything forcing this test only to run
+ 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;

Powered by Google App Engine
This is Rietveld 408576698