<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[211573] trunk/LayoutTests</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/211573">211573</a></dd>
<dt>Author</dt> <dd>ddkilzer@apple.com</dd>
<dt>Date</dt> <dd>2017-02-02 10:55:23 -0800 (Thu, 02 Feb 2017)</dd>
</dl>
<h3>Log Message</h3>
<pre>Clean up accessibility & animations tests to stop calling testRunner.waitUntilDone() twice
<https://webkit.org/b/167598>
Reviewed by Alexey Proskuryakov.
* accessibility/frame-disconnect-textmarker-cache-crash.html:
* accessibility/loading-iframe-sends-notification.html:
* accessibility/mac/combobox-activedescendant-notifications.html:
* accessibility/mac/ordered-textmarker-crash.html:
* accessibility/mac/stale-textmarker-crash.html:
* accessibility/notification-listeners.html:
* accessibility/scroll-to-global-point-iframe-nested.html:
* accessibility/scroll-to-global-point-iframe.html:
* accessibility/scroll-to-make-visible-iframe.html:
* accessibility/svg-element-press.html:
- Remove call to testRunner.waitUntilDone() since
window.jsTestIsAsync is set to true, and thus waitUntilDone()
is called from either js-test.js or js-test-post.js.
* animations/animation-delay-changed.html:
* animations/change-keyframes-name.html:
* animations/font-variations/font-variation-settings-order.html:
* animations/font-variations/font-variation-settings-unlike.html:
* animations/font-variations/font-variation-settings.html:
* animations/longhand-timing-function.html:
* animations/transition-and-animation-3.html:
* animations/unanimated-style.html:
- Remove call to testRunner.waitUntilDone() since that's moved
into animation-test-helpers.js.
* animations/resources/animation-test-helpers.js:
(runAnimationTest): Call testRunner.waitUntilDone() when this
script resource is included instead of when runAnimationTest()
is called since runAnimationTest() expects it to be set anyway.
* animations/animation-direction-reverse-fill-mode-hardware.html:
* animations/animation-direction-reverse-fill-mode.html:
* animations/animation-hit-test-transform.html:
* animations/animation-hit-test.html:
* animations/animation-internals-api-multiple-keyframes.html:
* animations/animation-internals-api.html:
* animations/change-transform-style-during-animation.html:
* animations/keyframes-iteration-count-non-integer.html:
- Remove call to testRunner.waitUntilDone() since that now
happens when loading animation-test-helpers.js. These tests
don't currently call runAnimationTest(), but they manage their
own state by calling testRunner.notifyDone() themselves.
* animations/crash-on-removing-animation.html:
* animations/resources/crash-on-removing-animation-window.html: Copied from LayoutTests/animations/crash-on-removing-animation.html.
- Split this test into a main resource and a window resource.
Previously testRunner.waitUntilDone() was being called twice
when js-test-post.js was loaded in the main window and then
loaded a second time when the window was opened. The new
resource file doesn't load either js-test-pre.js or
js-test-post.js, avoiding the duplicate call.</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkLayoutTestsChangeLog">trunk/LayoutTests/ChangeLog</a></li>
<li><a href="#trunkLayoutTestsaccessibilityframedisconnecttextmarkercachecrashhtml">trunk/LayoutTests/accessibility/frame-disconnect-textmarker-cache-crash.html</a></li>
<li><a href="#trunkLayoutTestsaccessibilityloadingiframesendsnotificationhtml">trunk/LayoutTests/accessibility/loading-iframe-sends-notification.html</a></li>
<li><a href="#trunkLayoutTestsaccessibilitymaccomboboxactivedescendantnotificationshtml">trunk/LayoutTests/accessibility/mac/combobox-activedescendant-notifications.html</a></li>
<li><a href="#trunkLayoutTestsaccessibilitymacorderedtextmarkercrashhtml">trunk/LayoutTests/accessibility/mac/ordered-textmarker-crash.html</a></li>
<li><a href="#trunkLayoutTestsaccessibilitymacstaletextmarkercrashhtml">trunk/LayoutTests/accessibility/mac/stale-textmarker-crash.html</a></li>
<li><a href="#trunkLayoutTestsaccessibilitynotificationlistenershtml">trunk/LayoutTests/accessibility/notification-listeners.html</a></li>
<li><a href="#trunkLayoutTestsaccessibilityscrolltoglobalpointiframenestedhtml">trunk/LayoutTests/accessibility/scroll-to-global-point-iframe-nested.html</a></li>
<li><a href="#trunkLayoutTestsaccessibilityscrolltoglobalpointiframehtml">trunk/LayoutTests/accessibility/scroll-to-global-point-iframe.html</a></li>
<li><a href="#trunkLayoutTestsaccessibilityscrolltomakevisibleiframehtml">trunk/LayoutTests/accessibility/scroll-to-make-visible-iframe.html</a></li>
<li><a href="#trunkLayoutTestsaccessibilitysvgelementpresshtml">trunk/LayoutTests/accessibility/svg-element-press.html</a></li>
<li><a href="#trunkLayoutTestsanimationsanimationdelaychangedhtml">trunk/LayoutTests/animations/animation-delay-changed.html</a></li>
<li><a href="#trunkLayoutTestsanimationsanimationdirectionreversefillmodehardwarehtml">trunk/LayoutTests/animations/animation-direction-reverse-fill-mode-hardware.html</a></li>
<li><a href="#trunkLayoutTestsanimationsanimationdirectionreversefillmodehtml">trunk/LayoutTests/animations/animation-direction-reverse-fill-mode.html</a></li>
<li><a href="#trunkLayoutTestsanimationsanimationhittesttransformhtml">trunk/LayoutTests/animations/animation-hit-test-transform.html</a></li>
<li><a href="#trunkLayoutTestsanimationsanimationhittesthtml">trunk/LayoutTests/animations/animation-hit-test.html</a></li>
<li><a href="#trunkLayoutTestsanimationsanimationinternalsapimultiplekeyframeshtml">trunk/LayoutTests/animations/animation-internals-api-multiple-keyframes.html</a></li>
<li><a href="#trunkLayoutTestsanimationsanimationinternalsapihtml">trunk/LayoutTests/animations/animation-internals-api.html</a></li>
<li><a href="#trunkLayoutTestsanimationschangekeyframesnamehtml">trunk/LayoutTests/animations/change-keyframes-name.html</a></li>
<li><a href="#trunkLayoutTestsanimationschangetransformstyleduringanimationhtml">trunk/LayoutTests/animations/change-transform-style-during-animation.html</a></li>
<li><a href="#trunkLayoutTestsanimationscrashonremovinganimationhtml">trunk/LayoutTests/animations/crash-on-removing-animation.html</a></li>
<li><a href="#trunkLayoutTestsanimationsfontvariationsfontvariationsettingsorderhtml">trunk/LayoutTests/animations/font-variations/font-variation-settings-order.html</a></li>
<li><a href="#trunkLayoutTestsanimationsfontvariationsfontvariationsettingsunlikehtml">trunk/LayoutTests/animations/font-variations/font-variation-settings-unlike.html</a></li>
<li><a href="#trunkLayoutTestsanimationsfontvariationsfontvariationsettingshtml">trunk/LayoutTests/animations/font-variations/font-variation-settings.html</a></li>
<li><a href="#trunkLayoutTestsanimationskeyframesiterationcountnonintegerhtml">trunk/LayoutTests/animations/keyframes-iteration-count-non-integer.html</a></li>
<li><a href="#trunkLayoutTestsanimationslonghandtimingfunctionhtml">trunk/LayoutTests/animations/longhand-timing-function.html</a></li>
<li><a href="#trunkLayoutTestsanimationsresourcesanimationtesthelpersjs">trunk/LayoutTests/animations/resources/animation-test-helpers.js</a></li>
<li><a href="#trunkLayoutTestsanimationstransitionandanimation3html">trunk/LayoutTests/animations/transition-and-animation-3.html</a></li>
<li><a href="#trunkLayoutTestsanimationsunanimatedstylehtml">trunk/LayoutTests/animations/unanimated-style.html</a></li>
</ul>
<h3>Added Paths</h3>
<ul>
<li><a href="#trunkLayoutTestsanimationsresourcescrashonremovinganimationwindowhtml">trunk/LayoutTests/animations/resources/crash-on-removing-animation-window.html</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkLayoutTestsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/ChangeLog (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/ChangeLog 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/ChangeLog 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -1,3 +1,62 @@
</span><ins>+2017-02-02 David Kilzer <ddkilzer@apple.com>
+
+ Clean up accessibility & animations tests to stop calling testRunner.waitUntilDone() twice
+ <https://webkit.org/b/167598>
+
+ Reviewed by Alexey Proskuryakov.
+
+ * accessibility/frame-disconnect-textmarker-cache-crash.html:
+ * accessibility/loading-iframe-sends-notification.html:
+ * accessibility/mac/combobox-activedescendant-notifications.html:
+ * accessibility/mac/ordered-textmarker-crash.html:
+ * accessibility/mac/stale-textmarker-crash.html:
+ * accessibility/notification-listeners.html:
+ * accessibility/scroll-to-global-point-iframe-nested.html:
+ * accessibility/scroll-to-global-point-iframe.html:
+ * accessibility/scroll-to-make-visible-iframe.html:
+ * accessibility/svg-element-press.html:
+ - Remove call to testRunner.waitUntilDone() since
+ window.jsTestIsAsync is set to true, and thus waitUntilDone()
+ is called from either js-test.js or js-test-post.js.
+
+ * animations/animation-delay-changed.html:
+ * animations/change-keyframes-name.html:
+ * animations/font-variations/font-variation-settings-order.html:
+ * animations/font-variations/font-variation-settings-unlike.html:
+ * animations/font-variations/font-variation-settings.html:
+ * animations/longhand-timing-function.html:
+ * animations/transition-and-animation-3.html:
+ * animations/unanimated-style.html:
+ - Remove call to testRunner.waitUntilDone() since that's moved
+ into animation-test-helpers.js.
+
+ * animations/resources/animation-test-helpers.js:
+ (runAnimationTest): Call testRunner.waitUntilDone() when this
+ script resource is included instead of when runAnimationTest()
+ is called since runAnimationTest() expects it to be set anyway.
+
+ * animations/animation-direction-reverse-fill-mode-hardware.html:
+ * animations/animation-direction-reverse-fill-mode.html:
+ * animations/animation-hit-test-transform.html:
+ * animations/animation-hit-test.html:
+ * animations/animation-internals-api-multiple-keyframes.html:
+ * animations/animation-internals-api.html:
+ * animations/change-transform-style-during-animation.html:
+ * animations/keyframes-iteration-count-non-integer.html:
+ - Remove call to testRunner.waitUntilDone() since that now
+ happens when loading animation-test-helpers.js. These tests
+ don't currently call runAnimationTest(), but they manage their
+ own state by calling testRunner.notifyDone() themselves.
+
+ * animations/crash-on-removing-animation.html:
+ * animations/resources/crash-on-removing-animation-window.html: Copied from LayoutTests/animations/crash-on-removing-animation.html.
+ - Split this test into a main resource and a window resource.
+ Previously testRunner.waitUntilDone() was being called twice
+ when js-test-post.js was loaded in the main window and then
+ loaded a second time when the window was opened. The new
+ resource file doesn't load either js-test-pre.js or
+ js-test-post.js, avoiding the duplicate call.
+
</ins><span class="cx"> 2017-02-02 Antti Koivisto <antti@apple.com>
</span><span class="cx">
</span><span class="cx"> Column progression wrong after enabling pagination on RTL document
</span></span></pre></div>
<a id="trunkLayoutTestsaccessibilityframedisconnecttextmarkercachecrashhtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/accessibility/frame-disconnect-textmarker-cache-crash.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/accessibility/frame-disconnect-textmarker-cache-crash.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/accessibility/frame-disconnect-textmarker-cache-crash.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -76,8 +76,6 @@
</span><span class="cx">
</span><span class="cx"> if (window.accessibilityController) {
</span><span class="cx"> window.jsTestIsAsync = true;
</span><del>- if (window.testRunner)
- testRunner.waitUntilDone();
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> </script>
</span></span></pre></div>
<a id="trunkLayoutTestsaccessibilityloadingiframesendsnotificationhtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/accessibility/loading-iframe-sends-notification.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/accessibility/loading-iframe-sends-notification.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/accessibility/loading-iframe-sends-notification.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -19,9 +19,6 @@
</span><span class="cx"> <script>
</span><span class="cx"> description("This tests that when an iframe finishes loading, it sends a notification.");
</span><span class="cx">
</span><del>- if (window.testRunner)
- testRunner.waitUntilDone();
-
</del><span class="cx"> window.jsTestIsAsync = true;
</span><span class="cx">
</span><span class="cx"> // Recursively search the entire accessibility tree starting at the given
</span></span></pre></div>
<a id="trunkLayoutTestsaccessibilitymaccomboboxactivedescendantnotificationshtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/accessibility/mac/combobox-activedescendant-notifications.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/accessibility/mac/combobox-activedescendant-notifications.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/accessibility/mac/combobox-activedescendant-notifications.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -20,7 +20,6 @@
</span><span class="cx">
</span><span class="cx"> if (window.testRunner && window.accessibilityController) {
</span><span class="cx"> jsTestIsAsync = true;
</span><del>- testRunner.waitUntilDone();
</del><span class="cx">
</span><span class="cx"> document.getElementById("combo").focus();
</span><span class="cx"> var combo = accessibilityController.accessibleElementById("combo");
</span></span></pre></div>
<a id="trunkLayoutTestsaccessibilitymacorderedtextmarkercrashhtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/accessibility/mac/ordered-textmarker-crash.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/accessibility/mac/ordered-textmarker-crash.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/accessibility/mac/ordered-textmarker-crash.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -16,7 +16,6 @@
</span><span class="cx"> description("This tests that comparing text markers that have no common tree scope won't crash");
</span><span class="cx">
</span><span class="cx"> if (window.accessibilityController && window.testRunner) {
</span><del>- window.testRunner.waitUntilDone();
</del><span class="cx"> window.jsTestIsAsync = true;
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsaccessibilitymacstaletextmarkercrashhtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/accessibility/mac/stale-textmarker-crash.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/accessibility/mac/stale-textmarker-crash.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/accessibility/mac/stale-textmarker-crash.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -18,7 +18,6 @@
</span><span class="cx"> var cachedMarker1 = 0;
</span><span class="cx"> var cachedMarker2 = 0;
</span><span class="cx"> if (window.accessibilityController && window.testRunner) {
</span><del>- window.testRunner.waitUntilDone();
</del><span class="cx"> window.jsTestIsAsync = true;
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsaccessibilitynotificationlistenershtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/accessibility/notification-listeners.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/accessibility/notification-listeners.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/accessibility/notification-listeners.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -16,9 +16,6 @@
</span><span class="cx"> description("This tests that a notification listener on an element only listens to that one element, and that a global notification listener listens to all notifications.");
</span><span class="cx">
</span><span class="cx"> function runTest() {
</span><del>- if (window.testRunner)
- testRunner.waitUntilDone();
-
</del><span class="cx"> window.jsTestIsAsync = true;
</span><span class="cx">
</span><span class="cx"> window.selectNotificationCount = 0;
</span></span></pre></div>
<a id="trunkLayoutTestsaccessibilityscrolltoglobalpointiframenestedhtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/accessibility/scroll-to-global-point-iframe-nested.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/accessibility/scroll-to-global-point-iframe-nested.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/accessibility/scroll-to-global-point-iframe-nested.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -26,9 +26,6 @@
</span><span class="cx"> <script>
</span><span class="cx"> description("Tests that scrolling to move an element to a specific point successfully scrolls both an iframe and a div with overflow.");
</span><span class="cx">
</span><del>-if (window.testRunner)
- testRunner.waitUntilDone();
-
</del><span class="cx"> window.jsTestIsAsync = true;
</span><span class="cx">
</span><span class="cx"> function runTest() {
</span></span></pre></div>
<a id="trunkLayoutTestsaccessibilityscrolltoglobalpointiframehtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/accessibility/scroll-to-global-point-iframe.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/accessibility/scroll-to-global-point-iframe.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/accessibility/scroll-to-global-point-iframe.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -23,9 +23,6 @@
</span><span class="cx"> <script>
</span><span class="cx"> description("Tests that scrolling to move an element to a specific point successfully scrolls an iframe.");
</span><span class="cx">
</span><del>-if (window.testRunner)
- testRunner.waitUntilDone();
-
</del><span class="cx"> window.jsTestIsAsync = true;
</span><span class="cx">
</span><span class="cx"> function runTest() {
</span></span></pre></div>
<a id="trunkLayoutTestsaccessibilityscrolltomakevisibleiframehtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/accessibility/scroll-to-make-visible-iframe.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/accessibility/scroll-to-make-visible-iframe.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/accessibility/scroll-to-make-visible-iframe.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -13,9 +13,6 @@
</span><span class="cx"> <script>
</span><span class="cx"> description("Tests that scrolling to make an element visible successfully scrolls an iframe.");
</span><span class="cx">
</span><del>-if (window.testRunner)
- testRunner.waitUntilDone();
-
</del><span class="cx"> window.jsTestIsAsync = true;
</span><span class="cx">
</span><span class="cx"> function runTest() {
</span></span></pre></div>
<a id="trunkLayoutTestsaccessibilitysvgelementpresshtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/accessibility/svg-element-press.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/accessibility/svg-element-press.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/accessibility/svg-element-press.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -19,6 +19,7 @@
</span><span class="cx"> <script>
</span><span class="cx">
</span><span class="cx"> window.jsTestIsAsync = true;
</span><ins>+
</ins><span class="cx"> description("This tests that AXPress works on SVG elements");
</span><span class="cx">
</span><span class="cx"> function svgClicked() {
</span><span class="lines">@@ -26,9 +27,6 @@
</span><span class="cx"> finishJSTest();
</span><span class="cx"> }
</span><span class="cx">
</span><del>- if (window.testRunner)
- testRunner.waitUntilDone();
-
</del><span class="cx"> if (window.accessibilityController) {
</span><span class="cx"> var svg = accessibilityController.accessibleElementById("svg");
</span><span class="cx"> svg.press();
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsanimationdelaychangedhtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/animation-delay-changed.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/animation-delay-changed.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/animation-delay-changed.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -40,9 +40,6 @@
</span><span class="cx"> </style>
</span><span class="cx"> <script src="resources/animation-test-helpers.js"></script>
</span><span class="cx"> <script>
</span><del>- if (window.testRunner) {
- testRunner.waitUntilDone();
- }
</del><span class="cx"> window.onload = function() {
</span><span class="cx"> document.querySelector('#box1').classList.add('move1');
</span><span class="cx"> // Start the animation on box2 after 500ms
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsanimationdirectionreversefillmodehardwarehtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/animation-direction-reverse-fill-mode-hardware.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/animation-direction-reverse-fill-mode-hardware.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/animation-direction-reverse-fill-mode-hardware.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -43,10 +43,8 @@
</span><span class="cx"> ];
</span><span class="cx"> var result = "";
</span><span class="cx">
</span><del>- if (window.testRunner) {
</del><ins>+ if (window.testRunner)
</ins><span class="cx"> testRunner.dumpAsText();
</span><del>- testRunner.waitUntilDone();
- }
</del><span class="cx">
</span><span class="cx"> function animationEnded(event) {
</span><span class="cx"> if (++animsFinished == numAnims) {
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsanimationdirectionreversefillmodehtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/animation-direction-reverse-fill-mode.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/animation-direction-reverse-fill-mode.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/animation-direction-reverse-fill-mode.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -56,10 +56,8 @@
</span><span class="cx"> ];
</span><span class="cx"> var result = "";
</span><span class="cx">
</span><del>- if (window.testRunner) {
</del><ins>+ if (window.testRunner)
</ins><span class="cx"> testRunner.dumpAsText();
</span><del>- testRunner.waitUntilDone();
- }
</del><span class="cx">
</span><span class="cx"> function animationEnded(event) {
</span><span class="cx"> if (++animsFinished == numAnims) {
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsanimationhittesttransformhtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/animation-hit-test-transform.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/animation-hit-test-transform.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/animation-hit-test-transform.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -74,10 +74,8 @@
</span><span class="cx">
</span><span class="cx"> function startTest()
</span><span class="cx"> {
</span><del>- if (window.testRunner) {
</del><ins>+ if (window.testRunner)
</ins><span class="cx"> testRunner.dumpAsText();
</span><del>- testRunner.waitUntilDone();
- }
</del><span class="cx">
</span><span class="cx"> document.getElementById("target").style.webkitAnimationName = "anim";
</span><span class="cx"> waitForAnimationToStart(document.getElementById('target'), doTest);
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsanimationhittesthtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/animation-hit-test.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/animation-hit-test.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/animation-hit-test.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -74,10 +74,8 @@
</span><span class="cx">
</span><span class="cx"> function startTest()
</span><span class="cx"> {
</span><del>- if (window.testRunner) {
</del><ins>+ if (window.testRunner)
</ins><span class="cx"> testRunner.dumpAsText();
</span><del>- testRunner.waitUntilDone();
- }
</del><span class="cx">
</span><span class="cx"> document.getElementById("target").style.webkitAnimationName = "anim";
</span><span class="cx"> waitForAnimationToStart(document.getElementById('target'), doTest);
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsanimationinternalsapimultiplekeyframeshtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/animation-internals-api-multiple-keyframes.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/animation-internals-api-multiple-keyframes.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/animation-internals-api-multiple-keyframes.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -23,10 +23,8 @@
</span><span class="cx">
</span><span class="cx"> <script src="resources/animation-test-helpers.js"></script>
</span><span class="cx"> <script>
</span><del>- if (window.testRunner) {
- testRunner.waitUntilDone();
- testRunner.dumpAsText();
- }
</del><ins>+ if (window.testRunner)
+ testRunner.dumpAsText();
</ins><span class="cx">
</span><span class="cx"> function animationStarted()
</span><span class="cx"> {
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsanimationinternalsapihtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/animation-internals-api.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/animation-internals-api.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/animation-internals-api.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -25,10 +25,8 @@
</span><span class="cx">
</span><span class="cx"> <script src="resources/animation-test-helpers.js"></script>
</span><span class="cx"> <script>
</span><del>- if (window.testRunner) {
- testRunner.waitUntilDone();
</del><ins>+ if (window.testRunner)
</ins><span class="cx"> testRunner.dumpAsText();
</span><del>- }
</del><span class="cx">
</span><span class="cx"> function animationStarted()
</span><span class="cx"> {
</span></span></pre></div>
<a id="trunkLayoutTestsanimationschangekeyframesnamehtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/change-keyframes-name.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/change-keyframes-name.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/change-keyframes-name.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -59,7 +59,6 @@
</span><span class="cx"> {
</span><span class="cx"> if (window.testRunner) {
</span><span class="cx"> testRunner.dumpAsText();
</span><del>- testRunner.waitUntilDone();
</del><span class="cx"> var box = document.getElementById('box');
</span><span class="cx"> if (internals.pauseAnimationAtTimeOnElement("bar", 0.5, box))
</span><span class="cx"> document.getElementById("pre-result").innerHTML = "FAIL: animation is running";
</span></span></pre></div>
<a id="trunkLayoutTestsanimationschangetransformstyleduringanimationhtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/change-transform-style-during-animation.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/change-transform-style-during-animation.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/change-transform-style-during-animation.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -37,10 +37,8 @@
</span><span class="cx">
</span><span class="cx"> <script src="resources/animation-test-helpers.js"></script>
</span><span class="cx"> <script>
</span><del>- if (window.testRunner) {
</del><ins>+ if (window.testRunner)
</ins><span class="cx"> testRunner.dumpAsText();
</span><del>- testRunner.waitUntilDone();
- }
</del><span class="cx">
</span><span class="cx"> function animationDone()
</span><span class="cx"> {
</span></span></pre></div>
<a id="trunkLayoutTestsanimationscrashonremovinganimationhtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/crash-on-removing-animation.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/crash-on-removing-animation.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/crash-on-removing-animation.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -6,28 +6,10 @@
</span><span class="cx"> var successfullyParsed = false;
</span><span class="cx"> window.jsTestIsAsync = true;
</span><span class="cx">
</span><del>- function tryToCrash() {
- var div = document.getElementById("test");
- div.style.webkitTransform = "translateY(1px)";
- div.style.webkitTransition = "1s ease-in-out";
- setTimeout(function() {
- div.style.webkitTransform = "none";
- div.style.webkitTransition = "none";
- setTimeout(function() {
- window.close();
- window.opener.finishJSTest();
- }, 0);
- }, 0);
- }
-
</del><span class="cx"> function setup() {
</span><span class="cx"> if (window.testRunner) {
</span><del>- if (location.href.indexOf("auxillary") != -1) {
- setTimeout(tryToCrash, 0);
- } else {
- testRunner.setCanOpenWindows();
- window.open("crash-on-removing-animation.html#auxillary");
- }
</del><ins>+ testRunner.setCanOpenWindows();
+ window.open("resources/crash-on-removing-animation-window.html");
</ins><span class="cx"> } else {
</span><span class="cx"> document.write("This test must be run in DRT/WKTR.");
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsfontvariationsfontvariationsettingsorderhtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/font-variations/font-variation-settings-order.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/font-variations/font-variation-settings-order.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/font-variations/font-variation-settings-order.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -35,10 +35,6 @@
</span><span class="cx"> ["TheAnimation", 2.0, "box", "font-variation-settings", "'wdth' 766.6666, 'hght' 666.6666", 5],
</span><span class="cx"> ];
</span><span class="cx">
</span><del>-if (window.testRunner) {
- testRunner.waitUntilDone();
-}
-
</del><span class="cx"> var promises = [];
</span><span class="cx">
</span><span class="cx"> document.fonts.forEach(function(f) {
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsfontvariationsfontvariationsettingsunlikehtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/font-variations/font-variation-settings-unlike.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/font-variations/font-variation-settings-unlike.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/font-variations/font-variation-settings-unlike.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -35,10 +35,6 @@
</span><span class="cx"> ["TheAnimation", 2.0, "box", "font-variation-settings", "normal", 5],
</span><span class="cx"> ];
</span><span class="cx">
</span><del>-if (window.testRunner) {
- testRunner.waitUntilDone();
-}
-
</del><span class="cx"> var promises = [];
</span><span class="cx">
</span><span class="cx"> document.fonts.forEach(function(f) {
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsfontvariationsfontvariationsettingshtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/font-variations/font-variation-settings.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/font-variations/font-variation-settings.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/font-variations/font-variation-settings.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -35,10 +35,6 @@
</span><span class="cx"> ["TheAnimation", 2.0, "box", "font-variation-settings", "'wdth' 733.3333", 5],
</span><span class="cx"> ];
</span><span class="cx">
</span><del>-if (window.testRunner) {
- testRunner.waitUntilDone();
-}
-
</del><span class="cx"> var promises = [];
</span><span class="cx">
</span><span class="cx"> document.fonts.forEach(function(f) {
</span></span></pre></div>
<a id="trunkLayoutTestsanimationskeyframesiterationcountnonintegerhtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/keyframes-iteration-count-non-integer.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/keyframes-iteration-count-non-integer.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/keyframes-iteration-count-non-integer.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -30,10 +30,8 @@
</span><span class="cx"> <script src="resources/animation-test-helpers.js" type="text/javascript" charset="utf-8"></script>
</span><span class="cx"> <script type="text/javascript" charset="utf-8">
</span><span class="cx">
</span><del>- if (window.testRunner) {
</del><ins>+ if (window.testRunner)
</ins><span class="cx"> testRunner.dumpAsText();
</span><del>- testRunner.waitUntilDone();
- }
</del><span class="cx">
</span><span class="cx"> document.addEventListener("webkitAnimationEnd", function(event){
</span><span class="cx"> var result;
</span></span></pre></div>
<a id="trunkLayoutTestsanimationslonghandtimingfunctionhtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/longhand-timing-function.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/longhand-timing-function.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/longhand-timing-function.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -26,14 +26,11 @@
</span><span class="cx"> </style>
</span><span class="cx"> <script src="resources/animation-test-helpers.js" type="text/javascript" charset="utf-8"></script>
</span><span class="cx"> <script type="text/javascript" charset="utf-8">
</span><del>- if (window.testRunner)
- testRunner.waitUntilDone();
-
- const expectedValues = [
</del><ins>+ const expectedValues = [
</ins><span class="cx"> // [animation-name, time, element-id, property, expected-value, tolerance]
</span><span class="cx"> ["move", 0.5, "control", "left", 200, 10],
</span><span class="cx"> ["move", 0.5, "test", "left", 200, 10],
</span><del>- ];
</del><ins>+ ];
</ins><span class="cx">
</span><span class="cx"> function setupTest()
</span><span class="cx"> {
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsresourcesanimationtesthelpersjs"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/resources/animation-test-helpers.js (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/resources/animation-test-helpers.js 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/resources/animation-test-helpers.js 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -533,6 +533,9 @@
</span><span class="cx"> var result = "";
</span><span class="cx"> var hasPauseAnimationAPI;
</span><span class="cx">
</span><ins>+if (window.testRunner)
+ testRunner.waitUntilDone();
+
</ins><span class="cx"> function runAnimationTest(expected, startCallback, event, disablePauseAnimationAPI, doPixelTest, finishCallback)
</span><span class="cx"> {
</span><span class="cx"> hasPauseAnimationAPI = 'internals' in window;
</span><span class="lines">@@ -542,7 +545,6 @@
</span><span class="cx"> if (window.testRunner) {
</span><span class="cx"> if (!doPixelTest)
</span><span class="cx"> testRunner.dumpAsText();
</span><del>- testRunner.waitUntilDone();
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if (!expected)
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsresourcescrashonremovinganimationwindowhtmlfromrev211572trunkLayoutTestsanimationscrashonremovinganimationhtml"></a>
<div class="copfile"><h4>Copied: trunk/LayoutTests/animations/resources/crash-on-removing-animation-window.html (from rev 211572, trunk/LayoutTests/animations/crash-on-removing-animation.html) (0 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/resources/crash-on-removing-animation-window.html (rev 0)
+++ trunk/LayoutTests/animations/resources/crash-on-removing-animation-window.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -0,0 +1,42 @@
</span><ins>+<html>
+
+<head>
+ <script>
+ function tryToCrash() {
+ var div = document.getElementById("test");
+ div.style.webkitTransform = "translateY(1px)";
+ div.style.webkitTransition = "1s ease-in-out";
+ setTimeout(function() {
+ div.style.webkitTransform = "none";
+ div.style.webkitTransition = "none";
+ setTimeout(function() {
+ window.close();
+ window.opener.finishJSTest();
+ }, 0);
+ }, 0);
+ }
+
+ function setup() {
+ if (window.testRunner) {
+ setTimeout(tryToCrash, 0);
+ } else {
+ document.write("This test must be run in DRT/WKTR.");
+ }
+ }
+ </script>
+
+ <style>
+ #test {
+ background-color: blue;
+ width: 10px;
+ height: 10px;
+ }
+ </style>
+<head>
+
+<body onload="setup()">
+ <div id="test"></div>
+ <p>If you are reading this, we managed to avoid crashing!</p>
+</body>
+
+</html>
</ins></span></pre></div>
<a id="trunkLayoutTestsanimationstransitionandanimation3html"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/transition-and-animation-3.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/transition-and-animation-3.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/transition-and-animation-3.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -38,9 +38,6 @@
</span><span class="cx"> <script src="resources/animation-test-helpers.js" type="text/javascript" charset="utf-8"></script>
</span><span class="cx"> <script type="text/javascript" charset="utf-8">
</span><span class="cx">
</span><del>- if (window.testRunner)
- testRunner.waitUntilDone();
-
</del><span class="cx"> const expectedValues = [
</span><span class="cx"> // [animation-name, time, element-id, property, expected-value, tolerance]
</span><span class="cx"> [null, 0.2, "test", "left", 100, 10],
</span></span></pre></div>
<a id="trunkLayoutTestsanimationsunanimatedstylehtml"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/animations/unanimated-style.html (211572 => 211573)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/animations/unanimated-style.html 2017-02-02 18:42:02 UTC (rev 211572)
+++ trunk/LayoutTests/animations/unanimated-style.html 2017-02-02 18:55:23 UTC (rev 211573)
</span><span class="lines">@@ -31,9 +31,6 @@
</span><span class="cx"> <script src="resources/animation-test-helpers.js" type="text/javascript" charset="utf-8"></script>
</span><span class="cx"> <script type="text/javascript" charset="utf-8">
</span><span class="cx">
</span><del>- if (window.testRunner)
- testRunner.waitUntilDone();
-
</del><span class="cx"> const expectedValues = [
</span><span class="cx"> // [animation-name, time, element-id, property, expected-value, tolerance]
</span><span class="cx"> ["move", 0.5, "test", "left", 450, 8],
</span></span></pre>
</div>
</div>
</body>
</html>