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

Unified Diff: LayoutTests/fullscreen/full-screen-iframe-allowed-video.html

Issue 187193002: Fix an issue that subtitle is not shown for native fullscreen video that is inside an iframe (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: addressing comments Created 6 years, 10 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: LayoutTests/fullscreen/full-screen-iframe-allowed-video.html
diff --git a/LayoutTests/fullscreen/full-screen-iframe-allowed-video.html b/LayoutTests/fullscreen/full-screen-iframe-allowed-video.html
new file mode 100644
index 0000000000000000000000000000000000000000..9a888c2132638240f6cb68f0ea440ec01fa94b42
--- /dev/null
+++ b/LayoutTests/fullscreen/full-screen-iframe-allowed-video.html
@@ -0,0 +1,31 @@
+<!DOCTYPE html>
+<script>
+ if (window.internals)
+ runPixelTests = internals.runtimeFlags.overlayFullscreenVideoEnabled;
+</script>
+<script src="full-screen-test.js"></script>
+<script>
+ var initialized = false;
+ window.onmessage = function(messageEvent) {
+ if (!initialized) {
+ // video has been loaded and messaged us.
+ // Send click to center of iframe.
+ iframe = document.getElementById("frame");
+ x = iframe.offsetLeft + iframe.offsetWidth / 2;
+ y = iframe.offsetTop + iframe.offsetHeight / 2;
+ if (window.eventSender) {
+ eventSender.mouseMoveTo(x, y);
+ eventSender.mouseDown();
+ eventSender.mouseUp();
+ }
+ initialized = true;
+ }
+ else {
esprehn 2014/03/05 04:36:03 else goes on the same line as the brace
qinmin 2014/03/05 18:45:25 Done.
+ consoleWrite(messageEvent.data);
+ endTest();
+ }
+ }
+</script>
+Video inside an iframe is allowed to enter fullscreen</p>
+<iframe id="frame" src="resources/video.html" width="320" height="240" allowfullscreen></iframe>
+

Powered by Google App Engine
This is Rietveld 408576698