Remove Chrome Frame's delay shutdown field trial.

chrome/browser/automation/chrome_frame_automation_provider_win.cc

 // 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_win.h"
 
 #include <algorithm>

[grt (UTC plus 2), 2013/04/11 18:15:57]

remove

[robertshield, 2013/04/12 19:49:07]

Done.

 
 #include "base/command_line.h"

[grt (UTC plus 2), 2013/04/11 18:15:57]

remove

[robertshield, 2013/04/12 19:49:07]

Done.

 #include "base/strings/string_number_conversions.h"

[grt (UTC plus 2), 2013/04/11 18:15:57]

remove

[robertshield, 2013/04/12 19:49:07]

Done.

 #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_channel.h"
 #include "ipc/ipc_message.h"
 
 const int kMaxChromeShutdownDelaySeconds = 60*60;

[grt (UTC plus 2), 2013/04/11 18:15:57]

remove

[robertshield, 2013/04/12 19:49:07]

Done.

 
 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) {

[grt (UTC plus 2), 2013/04/11 18:15:57]

remove braces

[robertshield, 2013/04/12 19:49:07]

Done.

-    CommandLine& cmd_line = *CommandLine::ForCurrentProcess();
-
-    CommandLine::StringType shutdown_delay(
-        cmd_line.GetSwitchValueNative(switches::kChromeFrameShutdownDelay));
-    if (!shutdown_delay.empty()) {
-      VLOG(1) << "ChromeFrameAutomationProvider: "
-                 "Scheduling ReleaseBrowserProcess.";
-
-      // Grab the specified shutdown delay.
-      int shutdown_delay_seconds = 0;
-      base::StringToInt(shutdown_delay, &shutdown_delay_seconds);
-
-      // Clamp to reasonable values.
-      shutdown_delay_seconds = std::max(0, shutdown_delay_seconds);
-      shutdown_delay_seconds = std::min(shutdown_delay_seconds,
-                                        kMaxChromeShutdownDelaySeconds);
-
-      // We have Chrome Frame defer Chrome shutdown for a time to improve
-      // intra-page load times.
-      // Note that we are tracking the perf impact of this under
-      // http://crbug.com/98506
-      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();
-    }
+    g_browser_process->ReleaseModule();
   }
 }
 
 bool ChromeFrameAutomationProvider::OnMessageReceived(
     const IPC::Message& message) {
   if (IsValidMessage(message.type()))
     return AutomationProvider::OnMessageReceived(message);
 
   OnUnhandledMessage(message);
   return false;
@@ skipping 42 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();
-  }
-}