Commit a5c7d422 authored by Alan Cutter's avatar Alan Cutter Committed by Commit Bot

Disable WebAppNonClientFrameViewAshTest.BrowserCommandFocusToolbarGeolocation on Chrome OS debug

This test started flakily crashing after: https://chromium-review.googlesource.com/c/chromium/src/+/2090877
First failing build: https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-dbg/17501

TBR=pkasting@chromium.org

Bug: 1060421
Change-Id: Ib24e672b8757992c0d10bc3e7b745d3cb8e79f76
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2100172Reviewed-by: default avatarAlan Cutter <alancutter@chromium.org>
Commit-Queue: Alan Cutter <alancutter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#749579}
parent 51efe83e
......@@ -1066,10 +1066,18 @@ IN_PROC_BROWSER_TEST_P(WebAppNonClientFrameViewAshTest,
EXPECT_TRUE(web_app_menu_button_->HasFocus());
}
// TODO(): Flaky crash on Chrome OS debug.
#if defined(OS_CHROMEOS) && !defined(NDEBUG)
#define MAYBE_BrowserCommandFocusToolbarGeolocation \
DISABLED_BrowserCommandFocusToolbarGeolocation
#else
#define MAYBE_BrowserCommandFocusToolbarGeolocation \
BrowserCommandFocusToolbarGeolocation
#endif
// Tests that the focus toolbar command focuses content settings icons before
// the app menu button when present in web-app windows.
IN_PROC_BROWSER_TEST_P(WebAppNonClientFrameViewAshTest,
BrowserCommandFocusToolbarGeolocation) {
MAYBE_BrowserCommandFocusToolbarGeolocation) {
SetUpWebApp();
ContentSettingImageView* geolocation_icon = GrantGeolocationPermission();
......
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