Defer shutting down Chrome after the CF automation connection goes away.

chrome/browser/automation/chrome_frame_automation_provider.cc

-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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 "chrome/browser/automation/chrome_frame_automation_provider.h"
+
+#include "base/command_line.h"
+#include "base/string_number_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/common/automation_messages.h"
+#include "chrome/common/chrome_switches.h"
 #include "ipc/ipc_message.h"
 #include "ipc/ipc_channel.h"
 
+
 ChromeFrameAutomationProvider::ChromeFrameAutomationProvider(Profile* profile)
     : AutomationProvider(profile) {
   DCHECK(g_browser_process);
   if (g_browser_process)
     g_browser_process->AddRefModule();
 }
 
 ChromeFrameAutomationProvider::~ChromeFrameAutomationProvider() {
   DCHECK(g_browser_process);
-  if (g_browser_process)
-    g_browser_process->ReleaseModule();
+  if (g_browser_process) {
+    CommandLine& cmd_line = *CommandLine::ForCurrentProcess();
+    if (cmd_line.HasSwitch(switches::kChromeFrameShutdownDelay)) {

[grt (UTC plus 2), 2012/03/02 20:13:37]

please rework this block to:

CommandLine::StringType shutdown_delay(
    cmd_line.GetSwitchValueNative(switches::kChromeFrameShutdownDelay));
if (!shutdown_delay.empty()) {
  // delay
} else {
  // no delay
}

this avoids a string conversion and an extra lookup into cmd_line.

[robertshield, 2012/03/02 20:33:37]

Done.

+      VLOG(1) << "ChromeFrameAutomationProvider: "
+                 "Scheduling ReleaseBrowserProcess.";
+
+      // Grab the specified shutdown delay.
+      int shutdown_delay_seconds = 0;
+      std::string shutdown_delay(
+          cmd_line.GetSwitchValueASCII(switches::kChromeFrameShutdownDelay));
+      base::StringToInt(shutdown_delay, &shutdown_delay_seconds);

[grt (UTC plus 2), 2012/03/02 20:13:37]

in worst-case conditions, shutdown_delay_seconds could be set to INT_MAX or
INT_MIN.  how about erring on the safe side and ensuring that
shutdown_delay_seconds is positive and less than some reasonable enormous
upper-bound (e.g., 60*60*24)?

[robertshield, 2012/03/02 20:33:37]

Done.

+
+      // We have Chrome Frame defer Chrome shutdown for a time to improve
+      // intra-page load times.
+      MessageLoop::current()->PostDelayedTask(
+          FROM_HERE,
+          base::Bind(&ChromeFrameAutomationProvider::ReleaseBrowserProcess),
+          base::TimeDelta::FromSeconds(shutdown_delay_seconds));
+    } else {
+      VLOG(1) << "ChromeFrameAutomationProvider: "
+                 "Releasing browser module with no delay.";
+      g_browser_process->ReleaseModule();
+    }
+  }
 }
 
 bool ChromeFrameAutomationProvider::OnMessageReceived(
     const IPC::Message& message) {
   if (IsValidMessage(message.type()))
     return AutomationProvider::OnMessageReceived(message);
 
   OnUnhandledMessage(message);
   return false;
 }
@@ skipping 45 matching lines @@
       is_valid_message = true;
       break;
     }
 
     default:
       break;
   }
 
   return is_valid_message;
 }
+
+// static
+void ChromeFrameAutomationProvider::ReleaseBrowserProcess() {
+  if (g_browser_process) {
+    VLOG(1) << "ChromeFrameAutomationProvider: "
+               "Releasing browser process.";
+    g_browser_process->ReleaseModule();
+  }
+}