Register Application Restart with Windows Restart Manager

chrome/browser/browser_main.cc

Index: chrome/browser/browser_main.cc
diff --git a/chrome/browser/browser_main.cc b/chrome/browser/browser_main.cc
index 0819289a032c8b84e8c8c4134cd74e529e951512..2e8785b9e977726f5a9049ce48db395bf3c3686a 100644
--- a/chrome/browser/browser_main.cc
+++ b/chrome/browser/browser_main.cc
@@ -1517,6 +1517,10 @@ int BrowserMain(const MainFunctionParams& parameters) {
   // testing against a bunch of special cases that are taken care early on.
   PrepareRestartOnCrashEnviroment(parsed_command_line);
 
+#if defined(OS_WIN)
+  RegisterApplicationRestart(parsed_command_line);

[Ben Goodger (Google), 2011/02/10 16:56:49]

You might add a quick note here saying what this does, like the comment for
PrepareRestartOnCrashEnvironment does above it.

Given the size of BrowserMain(), you should also note if doing it at this
particular point is at all important, or if you just threw it in here because
this seemed good enough :-)

[msw, 2011/02/10 18:35:06]

Done. Added your same comment from the function definition plus a rationale for
calling in browser_main; please let me know of any better code paths to host
this call.

+#endif  // OS_WIN
+
   // Initialize and maintain network predictor module, which handles DNS
   // pre-resolution, as well as TCP/IP connection pre-warming.
   // This also registers an observer to discard data when closing incognito
@@ -1539,8 +1543,8 @@ int BrowserMain(const MainFunctionParams& parameters) {
   // file thread to be run sometime later. If this is the first run we record
   // the installation event.
   RLZTracker::InitRlzDelayed(is_first_run, master_prefs.ping_delay);
-#endif
-#endif
+#endif  // GOOGLE_CHROME_BUILD
+#endif  // OS_WIN
 
   // Configure modules that need access to resources.
   net::NetModule::SetResourceProvider(chrome_common_net::NetResourceProvider);
@@ -1720,7 +1724,7 @@ int BrowserMain(const MainFunctionParams& parameters) {
     parameters.ui_task->Run();
     delete parameters.ui_task;
   } else {
-    // We are in regular browser boot sequence. Open initial stabs and enter
+    // We are in regular browser boot sequence. Open initial tabs and enter
     // the main message loop.
     if (browser_init.Start(parsed_command_line, FilePath(), profile,
                            &result_code)) {