Skip to content

Commit

Permalink
Disable FullscreenElementInMultipleSubframes
Browse files Browse the repository at this point in the history
Disables
SitePerProcessInteractiveBrowserTest.FullscreenElementInMultipleSubframes
on Chrome OS and Mac.

BUG=756338
NOTRY=true
TBR=alexmos

Change-Id: I3ca2bc9e06d32adc1cad056fe11fa07ee5545078
Reviewed-on: https://chromium-review.googlesource.com/618329
Reviewed-by: Max Morin <maxmorin@chromium.org>
Commit-Queue: Max Morin <maxmorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#495107}
  • Loading branch information
Max Morin authored and Commit Bot committed Aug 17, 2017
1 parent 13288b3 commit eb03356
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion chrome/browser/site_per_process_interactive_browsertest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
#include "base/strings/string_number_conversions.h"
#include "base/test/scoped_feature_list.h"
#include "base/test/test_timeouts.h"
#include "build/build_config.h"
#include "chrome/browser/password_manager/chrome_password_manager_client.h"
#include "chrome/browser/renderer_context_menu/render_view_context_menu_browsertest_util.h"
#include "chrome/browser/ui/browser.h"
Expand Down Expand Up @@ -864,8 +865,15 @@ IN_PROC_BROWSER_TEST_F(SitePerProcessInteractiveBrowserTest,
// The test also exits fullscreen by simulating pressing ESC rather than using
// document.webkitExitFullscreen(), which tests the browser-initiated
// fullscreen exit path.
#if defined(OS_CHROMEOS) || defined(OS_MACOSX)
#define MAYBE_FullscreenElementInMultipleSubframes \
DISABLED_FullscreenElementInMultipleSubframes
#else
#define MAYBE_FullscreenElementInMultipleSubframes \
FullscreenElementInMultipleSubframes
#endif
IN_PROC_BROWSER_TEST_F(SitePerProcessInteractiveBrowserTest,
FullscreenElementInMultipleSubframes) {
MAYBE_FullscreenElementInMultipleSubframes) {
// Allow fullscreen in all iframes descending to |c_middle|.
GURL main_url(embedded_test_server()->GetURL(
"a.com",
Expand Down

0 comments on commit eb03356

Please sign in to comment.