Commit 227c6776 authored by Jonathan Ross's avatar Jonathan Ross Committed by Commit Bot

Disable Flaky WebContentsImplBrowserTest NotifyFullscreenAcquired_Navigate

The test WebContentsImplBrowserTest NotifyFullscreenAcquired_Navigate Flakes
has been flaking on Android for some time. The owner of the test is currently OOO.
So I'm disabling it on Android.

TBR=boliu@chromium.org
TEST=WebContentsImplBrowserTest NotifyFullscreenAcquired_Navigate

Bug: 855018
Change-Id: I1e3f71e696e6186663ee162a81a1f3bed5ad80c5
Reviewed-on: https://chromium-review.googlesource.com/1146261
Commit-Queue: Jonathan Ross <jonross@chromium.org>
Reviewed-by: default avatarJonathan Ross <jonross@chromium.org>
Cr-Commit-Position: refs/heads/master@{#577164}
parent 5cd9ed31
...@@ -2630,8 +2630,15 @@ IN_PROC_BROWSER_TEST_F(WebContentsImplBrowserTest, NotifyFullscreenAcquired) { ...@@ -2630,8 +2630,15 @@ IN_PROC_BROWSER_TEST_F(WebContentsImplBrowserTest, NotifyFullscreenAcquired) {
} }
} }
// TODO(beccahughes@): This test is flaking on Android. https://crbug.com/855018
#if defined(OS_ANDROID)
#define MAYBE_NotifyFullscreenAcquired_Navigate \
DISABLED_NotifyFullscreenAcquired_Navigate
#else
#define MAYBE_NotifyFullscreenAcquired_Navigate NotifyFullscreenAcquired_Navigate
#endif
IN_PROC_BROWSER_TEST_F(WebContentsImplBrowserTest, IN_PROC_BROWSER_TEST_F(WebContentsImplBrowserTest,
NotifyFullscreenAcquired_Navigate) { MAYBE_NotifyFullscreenAcquired_Navigate) {
ASSERT_TRUE(embedded_test_server()->Start()); ASSERT_TRUE(embedded_test_server()->Start());
WebContentsImpl* web_contents = WebContentsImpl* web_contents =
static_cast<WebContentsImpl*>(shell()->web_contents()); static_cast<WebContentsImpl*>(shell()->web_contents());
......
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