Commit 5a046eaa authored by Greg Thompson's avatar Greg Thompson Committed by Commit Bot

Mark kOtherLaunch and kOtherClose as terminal states for the inactive user toast.

This prevents showing the toast to the user a second time in these cases.

BUG=786918
R=skare@chromium.org

Change-Id: I8858e7cb1acc247c011494cd4d446a75f19e18a9
Reviewed-on: https://chromium-review.googlesource.com/778844
Commit-Queue: Chris Hamilton <chrisha@chromium.org>
Reviewed-by: default avatarChris Hamilton <chrisha@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517825}
parent e5c30e0b
...@@ -11,9 +11,11 @@ bool ExperimentMetrics::InInitialState() const { ...@@ -11,9 +11,11 @@ bool ExperimentMetrics::InInitialState() const {
} }
bool ExperimentMetrics::InTerminalState() const { bool ExperimentMetrics::InTerminalState() const {
static_assert(NUM_STATES == 18,
"update the list of terminal states when adding a new state.");
return state == kSelectedNoThanks || state == kSelectedOpenChromeAndCrash || return state == kSelectedNoThanks || state == kSelectedOpenChromeAndCrash ||
state == kSelectedOpenChromeAndNoCrash || state == kSelectedClose || state == kSelectedOpenChromeAndNoCrash || state == kSelectedClose ||
state == kUserLogOff; state == kUserLogOff || state == kOtherLaunch || state == kOtherClose;
} }
bool ExperimentMetrics::operator==(const ExperimentMetrics& other) const { bool ExperimentMetrics::operator==(const ExperimentMetrics& other) const {
......
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