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"
 
+const int kMaxChromeShutdownDelaySeconds = 60*60;
+
 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();
+
+    CommandLine::StringType shutdown_delay(
+        cmd_line.GetSwitchValueNative(switches::kChromeFrameShutdownDelay));
+    if (!shutdown_delay.empty()) {
+      VLOG(1) << "ChromeFrameAutomationProvider: "

[slightlyoff, 2012/03/05 12:35:53]

this is going to use streamio which, IIRC, once had a static initializer. Do we
still do that in Chrome?

[robertshield, 2012/03/05 13:33:25]

We use this pattern to log all throughout the code base. What do you suggest as
an alternative?

+                 "Scheduling ReleaseBrowserProcess.";
+
+      // Grab the specified shutdown delay.
+      int shutdown_delay_seconds = 0;
+      base::StringToInt(shutdown_delay, &shutdown_delay_seconds);
+
+      // Clamp to reasonable values.
+      if (shutdown_delay_seconds < 0)
+        shutdown_delay_seconds = 0;
+      else if (shutdown_delay_seconds > kMaxChromeShutdownDelaySeconds)
+        shutdown_delay_seconds = kMaxChromeShutdownDelaySeconds;

[slightlyoff, 2012/03/05 12:35:53]

Are there min/max functions somewhere for this?

[robertshield, 2012/03/05 13:33:25]

Done.

+
+      // We have Chrome Frame defer Chrome shutdown for a time to improve
+      // intra-page load times.

[slightlyoff, 2012/03/05 12:35:53]

Add a bug # for tracking perf here?

[robertshield, 2012/03/05 13:33:25]

Good idea, I'll track it under the original bug number if that's alright.

+      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();
+  }
+}