CommandLine refactoring and cleanup.

chrome/installer/util/install_util.cc

 // Copyright (c) 2011 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.
 //
 // See the corresponding header file for description of the functions in this
 // file.
 
 #include "chrome/installer/util/install_util.h"
 
 #include <shellapi.h>
@@ skipping 341 matching lines @@
       return 0;
     default:
       return status;
   }
 }
 
 // static
 void InstallUtil::MakeUninstallCommand(const std::wstring& exe_path,
                                        const std::wstring& arguments,
                                        CommandLine* command_line) {
-  const bool no_program = exe_path.empty();
-
-  // Return a bunch of nothingness.
-  if (no_program && arguments.empty()) {
-    *command_line = CommandLine(CommandLine::NO_PROGRAM);
-  } else {
-    // Form a full command line string.
-    std::wstring command;
-    command.append(1, L'"')
-        .append(no_program ? L"" : exe_path)
-        .append(L"\" ")
-        .append(arguments);
-
-    // If we have a program name, return this complete command line.
-    *command_line = CommandLine::FromString(command);
-  }
+  // Surround non-empty program name with simple quotes.
+  std::wstring program(!exe_path.empty() ? L"\"" + exe_path + L"\" " : L"\"\"");
+  *command_line = CommandLine::FromString(program + arguments);

[Evan Martin, 2011/05/10 23:48:32]

This is a little bit terrifying to me.  How certain are you that it is correct?

I guess we don't expect our exe_path to be too crazy with the quotes...

[msw, 2011/05/11 02:28:12]

I agree that this isn't robust quoting/escaping, but I mostly maintained
existing behavior (except this also uses empty double quotes instead of
NO_PROGRAM/empty-string when |exe_path| AND |arguments| are empty). I would
update this to use the CommandLine::StringVector ctor (w/CommandLine quoting as
needed), but thought against naively splitting the |arguments| string, which is
retrieved from a registry entry (kUninstallArgumentsField) via
installer::ProductState::Initialize. Greg, what are your thoughts?

 }
 
 std::wstring InstallUtil::GetCurrentDate() {
   static const wchar_t kDateFormat[] = L"yyyyMMdd";
   wchar_t date_str[arraysize(kDateFormat)] = {0};
   int len = GetDateFormatW(LOCALE_INVARIANT, 0, NULL, kDateFormat,
                            date_str, arraysize(date_str));
   if (len) {
     --len;  // Subtract terminating \0.
   } else {
     PLOG(DFATAL) << "GetDateFormat";
   }
 
   return std::wstring(date_str, len);
 }