Changeset 66695 in webkit

Timestamp:

Sep 2, 2010 5:01:28 PM (14 years ago)

Author:

commit-queue@webkit.org

Message:

2010-09-02 James Robinson <​jamesr@chromium.org>

Reviewed by Darin Adler.
​https://bugs.webkit.org/show_bug.cgi?id=37830

Fix an ordering issue in this test. The isSuccessfullyParsed
function is not defined until the onload event, so avoid running
the script body until this point.

• fast/events/touch/script-tests/send-oncancel-event.js: (window.onload):

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• fast/events/touch/script-tests/send-oncancel-event.js (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2010-09-02  James Robinson  <jamesr@chromium.org>
+
+        Reviewed by Darin Adler.
+        https://bugs.webkit.org/show_bug.cgi?id=37830
+
+        Fix an ordering issue in this test.  The isSuccessfullyParsed
+        function is not defined until the onload event, so avoid running
+        the script body until this point.
+
+        * fast/events/touch/script-tests/send-oncancel-event.js:
+        (window.onload):
+
 2010-09-02  Tony Chang  <tony@chromium.org>
 

trunk/LayoutTests/fast/events/touch/script-tests/send-oncancel-event.js

@@ -21 +21 @@
     window.layoutTestController.waitUntilDone();
 
-if (window.eventSender) {
-    document.addEventListener("touchcancel", touchcancelHandler, false);
-    eventSender.addTouchPoint(touchX, touchY);
-    eventSender.touchStart();
-    eventSender.cancelTouchPoint(0);
-    eventSender.touchCancel();
-} else
-    debug("This test requires DumpRenderTree.");
+window.onload = function() {
+    if (window.eventSender) {
+        document.addEventListener("touchcancel", touchcancelHandler, false);
+        eventSender.addTouchPoint(touchX, touchY);
+        eventSender.touchStart();
+        eventSender.cancelTouchPoint(0);
+        eventSender.touchCancel();
+    } else
+        debug("This test requires DumpRenderTree.");
+}
 
 var successfullyParsed = true;