Because of metrics not being recorded.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/eca38d4605fa46a7822ca5522db352415d97eed4 commit eca38d4605fa46a7822ca5522db352415d97eed4 Author: sky <sky@chromium.org> Date: Fri Mar 03 05:21:56 2017 chromeos: moves more ash tests to run on mash And pointers to bugs for those that aren't moved to common. BUG= 631103 , 633782 , 634994 , 648733 , 695887 , 696752 ,696754, 698016 , 698024 ,698032,698033, 698043 , 698049 ,698060,698085, 698092 ,698093, 698129 TEST=test only changes R=msw@chromium.org Review-Url: https://codereview.chromium.org/2732573002 Cr-Commit-Position: refs/heads/master@{#454514} [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/BUILD.gn [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/focus_cycler_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/shelf/shelf_layout_manager_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/sticky_keys/sticky_keys_overlay_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/system/overview/overview_button_tray_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/system/toast/toast_manager_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/system/web_notification/ash_popup_alignment_delegate_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/system/web_notification/web_notification_tray_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/tooltips/tooltip_controller_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/wm/dock/docked_window_layout_manager_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/wm/dock/docked_window_resizer_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/wm/immersive_fullscreen_controller_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/wm/lock_state_controller_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/wm/maximize_mode/maximize_mode_window_manager_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/wm/overview/window_selector_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/wm/toplevel_window_event_handler_unittest.cc [modify] https://crrev.com/eca38d4605fa46a7822ca5522db352415d97eed4/ash/wm/workspace/workspace_window_resizer_unittest.cc
WindowSelectorTest.ActiveWindowChangedUserActionRecorded also fails, it looks like because of the same reason.
I changed the summary to reflect this effects a number of tests. The original bug was against OverviewButtonTrayTest.TrayOverviewUserAction.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ea37af717f1569dd93282e09b6fb49ff22b771f8 commit ea37af717f1569dd93282e09b6fb49ff22b771f8 Author: sky <sky@chromium.org> Date: Tue Mar 07 22:59:41 2017 Last sets of tests that can move to common_unittests This way they run in both mash_unittests and ash_unittests. Interesting things of note: . Had to remove DCHECKs for GetRootWindowForDisplayId() as some tests call with invalid. . Had to rename class in mus named WindowManagerTest as it conflicts with test with same name in ash. BUG= 622486 , 631103 , 637853 , 648733 ,695556, 696028 ,698091, 698129 , 698878 , 698887 ,698888,698892,698894, 698895 , 698914 ,699172,699175, 693114 TEST=test changes R=msw@chromium.org Review-Url: https://codereview.chromium.org/2739703003 Cr-Commit-Position: refs/heads/master@{#455272} [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/BUILD.gn [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/mus/bridge/wm_shell_mus.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/mus/window_manager_unittest.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/test/ash_test_base.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/wm/overview/window_selector_unittest.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/wm/panels/panel_layout_manager_unittest.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/wm/panels/panel_window_resizer_unittest.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/wm/system_modal_container_layout_manager_unittest.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/wm/window_cycle_controller_unittest.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/wm/window_manager_unittest.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/wm/window_modality_controller_unittest.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/wm/window_state_unittest.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/wm/window_util_unittest.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/wm/workspace/workspace_event_handler_unittest.cc [modify] https://crrev.com/ea37af717f1569dd93282e09b6fb49ff22b771f8/ash/wm/workspace/workspace_window_resizer_unittest.cc
Obsolete, metrics seem to be working fine now.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5 commit d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5 Author: James Cook <jamescook@chromium.org> Date: Fri Dec 08 00:53:39 2017 cros: Clean up mash early-exits in ash_unittests --mash Our policy for mash is to disable via filter file unless the test is never expected to work under mash. Also re-enable a bunch of tests that now pass under mash, likely due to the display manager work done several months ago. Bug: 695556, 696752 , 698024 , 696006 , 581462 , 698129 , 726838 Test: ash_unittests for classic, --mus and --mash Change-Id: Ia192d77b107c2a9f93ab4e0416b2f4fdb63b3f8d Reviewed-on: https://chromium-review.googlesource.com/815874 Commit-Queue: James Cook <jamescook@chromium.org> Reviewed-by: Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#522655} [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/app_list/app_list_presenter_delegate_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/autoclick/autoclick_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/display/display_configuration_controller_unittest.cc [delete] https://crrev.com/ef9ad27b1a663fa6e434fb32a03ca3275ea79134/ash/mus/window_manager_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/root_window_controller_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/rotator/screen_rotation_animator_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/screen_util_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/shelf/shelf_window_watcher_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/shell_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/sticky_keys/sticky_keys_overlay_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/system/overview/overview_button_tray_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/system/web_notification/ash_popup_alignment_delegate_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/system/web_notification/web_notification_tray_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/tooltips/tooltip_controller_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/wallpaper/wallpaper_controller_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/window_manager_common_unittests.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/wm/panels/panel_layout_manager_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/wm/window_manager_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/wm/workspace/workspace_event_handler_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/ash/wm/workspace/workspace_layout_manager_unittest.cc [modify] https://crrev.com/d7ea4d3d17ed7caa15537e0fb80ae09fe6b2e4d5/testing/buildbot/filters/ash_unittests_mash.filter
Comment 1 by bugdroid1@chromium.org
, Mar 3 2017