Commit fb77bf14 authored by Yi Gu's avatar Yi Gu Committed by Commit Bot

[WebOTP] Tentatively re-enable SmsBrowserTests

There were 3 tests disabled in Feb due to flakiness (RunLoop timeout).
This patch re-enables one of them to verify whether the flakiness has
been fixed by recent changes and gather up-to-date failure logs if not.

Bug: 1052385
Change-Id: Ib9dcabf769a4ff03d31e1ec1601c69717f590040
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302409Reviewed-by: default avatarSam Goto <goto@chromium.org>
Commit-Queue: Yi Gu <yigu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#789789}
parent 1c6c2396
......@@ -228,9 +228,7 @@ IN_PROC_BROWSER_TEST_F(SmsBrowserTest, AtMostOneSmsRequestPerOrigin) {
ExpectOutcomeUKM(url, blink::SMSReceiverOutcome::kSuccess);
}
// Disabled test: https://crbug.com/1052385
IN_PROC_BROWSER_TEST_F(SmsBrowserTest,
DISABLED_AtMostOneSmsRequestPerOriginPerTab) {
IN_PROC_BROWSER_TEST_F(SmsBrowserTest, AtMostOneSmsRequestPerOriginPerTab) {
auto provider = std::make_unique<MockSmsProvider>();
MockSmsProvider* mock_provider_ptr = provider.get();
BrowserMainLoop::GetInstance()->SetSmsProviderForTesting(std::move(provider));
......
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