Commit 77bda58d authored by David Van Cleve's avatar David Van Cleve Committed by Chromium LUCI CQ

[Sheriff] Disable SessionRestoreTest.NormalAndPopup

It seems like this test is pretty genuinely cross-platform flaky, so this CL moves it from conditionally disabled to disabled disabled.

Tbr: chrisha@chromium.org
Bug: 1158715, 1154345, 1166756
Change-Id: I9b0b71e3645ac272d907a6552ac7bcd4e8faff27
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2626531Reviewed-by: default avatarDavid Van Cleve <davidvc@chromium.org>
Commit-Queue: David Van Cleve <davidvc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#843729}
parent dbacc14f
......@@ -524,14 +524,10 @@ IN_PROC_BROWSER_TEST_F(SessionRestoreTest, MaximizedApps) {
#endif // BUILDFLAG(IS_CHROMEOS_ASH)
// Creates a tabbed browser and popup and makes sure we restore both.
#if defined(OS_MAC)
// Disabled for mac-arm64 bot stabilization: https://crbug.com/1154345
// Also disabled for Mac flakiness in general: https://crbug.com/1158715
#define MAYBE_NormalAndPopup DISABLED_NormalAndPopup
#else
#define MAYBE_NormalAndPopup NormalAndPopup
#endif
IN_PROC_BROWSER_TEST_F(SessionRestoreTest, MAYBE_NormalAndPopup) {
// Also disabled for cross-platform flakiness in general: https://crbug.com/1166756
IN_PROC_BROWSER_TEST_F(SessionRestoreTest, DISABLED_NormalAndPopup) {
// Open a popup.
Browser* popup = CreateBrowserForPopup(browser()->profile());
ASSERT_EQ(2u, active_browser_list_->size());
......
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