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

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/fullscreen/api/resources/attempt-full-screen.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/attempt-full-screen.html
diff --git a/third_party/WebKit/LayoutTests/external/wpt/fullscreen/api/resources/attempt-full-screen.html b/third_party/WebKit/LayoutTests/external/wpt/fullscreen/api/resources/attempt-full-screen.html
new file mode 100644
index 0000000000000000000000000000000000000000..a5ddc5f3e5dac14b68e2736087ce09c928c6367a
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/external/wpt/fullscreen/api/resources/attempt-full-screen.html
@@ -0,0 +1,24 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>IFrame fullscreenEnabled attribute and success reporter</title>
+<body>
+<script>
+reportFullscreenEnabled = (frame, success) => {
+ return () => {
+ parent.postMessage({"report": {
+ "api": "fullscreen",
+ "result": [document.fullscreenEnabled, success],
+ "frame": frame
+ }}, "*");
+ };
+};
+
+window.addEventListener('message', (e) => {
+ if (e.data.action == "report") {
+ var frame = e.data.frame;
+ document.onfullscreenchange = reportFullscreenEnabled(frame, true);
+ document.onfullscreenerror = reportFullscreenEnabled(frame, false);
+ document.body.requestFullscreen();
foolip 2017/08/15 09:44:22 This request ought to always fail since it's not i
iclelland 2017/08/16 13:31:55 Interesting... this is triggered by postmessage fr
iclelland 2017/08/16 14:20:57 And this is done; I've split the test into two, to
foolip 2017/08/16 21:43:14 Ah, so I guess it works because there was a user g
+ }
+});
+</script>

Powered by Google App Engine
This is Rietveld 408576698