Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1660)

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/fullscreen/api/resources/report-full-screen-enabled.html

Issue 2898503002: Reenable feature policy control over fullscreen (Closed)
Patch Set: Reinstate original (same-origin) wpt tests with failing expectations Created 3 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/external/wpt/fullscreen/api/resources/report-full-screen-enabled.html
diff --git a/third_party/WebKit/LayoutTests/external/wpt/fullscreen/api/resources/report-full-screen-enabled.html b/third_party/WebKit/LayoutTests/external/wpt/fullscreen/api/resources/report-full-screen-enabled.html
new file mode 100644
index 0000000000000000000000000000000000000000..0727d60ee396234f514f946c76ceff85be5f1d44
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/external/wpt/fullscreen/api/resources/report-full-screen-enabled.html
@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>IFrame fullscreenEnabled attribute reporter</title>
+<body>
+<script>
+reportFullscreenEnabled = (frame) => {
foolip 2017/08/15 09:44:22 I think this can be simplified. Per http://softwar
iclelland 2017/08/16 13:31:55 That's interesting... If the frame name is availab
+ parent.postMessage({"report": {
+ "api": "fullscreen",
+ "enabled": document.webkitFullscreenEnabled,
foolip 2017/08/15 09:44:22 Use document.fullscreenEnabled in wpt.
iclelland 2017/08/16 13:31:55 Thanks, done.
+ "frame": frame
+ }}, "*");
+}
+
+window.addEventListener('message', (e) => {
+ if (e.data.action == "report") reportFullscreenEnabled(e.data.frame);
+});
+</script>

Powered by Google App Engine
This is Rietveld 408576698