Changing the installer switches from wchar_t[] to char[]....

base/command_line_unittest.cc

Index: base/command_line_unittest.cc
===================================================================
--- base/command_line_unittest.cc	(revision 65856)
+++ base/command_line_unittest.cc	(working copy)
@@ -154,3 +154,17 @@
             cl.command_line_string());
 #endif
 }
+
+// Tests that when AppendArguments is called that both the program and
+// command line strings are equal.

[evanm, 2010/11/14 19:15:39]

Maybe change the end of this sentece to "...is called with an empty CommandLine,
nothing changes".

[tommi (sloooow) - chröme, 2010/11/14 21:28:07]

Changed the description after incorporating Robert's suggestions.

+TEST(CommandLineTest, AppendArguments) {
+  CommandLine cl1(FilePath(FILE_PATH_LITERAL("Program")));
+  cl1.AppendSwitch("switch1");
+  cl1.AppendSwitchASCII("switch2", "foo");
+
+  CommandLine cl2(CommandLine::NO_PROGRAM);
+  cl2.AppendArguments(cl1, true);

[evanm, 2010/11/14 19:15:39]

Shouldn't this hit the DCHECK(), since it's saying "include the other guy's
program", but the other guy's program is empty?

[evanm, 2010/11/14 19:16:18]

Never mind, ready it backwards.  :(

+  EXPECT_EQ(cl1.command_line_string(), cl2.command_line_string());
+  EXPECT_EQ(cl1.GetProgram(), cl2.GetProgram());
+}
+