Reviewed by Darin.
authorweinig <weinig@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 5 Feb 2007 02:54:45 +0000 (02:54 +0000)
committerweinig <weinig@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 5 Feb 2007 02:54:45 +0000 (02:54 +0000)
        Patch for http://bugs.webkit.org/show_bug.cgi?id=12562
        Fix test cases that are now spewing console errors that
        they are not testing for

        * fast/events/iframe-object-onload-expected.txt:
        * fast/events/iframe-object-onload.html: Change nil to null.
        * fast/forms/display-none-in-onchange-keyboard-expected.txt:
        * fast/forms/display-none-in-onchange-keyboard.html: Add missing id to div, change
        its text, and change the resulting text.
        * fast/forms/input-text-click-inside-expected.checksum:
        * fast/forms/input-text-click-inside-expected.png:
        * fast/forms/input-text-click-inside-expected.txt:
        * fast/forms/input-text-click-inside.html: Change moveTo(x,y) to mouseMoveTo(x,y).
        * fast/forms/input-text-click-outside-expected.txt:
        * fast/forms/input-text-click-outside.html: Ditto.
        * fast/js/array-foreach-expected.txt:
        * fast/js/array-foreach.html: Add try-catch blocks to catch exceptions and print them and
        general cleanup.
        * fast/js/array-some-expected.txt:
        * fast/js/array-some.html: Ditto.
        * fast/table/click-near-anonymous-table-expected.checksum:
        * fast/table/click-near-anonymous-table-expected.png:
        * fast/table/click-near-anonymous-table-expected.txt:
        * fast/table/click-near-anonymous-table.html: Change timeForward(ms) to leapForward(ms) and
        change the amount of time from 500ms to 1000ms to avoid clicking.

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

19 files changed:
LayoutTests/ChangeLog
LayoutTests/fast/events/iframe-object-onload-expected.txt
LayoutTests/fast/events/iframe-object-onload.html
LayoutTests/fast/forms/display-none-in-onchange-keyboard-expected.txt
LayoutTests/fast/forms/display-none-in-onchange-keyboard.html
LayoutTests/fast/forms/input-text-click-inside-expected.checksum
LayoutTests/fast/forms/input-text-click-inside-expected.png
LayoutTests/fast/forms/input-text-click-inside-expected.txt
LayoutTests/fast/forms/input-text-click-inside.html
LayoutTests/fast/forms/input-text-click-outside-expected.txt
LayoutTests/fast/forms/input-text-click-outside.html
LayoutTests/fast/js/array-foreach-expected.txt
LayoutTests/fast/js/array-foreach.html
LayoutTests/fast/js/array-some-expected.txt
LayoutTests/fast/js/array-some.html
LayoutTests/fast/table/click-near-anonymous-table-expected.checksum
LayoutTests/fast/table/click-near-anonymous-table-expected.png
LayoutTests/fast/table/click-near-anonymous-table-expected.txt
LayoutTests/fast/table/click-near-anonymous-table.html

index dfbab5c..d8ff8d3 100644 (file)
@@ -1,3 +1,33 @@
+2007-02-04  Sam Weinig  <sam@webkit.org>
+
+        Reviewed by Darin.
+
+        Patch for http://bugs.webkit.org/show_bug.cgi?id=12562
+        Fix test cases that are now spewing console errors that 
+        they are not testing for
+
+        * fast/events/iframe-object-onload-expected.txt:
+        * fast/events/iframe-object-onload.html: Change nil to null.
+        * fast/forms/display-none-in-onchange-keyboard-expected.txt:
+        * fast/forms/display-none-in-onchange-keyboard.html: Add missing id to div, change 
+        its text, and change the resulting text.
+        * fast/forms/input-text-click-inside-expected.checksum:
+        * fast/forms/input-text-click-inside-expected.png:
+        * fast/forms/input-text-click-inside-expected.txt:
+        * fast/forms/input-text-click-inside.html: Change moveTo(x,y) to mouseMoveTo(x,y).
+        * fast/forms/input-text-click-outside-expected.txt:
+        * fast/forms/input-text-click-outside.html: Ditto.
+        * fast/js/array-foreach-expected.txt:
+        * fast/js/array-foreach.html: Add try-catch blocks to catch exceptions and print them and 
+        general cleanup.
+        * fast/js/array-some-expected.txt:
+        * fast/js/array-some.html: Ditto.
+        * fast/table/click-near-anonymous-table-expected.checksum:
+        * fast/table/click-near-anonymous-table-expected.png:
+        * fast/table/click-near-anonymous-table-expected.txt:
+        * fast/table/click-near-anonymous-table.html: Change timeForward(ms) to leapForward(ms) and
+        change the amount of time from 500ms to 1000ms to avoid clicking.
+
 2007-02-03  Geoffrey Garen  <ggaren@apple.com>
 
         Landing results that I think Maciej intended to land, to get the buildbot
index 09ec615..bb0be3b 100644 (file)
@@ -1,4 +1,3 @@
-CONSOLE MESSAGE: line 19: ReferenceError: Can't find variable: nil
 This test checks that onload events fire on iframe and object elements when their contents finish loading, and that these are separate event objects from those that fire on the body inside the frame. If it passes, you will see 6 blocks below, each with a three-line message that ends in false.
 Fired load event on body inside iframe
 Fired event on iframe
index 555abfb..5670fa8 100644 (file)
@@ -16,7 +16,7 @@ function log(s)
     document.getElementById("console").innerHTML += s + "<br>";
 }
 
-var innerEvent = nil;
+var innerEvent = null;
 
 function logOuter(msg, event)
 {
index d522a8c..cef072c 100644 (file)
@@ -1,4 +1,3 @@
-CONSOLE MESSAGE: line 20: Null value
 
 This tests that setting display to 'none' for an element that's about to get focus doesn't crash.
 
index 02909cf..52b2d0c 100644 (file)
         // Simulate a tab. 
         eventSender.keyDown('\t', new Array());
 
-        document.getElementById('result').innerHTML = 'SUCCESS';        
+        document.getElementById('result').innerHTML = "SUCCESS - Didn't crash";
     }
     </script>
 </head>
 <body onload="runTest()">
     <input id="first" type="text" onchange="document.getElementById('u').style.display='none'"><input id="u" type="text">
     <p>This tests that setting display to 'none' for an element that's about to get focus doesn't crash.</p>
-    <div>SUCCESS - Didn't crash</div>
+    <div id="result">THE TEST DID NOT RUN</div>
 </body>
 </html>
index d6ba5e2..93332db 100644 (file)
@@ -1 +1 @@
-aba36613297b546d1d1100c7c45ed4ef
\ No newline at end of file
+6564425bebc5b7787fd842f379396474
\ No newline at end of file
index 1e35846..15dc15d 100644 (file)
Binary files a/LayoutTests/fast/forms/input-text-click-inside-expected.png and b/LayoutTests/fast/forms/input-text-click-inside-expected.png differ
index 41c98e3..10ff029 100644 (file)
@@ -1,4 +1,3 @@
-CONSOLE MESSAGE: line 5: TypeError: Value undefined (result of expression eventSender.moveTo) is not object.
 layer at (0,0) size 800x600
   RenderView at (0,0) size 800x600
 layer at (0,0) size 800x600
@@ -12,3 +11,4 @@ layer at (0,0) size 800x600
           text run at (0,18) width 158: "should have a focus ring."
 layer at (38,38) size 142x13
   RenderBlock {DIV} at (3,3) size 142x13
+caret: position 0 of child 0 {DIV} of child 0 {INPUT} of child 0 {DIV} of child 0 {BODY} of child 0 {HTML} of document
index 850cc9b..42c9cba 100644 (file)
@@ -2,7 +2,7 @@
 <p>This test clicks inside an input element, and must result in the element receiving focus. If the test succeeds the element should have a focus ring.</p>
 <script>
 if (window.eventSender) {
-    eventSender.moveTo(100, 45);
+    eventSender.mouseMoveTo(100, 45);
     eventSender.mouseDown();
     eventSender.mouseUp();
 }
index 72f3dee..4160778 100644 (file)
@@ -1,4 +1,3 @@
-CONSOLE MESSAGE: line 5: TypeError: Value undefined (result of expression eventSender.moveTo) is not object.
 layer at (0,0) size 800x600
   RenderView at (0,0) size 800x600
 layer at (0,0) size 800x600
@@ -12,3 +11,4 @@ layer at (0,0) size 800x600
           text run at (0,18) width 182: "should not have a focus ring."
 layer at (38,38) size 142x13
   RenderBlock {DIV} at (3,3) size 142x13
+caret: position 1 of child 0 {INPUT} of child 0 {DIV} of child 0 {BODY} of child 0 {HTML} of document
index b76942e..2ed44f7 100644 (file)
@@ -2,7 +2,7 @@
 <p>This test clicks outside an input element, and must not result in the element receiving focus. If the test succeeds the element should not have a focus ring.</p>
 <script>
 if (window.eventSender) {
-    eventSender.moveTo(225, 45);
+    eventSender.mouseMoveTo(200, 45);
     eventSender.mouseDown();
     eventSender.mouseUp();
 }
index d067fa5..6320e72 100644 (file)
@@ -1,4 +1,3 @@
-CONSOLE MESSAGE: line 100: ReferenceError: Can't find variable: exception
 1.0 Single Argument Testing
 The following tests forEach with one argument, the callback. It should print the contents of the array [2, 5, 9] alongside each index.
 
@@ -41,6 +40,7 @@ This test uses a function that throws an exception, and thus halts the execution
 
 [0] is 2
 [1] is 5
+Exception thrown, execution halted!
 
 5.0 Wrong Type for Callback Test This test sends in incorrect types for the callback parameter of forEach. An exception should be thrown in each case. There should be 6 type errors (and no crashes!):
 
index fabf5ca..3531192 100644 (file)
@@ -6,7 +6,8 @@ function print(str) {
     document.writeln(str+"<br/>");
 }
 
-if (window.layoutTestController)  layoutTestController.dumpAsText();
+if (window.layoutTestController)
+    layoutTestController.dumpAsText();
 </script>
 </head>
 <body>
@@ -74,7 +75,7 @@ This test changes elements in the array, these elements should appear in their m
 <script>
 function printEltAndChange(element, index, array) {
     print("[" + index + "] is " + element);
-    array[array.length-1-index]= "changed";
+    array[array.length - 1 - index] = "changed";
 }
 
 [2, 5, 9].forEach(printEltAndChange);
@@ -85,33 +86,56 @@ This test uses a function that throws an exception, and thus halts the execution
 <script>
 function printEltAndException(element, index, array) {
     print("[" + index + "] is " + element);
-    if(index==1) throw exception();
+    if (index == 1)
+        throw "Exception thrown, execution halted!";
+}
+try {
+    [2, 5, 9].forEach(printEltAndException);
+} catch (e) {
+    print(e);
 }
-
-[2, 5, 9].forEach(printEltAndException);
 </script>
+
 <br/>
 5.0 Wrong Type for Callback Test
 This test sends in incorrect types for the callback parameter of forEach.  An exception should be thrown in each case.  There should be 6 type errors (and no crashes!):<br/><br/>
 <script>
 
-try { [2, 5, 9].forEach(5); }
-catch (e) { print(e); }
+try {
+    [2, 5, 9].forEach(5);
+} catch (e) {
+    print(e);
+}
 
-try { [2, 5, 9].forEach("wrong"); }
-catch (e) { print(e); }
+try {
+    [2, 5, 9].forEach("wrong");
+} catch (e) {
+    print(e);
+}
 
-try { [2, 5, 9].forEach(new Object()); }
-catch (e) { print(e); }
+try {
+    [2, 5, 9].forEach(new Object());
+} catch (e) {
+    print(e);
+}
 
-try { [2, 5, 9].forEach(null); }
-catch (e) { print(e); }
+try {
+    [2, 5, 9].forEach(null);
+} catch (e) {
+    print(e);
+}
 
-try { [2, 5, 9].forEach(undefined); }
-catch (e) { print(e) }
+try {
+    [2, 5, 9].forEach(undefined);
+} catch (e) {
+    print(e)
+}
 
-try { [2, 5, 9].forEach(); }
-catch (e) { print(e); }
+try {
+    [2, 5, 9].forEach();
+} catch (e) {
+    print(e);
+}
 
 </script>
 
index d1cde4f..6ece20e 100644 (file)
@@ -1,4 +1,3 @@
-CONSOLE MESSAGE: line 100: ReferenceError: Can't find variable: isBigEnoughAndException
 1.0 Single Argument Testing
 The following tests every with one argument, the callback. It should print whether the arrays [2, 5, 8, 1, 4] and [12, 5, 8, 1, 44] contain any numbers >= to 10 (false and true, respectively).
 
@@ -33,8 +32,10 @@ true
 true
 
 4.0 Exception Test
-This test uses a function that throws an exception, and thus halts the execution of some. There should thus be no output.
+This test uses a function that throws an exception, and thus halts the execution of some. There should be 2 exceptions thrown.
 
+Exception thrown, execution halted!
+Exception thrown, execution halted!
 
 5.0 Wrong Type for Callback Test
 This test sends in incorrect types for the callback parameter of every. An exception should be thrown in each case. There should be 6 type errors (and no crashes!):
index 7c3208f..85b6975 100644 (file)
@@ -6,7 +6,8 @@ function print(str) {
     document.writeln(str+"<br/>");
 }
 
-if (window.layoutTestController)  layoutTestController.dumpAsText();
+if (window.layoutTestController)
+    layoutTestController.dumpAsText();
 </script>
 </head>
 <body>
@@ -71,7 +72,7 @@ print([12, 5, 8, 1, 44].some(isBigEnoughAndPush));
 This test is equivalent to 1.0, with the exception that it changes elements in the array to be > 10 in reverse order. These elements should appear in their mutated form when reached by every, and thus both tests should result in "true".<br/><br/>
 <script>
 function isBigEnoughAndChange(element, index, array) {
-    array[array.length-1-index]= 10;
+    array[array.length - 1 - index] = 10;
     return (element >= 10);
 }
 
@@ -80,15 +81,24 @@ print([12, 5, 8, 1, 44].some(isBigEnoughAndChange));
 </script>
 <br/>
 4.0 Exception Test<br/>
-This test uses a function that throws an exception, and thus halts the execution of some.  There should thus be no output.<br/><br/>
+This test uses a function that throws an exception, and thus halts the execution of some.  There should be 2 exceptions thrown.<br/><br/>
 <script>
-function isBigEnough(element, index, array) {
-    throw exception();
+function isBigEnoughAndException(element, index, array) {
+    throw "Exception thrown, execution halted!";
     return (element >= 10);
 }
 
-print([2, 5, 8, 1, 4].some(isBigEnoughAndException));
-print([12, 5, 8, 1, 44].some(isBigEnoughAndException));
+try {
+    [2, 5, 8, 1, 4].some(isBigEnoughAndException);
+} catch (e) {
+    print(e);
+}
+
+try {
+    [12, 5, 8, 1, 44].some(isBigEnoughAndException);
+} catch (e) {
+    print(e);
+}
 
 </script>
 <br/>
@@ -96,23 +106,41 @@ print([12, 5, 8, 1, 44].some(isBigEnoughAndException));
 This test sends in incorrect types for the callback parameter of every.  An exception should be thrown in each case.  There should be 6 type errors (and no crashes!):<br/><br/>
 <script>
 
-try { [12, 5, 8, 1, 44].some(5) }
-catch (e) { print(e); }
+try {
+    [12, 5, 8, 1, 44].some(5)
+} catch (e) {
+    print(e);
+}
 
-try { [12, 5, 8, 1, 44].some("wrong"); }
-catch (e) { print(e); }
+try {
+    [12, 5, 8, 1, 44].some("wrong");
+} catch (e) {
+    print(e);
+}
 
-try { [12, 5, 8, 1, 44].some(new Object()); }
-catch (e) { print(e); }
+try {
+    [12, 5, 8, 1, 44].some(new Object());
+} catch (e) {
+    print(e);
+}
 
-try { [12, 5, 8, 1, 44].some(null); }
-catch (e) { print(e); }
+try {
+    [12, 5, 8, 1, 44].some(null);
+} catch (e) {
+    print(e);
+}
 
-try { [12, 5, 8, 1, 44].some(undefined); }
-catch (e) { print(e) }
+try {
+    [12, 5, 8, 1, 44].some(undefined);
+} catch (e) {
+    print(e)
+}
 
-try { [12, 5, 8, 1, 44].some(); }
-catch (e) { print(e); }
+try {
+    [12, 5, 8, 1, 44].some();
+} catch (e) {
+    print(e);
+}
 
 </script>
 <br/>
@@ -120,7 +148,7 @@ catch (e) { print(e); }
 This test is nearly identical to 1.0, except that it prints upon every call to the designated callback function.  Since some aborts as soon as it finds one qualifying element, the first array should print 5 times, and the second only once.<br/><br/>
 <script>
 function isBigEnough(element, index, array) {
-    print("Testing element "+element+"...");
+    print("Testing element " + element + "...");
     return (element >= 10);
 }
 
index 9c2b3ac..8396abd 100644 (file)
@@ -1 +1 @@
-5229184684e7a329c7988e1e7e10a9d0
\ No newline at end of file
+7e58fe2709accf3efb12e524a46a1117
\ No newline at end of file
index e419478..f18be8e 100644 (file)
Binary files a/LayoutTests/fast/table/click-near-anonymous-table-expected.png and b/LayoutTests/fast/table/click-near-anonymous-table-expected.png differ
index 5df1e1d..532b008 100644 (file)
@@ -1,4 +1,3 @@
-CONSOLE MESSAGE: line 18: Value undefined (result of expression eventSender.timeForward) is not object.
 layer at (0,0) size 800x600
   RenderView at (0,0) size 800x600
 layer at (0,0) size 800x600
@@ -40,4 +39,4 @@ layer at (0,0) size 800x600
               RenderTableCell at (0,0) size 176x18 [r=0 c=0 rs=1 cs=1]
                 RenderText {#text} at (0,0) size 176x18
                   text run at (0,0) width 176: "Click to the right of this text"
-caret: position 9 of child 0 {#text} of child 7 {DIV} of child 7 {DIV} of child 1 {BODY} of child 0 {HTML} of document
+caret: position 40 of child 0 {#text} of child 3 {DIV} of child 7 {DIV} of child 1 {BODY} of child 0 {HTML} of document
index 5c5222f..a02fc30 100644 (file)
@@ -15,7 +15,7 @@
                 eventSender.mouseMoveTo(clickX, clickY);
                 eventSender.mouseDown();
                 eventSender.mouseUp();
-                eventSender.timeForward(500);   // we're not multi-clicking
+                eventSender.leapForward(1000);   // we're not multi-clicking
             }
         }
     </script>