Attempt to fix timeout failure of test plugins/navigator-plugin-crash.html in WebKit1
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 12 May 2017 22:43:22 +0000 (22:43 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 12 May 2017 22:43:22 +0000 (22:43 +0000)
following <http://trac.webkit.org/changeset/216789>
(https://bugs.webkit.org/show_bug.cgi?id=171712)

On WebKit1 we need to explicitly allow the test tool to open windows by calling
testRunner.setCanOpenWindows(). We should also call testRunner.setCloseRemainingWindowsWhenComplete(true)
to have the test tool close any open windows after running the test.

It is also good practice to explicitly disable the popup blocker by calling
testRunner.setPopupBlockingEnabled(true);.

* plugins/navigator-plugin-crash.html:

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

LayoutTests/ChangeLog
LayoutTests/plugins/navigator-plugin-crash.html

index 0b4e8bd..a3fa557 100644 (file)
@@ -1,3 +1,18 @@
+2017-05-12  Daniel Bates  <dabates@apple.com>
+
+        Attempt to fix timeout failure of test plugins/navigator-plugin-crash.html in WebKit1
+        following <http://trac.webkit.org/changeset/216789>
+        (https://bugs.webkit.org/show_bug.cgi?id=171712)
+
+        On WebKit1 we need to explicitly allow the test tool to open windows by calling
+        testRunner.setCanOpenWindows(). We should also call testRunner.setCloseRemainingWindowsWhenComplete(true)
+        to have the test tool close any open windows after running the test.
+
+        It is also good practice to explicitly disable the popup blocker by calling
+        testRunner.setPopupBlockingEnabled(true);.
+
+        * plugins/navigator-plugin-crash.html:
+
 2017-05-11  Jiewen Tan  <jiewen_tan@apple.com>
 
         Check existence of a page before accessing its plugins
index 8056f95..dd5544b 100644 (file)
@@ -7,6 +7,9 @@
     <script>
     if (window.testRunner) {
         testRunner.dumpAsText();
+        testRunner.setCanOpenWindows();
+        testRunner.setPopupBlockingEnabled(true);
+        testRunner.setCloseRemainingWindowsWhenComplete(true);
         testRunner.waitUntilDone();
     }