Moving away from shell api to support long path names on windows for filesystem.

webkit/fileapi/file_system_operation_unittest.cc

 // Copyright (c) 2010 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 "webkit/fileapi/file_system_operation.h"
 
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/scoped_ptr.h"
 #include "base/scoped_temp_dir.h"
@@ skipping 410 matching lines @@
   ScopedTempDir dir;
   ASSERT_TRUE(dir.CreateUniqueTempDir());
   FilePath file;
 
   file_util::CreateTemporaryFileInDir(dir.path(), &file);
   operation()->CreateFile(file, true);
   MessageLoop::current()->RunAllPending();
   EXPECT_EQ(base::PLATFORM_FILE_ERROR_EXISTS, status());
 }
 
+TEST_F(FileSystemOperationTest, TestCreateVeryLongName) {
+  ScopedTempDir dir;
+  ASSERT_TRUE(dir.CreateUniqueTempDir());
+
+#if defined(OS_WIN)
+  FilePath dir_path(FilePath::kExtendedPathPrefix + 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.
+  const char kLongPathChar160[] =
+      "012345678901234567890123456789012345678901234567890123456789"
+      "012345678901234567890123456789012345678901234567890123456789"
+      "0123456789012345678901234567890123456789";
+
+  FilePath dir1 = dir_path.AppendASCII(kLongPathChar160);
+
+  FilePath dir2 = dir1.AppendASCII(kLongPathChar160);
+
+  FilePath file = dir2.AppendASCII(kLongPathChar160);
+
+  operation()->CreateDirectory(dir1, false, true);
+  MessageLoop::current()->RunAllPending();
+
+  operation()->CreateDirectory(dir2, false, true);
+  MessageLoop::current()->RunAllPending();
+
+  operation()->CreateFile(file, true);
+  MessageLoop::current()->RunAllPending();
+  EXPECT_TRUE(file_util::PathExists(file));
+  // ScopedTempDir does not delete dir with long name since it does

[Paweł Hajdan Jr., 2010/12/16 08:58:40]

Oh, should we rather fix ScopedTempDir?

[Erik does not do reviews, 2010/12/16 18:34:54]

Agree.  We use ScopedTempDir in a number of places, so we should make sure that
actually works as well.

+  // not prefix \\?\. So Delete the long file here.
+  EXPECT_TRUE(file_util::Delete(dir1, true));
+}
+
 TEST_F(FileSystemOperationTest, TestCreateFileSuccessFileExists) {
   // Already existing file and exclusive false.
   ScopedTempDir dir;
   ASSERT_TRUE(dir.CreateUniqueTempDir());
   FilePath file;
   file_util::CreateTemporaryFileInDir(dir.path(), &file);
 
   operation()->CreateFile(file, false);
   MessageLoop::current()->RunAllPending();
   EXPECT_EQ(kFileOperationSucceeded, status());
@@ skipping 280 matching lines @@
 
   // Check that its length is now 3 and that it contains only bits of test data.
   EXPECT_TRUE(file_util::GetFileInfo(file, &info));
   EXPECT_EQ(length, info.size);
   EXPECT_EQ(length, file_util::ReadFile(file, data, length));
   for (int i = 0; i < length; ++i)
     EXPECT_EQ(test_data[i], data[i]);
 }
 
 }  // namespace fileapi