Skip to content

Commit

Permalink
Disable SubframeVisibleAfterRenderViewBecomesSwappedOut on Android.
Browse files Browse the repository at this point in the history
Bug: 897709
TBR: alexmos
Change-Id: I9e9a8c1b164b444f839053daa62114d9dbd2504e
Reviewed-on: https://chromium-review.googlesource.com/c/1293589
Reviewed-by: Michael van Ouwerkerk <mvanouwerkerk@chromium.org>
Commit-Queue: Michael van Ouwerkerk <mvanouwerkerk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#601556}
  • Loading branch information
mvano authored and Commit Bot committed Oct 22, 2018
1 parent efe8fd6 commit 80b4e40
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion content/browser/site_per_process_browsertest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -11638,8 +11638,17 @@ IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest,
// Check that when A opens a new window with B which embeds an A subframe, the
// subframe is visible and generates paint events. See
// https://crbug.com/638375.

#if defined(OS_ANDROID)
// Flaky on Android: https://crbug.com/897709
#define MAYBE_SubframeVisibleAfterRenderViewBecomesSwappedOut \
DISABLED_SubframeVisibleAfterRenderViewBecomesSwappedOut
#else
#define MAYBE_SubframeVisibleAfterRenderViewBecomesSwappedOut \
SubframeVisibleAfterRenderViewBecomesSwappedOut
#endif
IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest,
SubframeVisibleAfterRenderViewBecomesSwappedOut) {
MAYBE_SubframeVisibleAfterRenderViewBecomesSwappedOut) {
GURL main_url(embedded_test_server()->GetURL("a.com", "/title1.html"));
EXPECT_TRUE(NavigateToURL(shell(), main_url));

Expand Down

0 comments on commit 80b4e40

Please sign in to comment.