GN: Move towards only using / on Windows

tools/gn/function_rebase_path_unittest.cc

 // Copyright (c) 2013 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"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "tools/gn/functions.h"
 #include "tools/gn/parse_tree.h"
 #include "tools/gn/test_with_scope.h"
 
 namespace {
 
 std::string RebaseOne(Scope* scope,
                       const char* input,
                       const char* to_dir,
-                      const char* from_dir,
-                      const char* sep = NULL) {
+                      const char* from_dir) {
   std::vector<Value> args;
   args.push_back(Value(NULL, input));
   args.push_back(Value(NULL, to_dir));
   args.push_back(Value(NULL, from_dir));
-  if (sep)
-    args.push_back(Value(NULL, sep));
 
   Err err;
   FunctionCallNode function;
   Value result = functions::RunRebasePath(scope, &function, args, &err);
   bool is_string = result.type() == Value::STRING;
   EXPECT_TRUE(is_string);
 
   return result.string_value();
 }
 
@@ skipping 19 matching lines @@
   EXPECT_EQ("./", RebaseOne(scope, "//", "//", "//"));
   EXPECT_EQ("foo", RebaseOne(scope, "//foo", "//", "//"));
   EXPECT_EQ("foo/", RebaseOne(scope, "//foo/", "//", "//"));
   EXPECT_EQ("../../foo/bar", RebaseOne(scope, "//foo/bar", "//out/Debug", "."));
   EXPECT_EQ("./", RebaseOne(scope, "//foo/", "//foo/", "//"));
   // Thie one is technically correct but could be simplified to "." if
   // necessary.
   EXPECT_EQ("../foo", RebaseOne(scope, "//foo", "//foo", "//"));
 
   // Test slash conversion.
-  EXPECT_EQ("foo/bar", RebaseOne(scope, "foo/bar", ".", ".", "to_slash"));
-  EXPECT_EQ("foo/bar", RebaseOne(scope, "foo\\bar", ".", ".", "to_slash"));
-#if defined(OS_WIN)
-  EXPECT_EQ("foo\\bar", RebaseOne(scope, "foo/bar", ".", ".", "to_system"));
-  EXPECT_EQ("foo\\bar", RebaseOne(scope, "foo\\bar", ".", ".", "to_system"));
-#endif
+  EXPECT_EQ("foo/bar", RebaseOne(scope, "foo/bar", ".", "."));
+  EXPECT_EQ("foo/bar", RebaseOne(scope, "foo\\bar", ".", "."));
 
   // Test system path output.
 #if defined(OS_WIN)
-  setup.build_settings()->SetRootPath(base::FilePath(L"C:\\source"));
-  EXPECT_EQ("C:\\source", RebaseOne(scope, ".", "", "//"));
-  EXPECT_EQ("C:\\source\\", RebaseOne(scope, "//", "", "//"));
-  EXPECT_EQ("C:\\source\\foo", RebaseOne(scope, "foo", "", "//"));
-  EXPECT_EQ("C:\\source\\foo\\", RebaseOne(scope, "foo/", "", "//"));
-  EXPECT_EQ("C:\\source\\tools\\gn\\foo", RebaseOne(scope, "foo", "", "."));
+  setup.build_settings()->SetRootPath(base::FilePath(L"C:/source"));
+  EXPECT_EQ("C:/source", RebaseOne(scope, ".", "", "//"));
+  EXPECT_EQ("C:/source/", RebaseOne(scope, "//", "", "//"));
+  EXPECT_EQ("C:/source/foo", RebaseOne(scope, "foo", "", "//"));
+  EXPECT_EQ("C:/source/foo/", RebaseOne(scope, "foo/", "", "//"));
+  EXPECT_EQ("C:/source/tools/gn/foo", RebaseOne(scope, "foo", "", "."));
 #else
   setup.build_settings()->SetRootPath(base::FilePath("/source"));
   EXPECT_EQ("/source", RebaseOne(scope, ".", "", "//"));
   EXPECT_EQ("/source/", RebaseOne(scope, "//", "", "//"));
   EXPECT_EQ("/source/foo", RebaseOne(scope, "foo", "", "//"));
   EXPECT_EQ("/source/foo/", RebaseOne(scope, "foo/", "", "//"));
   EXPECT_EQ("/source/tools/gn/foo", RebaseOne(scope, "foo", "", "."));
 #endif
 }
 
@@ skipping 26 matching lines @@
   TestWithScope setup;
   setup.build_settings()->SetBuildDir(SourceDir("//out/Debug/"));
 
   // No arg input should issue an error.
   Err err;
   std::vector<Value> args;
   FunctionCallNode function;
   Value ret = functions::RunRebasePath(setup.scope(), &function, args, &err);
   EXPECT_TRUE(err.has_error());
 }