Commit 100db146 authored by Marijn Kruisselbrink's avatar Marijn Kruisselbrink Committed by Commit Bot

Disable more flaky SmsBrowserTests.

Tbr: juncai@chromium.org
Bug: 997549
Change-Id: I72c5b6949408d03b0716ab0223ce10654f11bf3f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1771553Reviewed-by: default avatarMarijn Kruisselbrink <mek@chromium.org>
Reviewed-by: default avatarJun Cai <juncai@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690398}
parent 4737caa5
...@@ -95,7 +95,8 @@ class SmsBrowserTest : public ContentBrowserTest { ...@@ -95,7 +95,8 @@ class SmsBrowserTest : public ContentBrowserTest {
} // namespace } // namespace
IN_PROC_BROWSER_TEST_F(SmsBrowserTest, Receive) { // Flaky. crbug.com/997549
IN_PROC_BROWSER_TEST_F(SmsBrowserTest, DISABLED_Receive) {
GURL url = GetTestUrl(nullptr, "simple_page.html"); GURL url = GetTestUrl(nullptr, "simple_page.html");
NavigateToURL(shell(), url); NavigateToURL(shell(), url);
...@@ -173,7 +174,8 @@ IN_PROC_BROWSER_TEST_F(SmsBrowserTest, DISABLED_AtMostOnePendingSmsRequest) { ...@@ -173,7 +174,8 @@ IN_PROC_BROWSER_TEST_F(SmsBrowserTest, DISABLED_AtMostOnePendingSmsRequest) {
ExpectOutcomeUKM(url, blink::SMSReceiverOutcome::kSuccess); ExpectOutcomeUKM(url, blink::SMSReceiverOutcome::kSuccess);
} }
IN_PROC_BROWSER_TEST_F(SmsBrowserTest, Reload) { // Flaky. crbug.com/997549
IN_PROC_BROWSER_TEST_F(SmsBrowserTest, DISABLED_Reload) {
GURL url = GetTestUrl(nullptr, "simple_page.html"); GURL url = GetTestUrl(nullptr, "simple_page.html");
NavigateToURL(shell(), url); NavigateToURL(shell(), url);
...@@ -250,7 +252,8 @@ IN_PROC_BROWSER_TEST_F(SmsBrowserTest, Close) { ...@@ -250,7 +252,8 @@ IN_PROC_BROWSER_TEST_F(SmsBrowserTest, Close) {
ExpectNoOutcomeUKM(); ExpectNoOutcomeUKM();
} }
IN_PROC_BROWSER_TEST_F(SmsBrowserTest, TwoTabsSameOrigin) { // Flaky. crbug.com/997549
IN_PROC_BROWSER_TEST_F(SmsBrowserTest, DISABLED_TwoTabsSameOrigin) {
auto* provider = new NiceMock<MockSmsProvider>(); auto* provider = new NiceMock<MockSmsProvider>();
BrowserMainLoop::GetInstance()->SetSmsProviderForTesting( BrowserMainLoop::GetInstance()->SetSmsProviderForTesting(
base::WrapUnique(provider)); base::WrapUnique(provider));
...@@ -348,7 +351,8 @@ IN_PROC_BROWSER_TEST_F(SmsBrowserTest, TwoTabsSameOrigin) { ...@@ -348,7 +351,8 @@ IN_PROC_BROWSER_TEST_F(SmsBrowserTest, TwoTabsSameOrigin) {
ExpectOutcomeUKM(url, blink::SMSReceiverOutcome::kSuccess); ExpectOutcomeUKM(url, blink::SMSReceiverOutcome::kSuccess);
} }
IN_PROC_BROWSER_TEST_F(SmsBrowserTest, TwoTabsDifferentOrigin) { // Flaky. crbug.com/997549
IN_PROC_BROWSER_TEST_F(SmsBrowserTest, DISABLED_TwoTabsDifferentOrigin) {
auto* provider = new NiceMock<MockSmsProvider>(); auto* provider = new NiceMock<MockSmsProvider>();
BrowserMainLoop::GetInstance()->SetSmsProviderForTesting( BrowserMainLoop::GetInstance()->SetSmsProviderForTesting(
base::WrapUnique(provider)); base::WrapUnique(provider));
...@@ -459,7 +463,8 @@ IN_PROC_BROWSER_TEST_F(SmsBrowserTest, SmsReceivedAfterTabIsClosed) { ...@@ -459,7 +463,8 @@ IN_PROC_BROWSER_TEST_F(SmsBrowserTest, SmsReceivedAfterTabIsClosed) {
ExpectNoOutcomeUKM(); ExpectNoOutcomeUKM();
} }
IN_PROC_BROWSER_TEST_F(SmsBrowserTest, Cancels) { // Flaky. crbug.com/997549
IN_PROC_BROWSER_TEST_F(SmsBrowserTest, DISABLED_Cancels) {
GURL url = GetTestUrl(nullptr, "simple_page.html"); GURL url = GetTestUrl(nullptr, "simple_page.html");
NavigateToURL(shell(), url); NavigateToURL(shell(), url);
......
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