Index: chrome_frame/chrome_frame_automation.cc |
diff --git a/chrome_frame/chrome_frame_automation.cc b/chrome_frame/chrome_frame_automation.cc |
index 633a1ae2ca4dce8cfdf4d3223ceda6d067773c61..3247343d881b339825a4a16996e6b256887cff65 100644 |
--- a/chrome_frame/chrome_frame_automation.cc |
+++ b/chrome_frame/chrome_frame_automation.cc |
@@ -14,7 +14,6 @@ |
#include "base/file_version_info.h" |
#include "base/lazy_instance.h" |
#include "base/logging.h" |
-#include "base/metrics/field_trial.h" |
#include "base/path_service.h" |
#include "base/process_util.h" |
#include "base/string_util.h" |
@@ -43,10 +42,6 @@ int64 kAutomationServerReasonableLaunchDelay = 1000; // in milliseconds |
int64 kAutomationServerReasonableLaunchDelay = 1000 * 10; |
#endif |
-const char kChromeShutdownDelaySeconds[] = "30"; |
-const char kWithDelayFieldTrialName[] = "WithShutdownDelay"; |
-const char kNoDelayFieldTrialName[] = "NoShutdownDelay"; |
- |
} // namespace |
class ChromeFrameAutomationProxyImpl::TabProxyNotificationMessageFilter |
@@ -302,10 +297,6 @@ void AutomationProxyCacheEntry::CreateProxy(ChromeFrameLaunchParams* params, |
if (IsAccessibleMode()) |
command_line->AppendSwitch(switches::kForceRendererAccessibility); |
- if (params->send_shutdown_delay_switch()) |
- command_line->AppendSwitchASCII(switches::kChromeFrameShutdownDelay, |
- kChromeShutdownDelaySeconds); |
- |
DVLOG(1) << "Profile path: " << params->profile_path().value(); |
command_line->AppendSwitchPath(switches::kUserDataDir, |
params->profile_path()); |
@@ -350,19 +341,9 @@ void AutomationProxyCacheEntry::CreateProxy(ChromeFrameLaunchParams* params, |
if (launch_result_ == AUTOMATION_SUCCESS) { |
UMA_HISTOGRAM_TIMES( |
"ChromeFrame.AutomationServerLaunchSuccessTime", delta); |
- UMA_HISTOGRAM_TIMES( |
- base::FieldTrial::MakeName( |
- "ChromeFrame.AutomationServerLaunchSuccessTime", |
- "ChromeShutdownDelay"), |
- delta); |
} else { |
UMA_HISTOGRAM_TIMES( |
"ChromeFrame.AutomationServerLaunchFailedTime", delta); |
- UMA_HISTOGRAM_TIMES( |
- base::FieldTrial::MakeName( |
- "ChromeFrame.AutomationServerLaunchFailedTime", |
- "ChromeShutdownDelay"), |
- delta); |
} |
UMA_HISTOGRAM_CUSTOM_COUNTS("ChromeFrame.LaunchResult", |
@@ -557,9 +538,7 @@ ChromeFrameAutomationClient::ChromeFrameAutomationClient() |
url_fetcher_(NULL), |
url_fetcher_flags_(PluginUrlRequestManager::NOT_THREADSAFE), |
navigate_after_initialization_(false), |
- route_all_top_level_navigations_(false), |
- send_shutdown_delay_switch_(true) { |
- InitializeFieldTrials(); |
+ route_all_top_level_navigations_(false) { |
} |
ChromeFrameAutomationClient::~ChromeFrameAutomationClient() { |
@@ -698,8 +677,7 @@ bool ChromeFrameAutomationClient::InitiateNavigation( |
base::FilePath profile_path; |
chrome_launch_params_ = new ChromeFrameLaunchParams(parsed_url, |
referrer_gurl, profile_path, L"", SimpleResourceLoader::GetLanguage(), |
- false, false, route_all_top_level_navigations_, |
- send_shutdown_delay_switch_); |
+ false, false, route_all_top_level_navigations_); |
} else { |
chrome_launch_params_->set_referrer(referrer_gurl); |
chrome_launch_params_->set_url(parsed_url); |
@@ -1037,33 +1015,6 @@ bool ChromeFrameAutomationClient::ProcessUrlRequestMessage(TabProxy* tab, |
return true; |
} |
-void ChromeFrameAutomationClient::InitializeFieldTrials() { |
- static base::FieldTrial* trial = NULL; |
- if (!trial) { |
- // Do one-time initialization of the field trial here. |
- // TODO(robertshield): End the field trial before March 7th 2013. |
- scoped_refptr<base::FieldTrial> new_trial = |
- base::FieldTrialList::FactoryGetFieldTrial( |
- "ChromeShutdownDelay", 1000, kWithDelayFieldTrialName, |
- 2013, 3, 7, NULL); |
- |
- // Be consistent for this client. Note that this will only have an effect |
- // once the client id is persisted. See http://crbug.com/117188 |
- new_trial->UseOneTimeRandomization(); |
- |
- new_trial->AppendGroup(kNoDelayFieldTrialName, 500); // 50% without. |
- |
- trial = new_trial.get(); |
- } |
- |
- // Take action depending of which group we randomly land in. |
- if (trial->group_name() == kWithDelayFieldTrialName) |
- send_shutdown_delay_switch_ = true; |
- else |
- send_shutdown_delay_switch_ = false; |
- |
-} |
- |
// These are invoked in channel's background thread. |
// Cannot call any method of the activex here since it is a STA kind of being. |
// By default we marshal the IPC message to the main/GUI thread and from there |