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

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,34 @@
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