Make video.webkitSupportsFullscreen an alias of document.fullscreenEnabled

LayoutTests/media/video-controls-fullscreen.js

 "use strict";
 
+function hasFullscreenButton(element)
+{
+    var size = mediaControlsButtonDimensions(element, "fullscreen-button");
+    return size[0] > 0 && size[1] > 0;
+}
+
 function fullscreen_test(controller)
 {
     async_test(function(t)
     {
         var v1 = document.createElement("video");
         var v2 = document.createElement("video");
         v1.controls = v2.controls = true;
         v1.controller = v2.controller = controller;
         v1.src = findMediaFile("video", "content/test");
         v2.src = findMediaFile("audio", "content/test");
         document.body.appendChild(v1);
         document.body.appendChild(v2);
 
         // load event fires when both video elements are ready
         window.addEventListener("load", t.step_func(function()
         {
-            // no fullscreen button for a video element with no video track
-            assert_button_hidden(v2);
+            // the fullscreen button is shown only for the video element with a
+            // video track
+            assert_true(hasFullscreenButton(v1));
+            assert_false(hasFullscreenButton(v2));
 
             // click the fullscreen button
             var coords = mediaControlsButtonCoordinates(v1, "fullscreen-button");
             eventSender.mouseMoveTo(coords[0], coords[1]);
             eventSender.mouseDown();
             eventSender.mouseUp();
             // wait for the fullscreenchange event
         }));
 
-        v1.addEventListener("webkitfullscreenchange", t.step_func(function()
+        v1.addEventListener("webkitfullscreenchange", t.step_func_done());
+        v2.addEventListener("webkitfullscreenchange", t.unreached_func());
+    });
+}
+
+function fullscreen_iframe_test()
+{
+    async_test(function(t)
+    {
+        var iframe = document.querySelector("iframe");
+        var doc = iframe.contentDocument;
+        var v = doc.createElement("video");
+        v.controls = true;
+        v.src = findMediaFile("video", "content/test");
+        doc.body.appendChild(v);
+
+        v.addEventListener("loadeddata", t.step_func_done(function()
         {
-            t.done();
-        }));
-
-        v2.addEventListener("webkitfullscreenchange", t.step_func(function()
-        {
-            assert_unreached();
+            // the fullscreen button is shown only if the iframe has the
+            // allowfullscreen attribute

[falken, 2015/07/22 23:14:56]

Would this comment be better as the third argument to assert_equals?

[philipj_slow, 2015/07/23 07:02:55]

Sure, I can do that.

+            assert_equals(hasFullscreenButton(v), iframe.allowFullscreen);
         }));
     });
 }
 
 function fullscreen_not_supported_test()
 {
     async_test(function(t)
     {
         var v = document.createElement("video");
         v.controls = true;
         v.src = findMediaFile("video", "content/test");
         document.body.appendChild(v);
 
-        // load event fires when video elements is ready
-        window.addEventListener("load", t.step_func(function()
+        v.addEventListener("loadeddata", t.step_func_done(function()

[falken, 2015/07/22 23:14:56]

why does load become loadeddata?

[philipj_slow, 2015/07/23 07:02:55]

The load event on window doesn't work in the iframe case, so I changed this for
consistency. The window load event is most useful to avoid waiting for two
loadeddata events in the first test.

         {
             // no fullscreen button for a video element when fullscreen is not
             // supported
-            assert_button_hidden(v);
-            t.done();
+            assert_false(hasFullscreenButton(v));

[falken, 2015/07/22 23:14:56]

ditto

         }));
     });
 }
-
-function assert_button_hidden(elm)
-{
-    assert_array_equals(mediaControlsButtonDimensions(elm, "fullscreen-button"), [0, 0]);
-}