fullscreen/full-screen-plugin.html is very flaky on Yosemite WK2
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 8 Feb 2015 05:24:25 +0000 (05:24 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 8 Feb 2015 05:24:25 +0000 (05:24 +0000)
https://bugs.webkit.org/show_bug.cgi?id=141364

Reviewed by Zalan Bujtas.

Speculative fix. Make sure that the plug-in has loaded before using it.

* fullscreen/full-screen-plugin.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179794 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/fullscreen/full-screen-plugin.html

index ff0b4c1..7e9e188 100644 (file)
@@ -1,3 +1,14 @@
+2015-02-07  Alexey Proskuryakov  <ap@apple.com>
+
+        fullscreen/full-screen-plugin.html is very flaky on Yosemite WK2
+        https://bugs.webkit.org/show_bug.cgi?id=141364
+
+        Reviewed by Zalan Bujtas.
+
+        Speculative fix. Make sure that the plug-in has loaded before using it.
+
+        * fullscreen/full-screen-plugin.html:
+
 2015-02-07  Tim Horton  <timothy_horton@apple.com>
 
         Add some dictionary lookup tests
index 5f71d6b..43c626b 100644 (file)
@@ -6,12 +6,12 @@ if (window.testRunner) {
 </script>
 <div>
     <div>Test that plugin doesn't restart when taking it to full screen and back.</div>
-    <div id="result">Test didn't run'</div>
+    <div id="result">Test didn't run</div>
     <embed id="plugin" type="application/x-webkit-test-netscape" width="200" height="200"></embed>
 </div>
 
 <script>
-if ("webkitRequestFullScreen" in Element.prototype) {
+onload = function() {
     var callback;
     var fullscreenChanged = function(event)
     {
@@ -23,6 +23,7 @@ if ("webkitRequestFullScreen" in Element.prototype) {
     document.body.offsetTop;
     var plugin = document.getElementById('plugin');
     var testObject = plugin.testObject;
+    testObject.property = 'foo'; // Verify that setting a property works before going fullscreen.
 
     var spanEnteredFullScreen = function(event) {
         setTimeout(function () {