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

Unified Diff: chrome/browser/chrome_browser_field_trials.cc

Issue 10855260: Safe Browsing malware interstitial redesign field trial. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: webui style fixes Created 8 years, 4 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/chrome_browser_field_trials.cc
diff --git a/chrome/browser/chrome_browser_field_trials.cc b/chrome/browser/chrome_browser_field_trials.cc
index f1e27a852d57317c77120cf072304e0ef51e6876..9642b6814b1e7cc5f0e4f25b37e6053f950b7fe3 100644
--- a/chrome/browser/chrome_browser_field_trials.cc
+++ b/chrome/browser/chrome_browser_field_trials.cc
@@ -20,6 +20,7 @@
#include "chrome/browser/gpu_util.h"
#include "chrome/browser/net/predictor.h"
#include "chrome/browser/prerender/prerender_field_trial.h"
+#include "chrome/browser/safe_browsing/safe_browsing_blocking_page.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/chrome_version_info.h"
#include "chrome/common/metrics/variations/variations_util.h"
@@ -124,6 +125,7 @@ void ChromeBrowserFieldTrials::SetupFieldTrials(bool proxy_policy_is_set) {
SetupUniformityFieldTrials();
AutocompleteFieldTrial::Activate();
DisableNewTabFieldTrialIfNecesssary();
+ SetUpSafeBrowsingInterstitialFieldTrial();
}
// This is an A/B test for the maximum number of persistent connections per
@@ -582,3 +584,11 @@ void ChromeBrowserFieldTrials::DisableNewTabFieldTrialIfNecesssary() {
trial->Disable();
}
}
+
+void ChromeBrowserFieldTrials::SetUpSafeBrowsingInterstitialFieldTrial() {
+ scoped_refptr<base::FieldTrial> trial(
+ base::FieldTrialList::FactoryGetFieldTrial("SBInterstitial", 100,
arv (Not doing code reviews) 2012/08/22 14:33:45 What are these constant? Same Q below.
mattm 2012/08/22 20:33:47 Done.
+ "V1", 2012, 9, 19, NULL));
+ trial->UseOneTimeRandomization();
+ trial->AppendGroup("V2", 50);
+}

Powered by Google App Engine
This is Rietveld 408576698