Prefix Chrome switches with the switch prefix.

win8/delegate_execute/delegate_execute_util.cc

+// Copyright (c) 2012 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 "win8/delegate_execute/delegate_execute_util.h"
+
+#include "base/file_path.h"
+#include "base/string_util.h"
+
+namespace delegate_execute {
+
+CommandLine CommandLineFromParameters(const wchar_t* params) {
+  CommandLine command_line(CommandLine::NO_PROGRAM);
+
+  if (params) {

[cpu_(ooo_6.6-7.5), 2012/09/28 20:03:13]

is it valid to pass no params? or rather just CHECK here?

[grt (UTC plus 2), 2012/09/28 20:11:23]

i'm not sure if the shell will ever call CommandExecuteImpl::SetParameters with
NULL.  the previous code in SetParameters handled NULL similarly to this code. 
i don't see any harm in handling it this way.

+    string16 command_string(L"noprogram.exe ");
+    command_string.append(params);
+    command_line.ParseFromString(command_string);
+    command_line.SetProgram(FilePath());
+  }
+
+  return command_line;
+}
+
+CommandLine MakeChromeCommandLine(const FilePath& chrome_exe,
+                                  const CommandLine& params,
+                                  const string16& argument) {
+  CommandLine chrome_cmd(params);
+  chrome_cmd.SetProgram(chrome_exe);
+
+  if (!argument.empty())
+    chrome_cmd.AppendArgNative(argument);
+
+  return chrome_cmd;
+}
+
+string16 ParametersFromSwitch(const char* a_switch) {
+  if (!a_switch)
+    return string16();
+
+  CommandLine cmd_line(CommandLine::NO_PROGRAM);
+
+  cmd_line.AppendSwitch(a_switch);
+
+  string16 command_string(cmd_line.GetCommandLineString());
+  TrimWhitespace(command_string, TRIM_ALL, &command_string);
+  return command_string;
+}
+
+}  // namespace delegate_execute