CommandLine: eliminate wstring-accepting AppendLooseValue

Instead use AppendArg variants which accept a FilePath or an ASCII string.

Review URL: http://codereview.chromium.org/3134008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56100 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 6 deletions

diff --git a/chrome_frame/test/chrome_frame_test_utils.cc b/chrome_frame/test/chrome_frame_test_utils.cc
index 5f59f6c..f5b1315 100644
--- a/chrome_frame/test/chrome_frame_test_utils.cc
+++ b/chrome_frame/test/chrome_frame_test_utils.cc
@@ -159,7 +159,7 @@ base::ProcessHandle LaunchExecutable(const std::wstring& executable,
     }
   } else {
     CommandLine cmdline((FilePath(path)));
-    cmdline.AppendLooseValue(argument);
+    cmdline.AppendArgNative(argument);
     if (!base::LaunchApp(cmdline, false, false, &process)) {
       LOG(ERROR) << "LaunchApp failed: " << ::GetLastError();
     }
@@ -181,11 +181,8 @@ base::ProcessHandle LaunchChrome(const std::wstring& url) {
   path = path.AppendASCII(kChromeImageName);
 
   CommandLine cmd(path);
-  std::wstring args = L"--";
-  args += ASCIIToWide(switches::kNoFirstRun);
-  args += L" ";
-  args += url;
-  cmd.AppendLooseValue(args);
+  cmd.AppendSwitch(switches::kNoFirstRun);
+  cmd.AppendArgNative(url);
 
   base::ProcessHandle process = NULL;
   base::LaunchApp(cmd, false, false, &process);