Commit 0a18637b authored by Greg Thompson's avatar Greg Thompson Committed by Commit Bot

Enable the Windows 10 inactive user toast experiment.

BUG=717091
R=nikunjb@chromium.org, skare@chromium.org

Change-Id: I1e8d1d9ac4dd7f5f9e1feb80aaad38dfea6ba7cc
Reviewed-on: https://chromium-review.googlesource.com/544817
Commit-Queue: Greg Thompson <grt@chromium.org>
Reviewed-by: default avatarPatrick Monette <pmonette@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517041}
parent fe0543c9
...@@ -53,17 +53,10 @@ constexpr ExperimentStorage::Study kCurrentStudy = ExperimentStorage::kStudyOne; ...@@ -53,17 +53,10 @@ constexpr ExperimentStorage::Study kCurrentStudy = ExperimentStorage::kStudyOne;
constexpr int kStudyTwoGroup = 0; constexpr int kStudyTwoGroup = 0;
// Test switches. // Test switches.
constexpr char kExperimentEnableForTesting[] = "experiment-enable-for-testing";
constexpr char kExperimentEnterpriseBypass[] = "experiment-enterprise-bypass"; constexpr char kExperimentEnterpriseBypass[] = "experiment-enterprise-bypass";
constexpr char kExperimentParticipation[] = "experiment-participation"; constexpr char kExperimentParticipation[] = "experiment-participation";
constexpr char kExperimentRetryDelay[] = "experiment-retry-delay"; constexpr char kExperimentRetryDelay[] = "experiment-retry-delay";
// Returns true if the experiment is enabled for testing.
bool IsExperimentEnabledForTesting() {
return base::CommandLine::ForCurrentProcess()->HasSwitch(
kExperimentEnableForTesting);
}
// Returns true if the install originated from the MSI or if the machine is // Returns true if the install originated from the MSI or if the machine is
// joined to a domain. This check can be bypassed via // joined to a domain. This check can be bypassed via
// --experiment-enterprise-bypass. // --experiment-enterprise-bypass.
...@@ -236,11 +229,8 @@ bool ShouldRunUserExperiment(const InstallerState& installer_state) { ...@@ -236,11 +229,8 @@ bool ShouldRunUserExperiment(const InstallerState& installer_state) {
// Bail out if this install is not selected into the fraction participating in // Bail out if this install is not selected into the fraction participating in
// the current study. // the current study.
// NOTE: No clients will participate while this feature is under development. if (!IsSelectedForStudy(lock.get(), kCurrentStudy))
if (!IsExperimentEnabledForTesting() ||
!IsSelectedForStudy(lock.get(), kCurrentStudy)) {
return false; return false;
}
// Skip the experiment if a user on the machine has already reached a terminal // Skip the experiment if a user on the machine has already reached a terminal
// state. // state.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment