Commit ec4c5a3b authored by Owen Min's avatar Owen Min Committed by Chromium LUCI CQ

Try to resolve flaky ExtensionRequestNotificationTest.NoExtension

The test is timeout on trybots which is likely caused by the
RunUntilIdle() function which is put here just in case. Trying to solve
the timeout issue by removing the function.

Bug: 1137094
Change-Id: I4e48b937901ffa23f2daff6c41050eaa7b9f7f5f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2615324Reviewed-by: default avatarJulian Pastarmov <pastarmovj@chromium.org>
Commit-Queue: Owen Min <zmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#841482}
parent c5db8757
...@@ -81,7 +81,6 @@ TEST_P(ExtensionRequestNotificationTest, NoExtension) { ...@@ -81,7 +81,6 @@ TEST_P(ExtensionRequestNotificationTest, NoExtension) {
#else #else
request_notification.Show(base::BindOnce(&OnNotificationClosed, false)); request_notification.Show(base::BindOnce(&OnNotificationClosed, false));
#endif #endif
task_environment()->RunUntilIdle();
EXPECT_FALSE(GetNotification().has_value()); EXPECT_FALSE(GetNotification().has_value());
} }
......
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