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

base/file_path_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 "base/basictypes.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
+#include "base/string_util.h"
 #include "base/utf_string_conversions.h"
+#include "base/scoped_temp_dir.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/platform_test.h"
 
 // This macro helps avoid wrapped lines in the test structs.
 #define FPL(x) FILE_PATH_LITERAL(x)
 
 struct UnaryTestData {
   const FilePath::CharType* input;
   const FilePath::CharType* expected;
 };
@@ skipping 1061 matching lines @@
     { FPL("/\\foo\\/bar"), FPL("\\\\foo\\\\bar") },
   };
   for (size_t i = 0; i < arraysize(cases); ++i) {
     FilePath input(cases[i].input);
     FilePath observed = input.NormalizeWindowsPathSeparators();
     EXPECT_EQ(FilePath::StringType(cases[i].expected), observed.value()) <<
               "i: " << i << ", input: " << input.value();
   }
 }
 #endif
+
+#if defined(OS_WIN)
+TEST_F(FilePathTest, TestGetLongPathHack) {
+  ScopedTempDir dir;
+  ASSERT_TRUE(dir.CreateUniqueTempDir());
+
+  // Create long path.
+  const char kLongPathChar160[] =
+      "012345678901234567890123456789012345678901234567890123456789";
+      "012345678901234567890123456789012345678901234567890123456789"
+      "0123456789012345678901234567890123456789";
+  FilePath long_path1(dir.path().AppendASCII(kLongPathChar160));
+
+  ASSERT_TRUE(file_util::CreateDirectory(long_path1));
+  FilePath long_path2 = long_path1.AppendASCII(kLongPathChar160);
+  ASSERT_TRUE(file_util::CreateDirectory(long_path2));
+
+  // Get its 8.3 name.
+  FilePath::StringType long_path2_value(long_path2.value());
+  FilePath::StringType short_path;
+  DWORD length = ::GetShortPathName(
+      long_path2_value.c_str(), WriteInto(&short_path, MAX_PATH), MAX_PATH);

[kinuko, 2010/12/22 23:20:49]

I think we can just use long_path2.value().c_str() here.

[Kavita Kanetkar, 2010/12/23 03:14:28]

Done.

+  short_path.resize(length);
+
+  // Test converstion from 8.3 to long path.
+  FilePath path8_3(short_path);

[kinuko, 2010/12/22 23:20:49]

Do we need path8_3?  Using short_path looks fine to me.

[Kavita Kanetkar, 2010/12/23 03:14:28]

Sorry. That was before I changed the GetLongPathHack() in FilePath to return a
FilePath than manipulate the path_ internally.
Done.
On 2010/12/22 23:20:49, kinuko wrote:

+  EXPECT_LT(path8_3.value().length(), long_path2.value().length());
+  FilePath converted_long_path(path8_3.GetLongPathHack());
+  EXPECT_TRUE(StartsWith(
+      converted_long_path.value(), FilePath::kExtendedPathPrefix, false));
+
+  // Should be a no-op if path starts with extended path prefix.

[kinuko, 2010/12/22 23:20:49]

nit: if path starts ... -> if path already starts

[Kavita Kanetkar, 2010/12/23 03:14:28]

Done.

+  EXPECT_EQ(converted_long_path.value(),
+      converted_long_path.GetLongPathHack().value());
+
+  // Should be a no-op if path starts with UNC prefix.

[kinuko, 2010/12/22 23:20:49]

ditto.

[Kavita Kanetkar, 2010/12/23 03:14:28]

Done.

+  FilePath starts_with_unc_prefix(FilePath::kUNCExtendedPathPrefix);
+  EXPECT_EQ(starts_with_unc_prefix.value(),
+            starts_with_unc_prefix.GetLongPathHack().value());
+
+  // Test for share drive prefix.
+  FilePath filer(FPL("\\\\filer\\home\\me"));
+  FilePath::StringType expected_prefix(FilePath::kUNCExtendedPathPrefix);
+  expected_prefix.append(FPL("filer\\home\\me"));

[kinuko, 2010/12/22 23:20:49]

nit: s/filer/unc_path/ and s/expected_prefix/prefixed_unc_path/?

[Kavita Kanetkar, 2010/12/23 03:14:28]

Done.

+  EXPECT_EQ(expected_prefix, filer.GetLongPathHack().value());
+
+  // TODO(kkanetkar): When ScopedTempDir is fixed to handle longer
+  // paths, remove this.
+  EXPECT_TRUE(file_util::Delete(long_path1, true));
+}
+#endif