Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(194)

Unified Diff: chrome/browser/browser_main.cc

Issue 7113008: Add revocation checking field trial. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/browser_main.cc
diff --git a/chrome/browser/browser_main.cc b/chrome/browser/browser_main.cc
index 81662bbc6d0672ec33e36b01295575e6936243d2..c93481061ff5562822e14a44d0ce79b5624115db 100644
--- a/chrome/browser/browser_main.cc
+++ b/chrome/browser/browser_main.cc
@@ -502,6 +502,23 @@ void BrowserMainParts::ConnectBackupJobsFieldTrial() {
}
}
+void BrowserMainParts::RevocationCheckingDisabledFieldTrial() {
+ const base::FieldTrial::Probability kDivisor = 100;
+ base::FieldTrial::Probability probability = 50; // 50/50 trial
+
+ // After August 30, 2011 builds, it will always be in default group.
+ scoped_refptr<base::FieldTrial> trial(
+ new base::FieldTrial(
+ "RevCheckingImpact", kDivisor, "RevChecking_control", 2011, 8, 30));
Mike Belshe 2011/06/08 04:50:38 nit: change "RevChecking_control" to "_control"?
agl 2011/06/08 18:57:12 Have changed to "control" since none of the other
+
+ int disabled_group = trial->AppendGroup(
+ "RevChecking_disabled_for_pinned_sites", probability);
Mike Belshe 2011/06/08 04:50:38 nit: change to "_disabled"
agl 2011/06/08 18:57:12 Done (with the same remark about the leading under
+
+ int trial_grp = trial->group();
+ if (trial_grp == disabled_group)
+ net::SSLConfigService::DisableRevCheckingForPinnedSites();
+}
+
// BrowserMainParts: |MainMessageLoopStart()| and related ----------------------
void BrowserMainParts::MainMessageLoopStart() {
@@ -592,6 +609,7 @@ void BrowserMainParts::SetupFieldTrials(bool metrics_recording_enabled) {
prerender::ConfigurePrefetchAndPrerender(parsed_command_line());
SpdyFieldTrial();
ConnectBackupJobsFieldTrial();
+ RevocationCheckingDisabledFieldTrial();
}
// -----------------------------------------------------------------------------

Powered by Google App Engine
This is Rietveld 408576698