Handle spaces in the name of the mini_installer.

chrome/installer/mini_installer/mini_installer.h

Index: chrome/installer/mini_installer/mini_installer.h
diff --git a/chrome/installer/mini_installer/mini_installer.h b/chrome/installer/mini_installer/mini_installer.h
index 8f2703a1899954220fb5d20a9354543d79416fd6..cb62f38324f4d23ff46f075adf1172a827699591 100644
--- a/chrome/installer/mini_installer/mini_installer.h
+++ b/chrome/installer/mini_installer/mini_installer.h
@@ -8,6 +8,7 @@
 #include <windows.h>
 
 #include "chrome/installer/mini_installer/exit_code.h"
+#include "chrome/installer/mini_installer/mini_string.h"
 
 namespace mini_installer {
 
@@ -24,6 +25,14 @@ struct ProcessExitResult {
   bool IsSuccess() const { return exit_code == SUCCESS_EXIT_CODE; }
 };
 
+// A stack-based string large enough to hold an executable to run, two path
+// arguments, plus a few extra arguments.
+using CommandString = StackString<MAX_PATH * 4>;

[bcwhite, 2017/01/27 14:52:24]

Change to

  MAX_PATH * 2 + NNN

to match the comment?

[grt (UTC plus 2), 2017/01/27 15:31:38]

If you mean "NNN" to be the room for extra arguments, it would be MAX_PATH * 3 +
NNN (the executable to run is itself a path). I've beefed up the comment here.

+
+// Appends all command line flags (i.e., switches and arguments) in
+// |command_line| to |buffer|. |buffer| is unchanged in case of error.

[bcwhite, 2017/01/27 14:52:24]

// The flags section is copied exactly: spaces, quotes and all.

[grt (UTC plus 2), 2017/01/27 15:31:38]

Done.

+void AppendCommandLineFlags(const wchar_t* command_line, CommandString* buffer);
+
 // Main function for Chrome's mini_installer. First gets a working dir, unpacks
 // the resources, and finally executes setup.exe to do the install/update. Also
 // handles invoking a previous version's setup.exe to patch itself in the case