Commit adee9861 authored by Adithya Srinivasan's avatar Adithya Srinivasan Committed by Commit Bot

Sheriff: Disable AppListController.CloseNotificationWithAppListShown

Bug: 1120501
Change-Id: Ica8546b86353d86a5001b3d7bad161d8ee0e68f5
Tbr: calamity@chromium.org
No-Try: True
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401850Reviewed-by: default avatarAdithya Srinivasan <adithyas@chromium.org>
Commit-Queue: Adithya Srinivasan <adithyas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#805430}
parent 4255ec36
......@@ -461,9 +461,18 @@ TEST_F(AppListControllerImplTest, CheckAppListViewBoundsWhenDismissVKeyboard) {
GetAppListViewNativeWindow()->bounds());
}
#if defined(ADDRESS_SANITIZER)
#define MAYBE_CloseNotificationWithAppListShown \
DISABLED_CloseNotificationWithAppListShown
#else
#define MAYBE_CloseNotificationWithAppListShown \
CloseNotificationWithAppListShown
#endif
// Verifies that closing notification by gesture should not dismiss the AppList.
// (see https://crbug.com/948344)
TEST_F(AppListControllerImplTest, CloseNotificationWithAppListShown) {
// TODO(crbug.com/1120501): Test is flaky on ASAN builds.
TEST_F(AppListControllerImplTest, MAYBE_CloseNotificationWithAppListShown) {
ShowAppListNow();
// Add one notification.
......
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