Get rid of kPathSeparator on windows.

base/file_util_unittest.cc

 // Copyright (c) 2006-2008 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 "build/build_config.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shellapi.h>
 #include <shlobj.h>
@@ skipping 129 matching lines @@
     std::wstring result = value.path;
     file_util::AppendToPath(&result, value.ending);
     EXPECT_EQ(value.result, result);
   }
 
 #ifdef NDEBUG
   file_util::AppendToPath(NULL, L"path");  // asserts in debug mode
 #endif
 }
 
-// TODO(port): enable this test for non-Windows.
-#if defined(OS_WIN)
 static const struct InsertBeforeExtensionCase {
   std::wstring path;
-  std::wstring suffix;
+  FilePath::StringType suffix;
   std::wstring result;
 } kInsertBeforeExtension[] = {
-  {L"", L"", L""},
-  {L"", L"txt", L"txt"},
-  {L".", L"txt", L"txt."},
-  {L".", L"", L"."},
-  {L"foo.dll", L"txt", L"footxt.dll"},
-  {L"foo.dll", L".txt", L"foo.txt.dll"},
-  {L"foo", L"txt", L"footxt"},
-  {L"foo", L".txt", L"foo.txt"},
-  {L"foo.baz.dll", L"txt", L"foo.baztxt.dll"},
-  {L"foo.baz.dll", L".txt", L"foo.baz.txt.dll"},
-  {L"foo.dll", L"", L"foo.dll"},
-  {L"foo.dll", L".", L"foo..dll"},
-  {L"foo", L"", L"foo"},
-  {L"foo", L".", L"foo."},
-  {L"foo.baz.dll", L"", L"foo.baz.dll"},
-  {L"foo.baz.dll", L".", L"foo.baz..dll"},
+  {L"", FILE_PATH_LITERAL(""), L""},
+  {L"", FILE_PATH_LITERAL("txt"), L"txt"},
+  {L".", FILE_PATH_LITERAL("txt"), L"txt."},
+  {L".", FILE_PATH_LITERAL(""), L"."},
+  {L"foo.dll", FILE_PATH_LITERAL("txt"), L"footxt.dll"},
+  {L"foo.dll", FILE_PATH_LITERAL(".txt"), L"foo.txt.dll"},
+  {L"foo", FILE_PATH_LITERAL("txt"), L"footxt"},
+  {L"foo", FILE_PATH_LITERAL(".txt"), L"foo.txt"},
+  {L"foo.baz.dll", FILE_PATH_LITERAL("txt"), L"foo.baztxt.dll"},
+  {L"foo.baz.dll", FILE_PATH_LITERAL(".txt"), L"foo.baz.txt.dll"},
+  {L"foo.dll", FILE_PATH_LITERAL(""), L"foo.dll"},
+  {L"foo.dll", FILE_PATH_LITERAL("."), L"foo..dll"},
+  {L"foo", FILE_PATH_LITERAL(""), L"foo"},
+  {L"foo", FILE_PATH_LITERAL("."), L"foo."},
+  {L"foo.baz.dll", FILE_PATH_LITERAL(""), L"foo.baz.dll"},
+  {L"foo.baz.dll", FILE_PATH_LITERAL("."), L"foo.baz..dll"},
 #if defined(OS_WIN)
   {L"\\", L"", L"\\"},
   {L"\\", L"txt", L"\\txt"},
   {L"\\.", L"txt", L"\\txt."},
   {L"\\.", L"", L"\\."},
   {L"C:\\bar\\foo.dll", L"txt", L"C:\\bar\\footxt.dll"},
   {L"C:\\bar.baz\\foodll", L"txt", L"C:\\bar.baz\\foodlltxt"},
   {L"C:\\bar.baz\\foo.dll", L"txt", L"C:\\bar.baz\\footxt.dll"},
   {L"C:\\bar.baz\\foo.dll.exe", L"txt", L"C:\\bar.baz\\foo.dlltxt.exe"},
   {L"C:\\bar.baz\\foo", L"", L"C:\\bar.baz\\foo"},
   {L"C:\\bar.baz\\foo.exe", L"", L"C:\\bar.baz\\foo.exe"},
   {L"C:\\bar.baz\\foo.dll.exe", L"", L"C:\\bar.baz\\foo.dll.exe"},
   {L"C:\\bar\\baz\\foo.exe", L" (1)", L"C:\\bar\\baz\\foo (1).exe"},
 #elif defined(OS_POSIX)
-  {L"/", L"", L"/"},
-  {L"/", L"txt", L"/txt"},
-  {L"/.", L"txt", L"/txt."},
-  {L"/.", L"", L"/."},
-  {L"/bar/foo.dll", L"txt", L"/bar/footxt.dll"},
-  {L"/bar.baz/foodll", L"txt", L"/bar.baz/foodlltxt"},
-  {L"/bar.baz/foo.dll", L"txt", L"/bar.baz/footxt.dll"},
-  {L"/bar.baz/foo.dll.exe", L"txt", L"/bar.baz/foo.dlltxt.exe"},
-  {L"/bar.baz/foo", L"", L"/bar.baz/foo"},
-  {L"/bar.baz/foo.exe", L"", L"/bar.baz/foo.exe"},
-  {L"/bar.baz/foo.dll.exe", L"", L"/bar.baz/foo.dll.exe"},
-  {L"/bar/baz/foo.exe", L" (1)", L"/bar/baz/foo (1).exe"},
+  {L"/", "", L"/"},
+  {L"/", "txt", L"/txt"},
+  {L"/.", "txt", L"/txt."},
+  {L"/.", "", L"/."},
+  {L"/bar/foo.dll", "txt", L"/bar/footxt.dll"},
+  {L"/bar.baz/foodll", "txt", L"/bar.baz/foodlltxt"},
+  {L"/bar.baz/foo.dll", "txt", L"/bar.baz/footxt.dll"},
+  {L"/bar.baz/foo.dll.exe", "txt", L"/bar.baz/foo.dlltxt.exe"},
+  {L"/bar.baz/foo", "", L"/bar.baz/foo"},
+  {L"/bar.baz/foo.exe", "", L"/bar.baz/foo.exe"},
+  {L"/bar.baz/foo.dll.exe", "", L"/bar.baz/foo.dll.exe"},
+  {L"/bar/baz/foo.exe", " (1)", L"/bar/baz/foo (1).exe"},
 #endif
 };
 
 TEST_F(FileUtilTest, InsertBeforeExtensionTest) {
   for (unsigned int i = 0; i < arraysize(kInsertBeforeExtension); ++i) {
-    std::wstring path(kInsertBeforeExtension[i].path);
+    FilePath path = FilePath::FromWStringHack(kInsertBeforeExtension[i].path);
     file_util::InsertBeforeExtension(&path, kInsertBeforeExtension[i].suffix);
-    EXPECT_EQ(path, kInsertBeforeExtension[i].result);
+    EXPECT_EQ(kInsertBeforeExtension[i].result, path.ToWStringHack());
   }
 }
-#endif  // defined(OS_WIN)
 
 static const struct filename_case {
   const wchar_t* path;
   const wchar_t* filename;
 } filename_cases[] = {
 #if defined(OS_WIN)
   {L"c:\\colon\\backslash", L"backslash"},
   {L"c:\\colon\\backslash\\", L""},
   {L"\\\\filename.exe", L"filename.exe"},
   {L"filename.exe", L"filename.exe"},
@@ skipping 561 matching lines @@
 };
 
 TEST_F(FileUtilTest, ReplaceIllegalCharactersTest) {
   for (unsigned int i = 0; i < arraysize(kIllegalCharacterCases); ++i) {
     std::wstring bad_name(kIllegalCharacterCases[i].bad_name);
     file_util::ReplaceIllegalCharacters(&bad_name, L'-');
     EXPECT_EQ(kIllegalCharacterCases[i].good_name, bad_name);
   }
 }
 
-// TODO(port): enable this test for non-windows.
-#if defined(OS_WIN)
 static const struct ReplaceExtensionCase {
   std::wstring file_name;
-  std::wstring extension;
+  FilePath::StringType extension;
   std::wstring result;
 } kReplaceExtension[] = {
-  {L"", L"", L""},
-  {L"", L"txt", L".txt"},
-  {L".", L"txt", L".txt"},
-  {L".", L"", L""},
-  {L"foo.dll", L"txt", L"foo.txt"},
-  {L"foo.dll", L".txt", L"foo.txt"},
-  {L"foo", L"txt", L"foo.txt"},
-  {L"foo", L".txt", L"foo.txt"},
-  {L"foo.baz.dll", L"txt", L"foo.baz.txt"},
-  {L"foo.baz.dll", L".txt", L"foo.baz.txt"},
-  {L"foo.dll", L"", L"foo"},
-  {L"foo.dll", L".", L"foo"},
-  {L"foo", L"", L"foo"},
-  {L"foo", L".", L"foo"},
-  {L"foo.baz.dll", L"", L"foo.baz"},
-  {L"foo.baz.dll", L".", L"foo.baz"},
+  {L"", FILE_PATH_LITERAL(""), L""},
+  {L"", FILE_PATH_LITERAL("txt"), L".txt"},
+  {L".", FILE_PATH_LITERAL("txt"), L".txt"},
+  {L".", FILE_PATH_LITERAL(""), L""},
+  {L"foo.dll", FILE_PATH_LITERAL("txt"), L"foo.txt"},
+  {L"foo.dll", FILE_PATH_LITERAL(".txt"), L"foo.txt"},
+  {L"foo", FILE_PATH_LITERAL("txt"), L"foo.txt"},
+  {L"foo", FILE_PATH_LITERAL(".txt"), L"foo.txt"},
+  {L"foo.baz.dll", FILE_PATH_LITERAL("txt"), L"foo.baz.txt"},
+  {L"foo.baz.dll", FILE_PATH_LITERAL(".txt"), L"foo.baz.txt"},
+  {L"foo.dll", FILE_PATH_LITERAL(""), L"foo"},
+  {L"foo.dll", FILE_PATH_LITERAL("."), L"foo"},
+  {L"foo", FILE_PATH_LITERAL(""), L"foo"},
+  {L"foo", FILE_PATH_LITERAL("."), L"foo"},
+  {L"foo.baz.dll", FILE_PATH_LITERAL(""), L"foo.baz"},
+  {L"foo.baz.dll", FILE_PATH_LITERAL("."), L"foo.baz"},
 };
 
 TEST_F(FileUtilTest, ReplaceExtensionTest) {
   for (unsigned int i = 0; i < arraysize(kReplaceExtension); ++i) {
-    std::wstring file_name(kReplaceExtension[i].file_name);
-    file_util::ReplaceExtension(&file_name, kReplaceExtension[i].extension);
-    EXPECT_EQ(file_name, kReplaceExtension[i].result);
+    FilePath path = FilePath::FromWStringHack(kReplaceExtension[i].file_name);
+    file_util::ReplaceExtension(&path, kReplaceExtension[i].extension);
+    EXPECT_EQ(kReplaceExtension[i].result, path.ToWStringHack());
   }
 }
 
 // Make sure ReplaceExtension doesn't replace an extension that occurs as one of
 // the directory names of the path.
 TEST_F(FileUtilTest, ReplaceExtensionTestWithPathSeparators) {
-  std::wstring path;
-  file_util::AppendToPath(&path, L"foo.bar");
-  file_util::AppendToPath(&path, L"foo");
+  FilePath path;
+  path = path.Append(FILE_PATH_LITERAL("foo.bar"));
+  path = path.Append(FILE_PATH_LITERAL("foo"));
   // '/foo.bar/foo' with extension '.baz'
-  std::wstring result_path = path;
-  file_util::ReplaceExtension(&result_path, L".baz");
-  EXPECT_EQ(path + L".baz", result_path);
+  FilePath result_path = path;
+  file_util::ReplaceExtension(&result_path, FILE_PATH_LITERAL(".baz"));
+  EXPECT_EQ(path.ToWStringHack() + L".baz", result_path.ToWStringHack());
 }
-#endif  // defined(OS_WIN)
 
 TEST_F(FileUtilTest, FileEnumeratorTest) {
   // Test an empty directory.
   file_util::FileEnumerator f0(test_dir_.ToWStringHack(), true,
       file_util::FileEnumerator::FILES_AND_DIRECTORIES);
   EXPECT_EQ(f0.Next(), L"");
   EXPECT_EQ(f0.Next(), L"");
 
   // create the directories
   FilePath dir1 = test_dir_.Append(FILE_PATH_LITERAL("dir1"));
@@ skipping 131 matching lines @@
     for (size_t j = 0; j < comps.size(); ++j) {
       result.append(comps[j]);
       if (j < comps.size() - 1)
         result.append(FILE_PATH_LITERAL("|"), 1);
     }
     EXPECT_EQ(kPathComponents[i].result, result);
   }
 }
 
 }  // namespace