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

Unified Diff: ash/metrics/task_switch_metrics_recorder_unittest.cc

Issue 2390793002: Removed Ash.AppList.TimeBetweenTaskSwitches and Ash.Tab.TimeBetweenSwitchToExistingTabUserActions h… (Closed)
Patch Set: Fixed up ash_unittests. Created 4 years, 2 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
« no previous file with comments | « ash/metrics/task_switch_metrics_recorder.cc ('k') | chrome/browser/ui/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/metrics/task_switch_metrics_recorder_unittest.cc
diff --git a/ash/metrics/task_switch_metrics_recorder_unittest.cc b/ash/metrics/task_switch_metrics_recorder_unittest.cc
index 9a819316de6c147c1a7a365ae08cd7f9293b08da..804d73060389cb347930d42089df1b6931d9c726 100644
--- a/ash/metrics/task_switch_metrics_recorder_unittest.cc
+++ b/ash/metrics/task_switch_metrics_recorder_unittest.cc
@@ -114,21 +114,6 @@ TEST_F(TaskSwitchMetricsRecorderTest,
histogram_tester_->ExpectTotalCount(kHistogramName, 2);
}
-// Verifies that the TaskSwitchSource::TAB_STRIP source adds data to
-// the Ash.Tab.TimeBetweenSwitchToExistingTabUserActions histogram.
-TEST_F(TaskSwitchMetricsRecorderTest,
- VerifyTaskSwitchesFromTheTabStripAreRecorded) {
- const std::string kHistogramName =
- "Ash.Tab.TimeBetweenSwitchToExistingTabUserActions";
-
- OnTaskSwitch(TaskSwitchSource::TAB_STRIP);
- OnTaskSwitch(TaskSwitchSource::TAB_STRIP);
- histogram_tester_->ExpectTotalCount(kHistogramName, 1);
-
- OnTaskSwitch(TaskSwitchSource::TAB_STRIP);
- histogram_tester_->ExpectTotalCount(kHistogramName, 2);
-}
-
// Verifies that the TaskSwitchSource::OVERVIEW_MODE source adds data
// to the Ash.WindowSelector.TimeBetweenActiveWindowChanges histogram.
TEST_F(TaskSwitchMetricsRecorderTest,
@@ -144,18 +129,4 @@ TEST_F(TaskSwitchMetricsRecorderTest,
histogram_tester_->ExpectTotalCount(kHistogramName, 2);
}
-// Verifies that the TaskSwitchSource::APP_LIST source adds data to the
-// Ash.AppList.TimeBetweenTaskSwitches histogram.
-TEST_F(TaskSwitchMetricsRecorderTest,
- VerifyTaskSwitchesFromApplistAreRecorded) {
- const std::string kHistogramName = "Ash.AppList.TimeBetweenTaskSwitches";
-
- OnTaskSwitch(TaskSwitchSource::APP_LIST);
- OnTaskSwitch(TaskSwitchSource::APP_LIST);
- histogram_tester_->ExpectTotalCount(kHistogramName, 1);
-
- OnTaskSwitch(TaskSwitchSource::APP_LIST);
- histogram_tester_->ExpectTotalCount(kHistogramName, 2);
-}
-
} // namespace ash
« no previous file with comments | « ash/metrics/task_switch_metrics_recorder.cc ('k') | chrome/browser/ui/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698