Title: [89842] trunk/LayoutTests
Revision
89842
Author
[email protected]
Date
2011-06-27 11:40:18 -0700 (Mon, 27 Jun 2011)

Log Message

The fast/js/recursion-limit-equal test is too fragile
https://bugs.webkit.org/show_bug.cgi?id=63352

Patch by Filip Pizlo <[email protected]> on 2011-06-27
Reviewed by Gavin Barraclough.

* fast/js/recursion-limit-equal-expected.txt:
* fast/js/script-tests/recursion-limit-equal.js:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (89841 => 89842)


--- trunk/LayoutTests/ChangeLog	2011-06-27 18:36:06 UTC (rev 89841)
+++ trunk/LayoutTests/ChangeLog	2011-06-27 18:40:18 UTC (rev 89842)
@@ -1,3 +1,13 @@
+2011-06-27  Filip Pizlo  <[email protected]>
+
+        Reviewed by Gavin Barraclough.
+
+        The fast/js/recursion-limit-equal test is too fragile
+        https://bugs.webkit.org/show_bug.cgi?id=63352
+
+        * fast/js/recursion-limit-equal-expected.txt:
+        * fast/js/script-tests/recursion-limit-equal.js:
+
 2011-06-27  Abhishek Arya  <[email protected]>
 
         Reviewed by Simon Fraser.

Modified: trunk/LayoutTests/fast/js/recursion-limit-equal-expected.txt (89841 => 89842)


--- trunk/LayoutTests/fast/js/recursion-limit-equal-expected.txt	2011-06-27 18:36:06 UTC (rev 89841)
+++ trunk/LayoutTests/fast/js/recursion-limit-equal-expected.txt	2011-06-27 18:40:18 UTC (rev 89842)
@@ -1,4 +1,3 @@
-CONSOLE MESSAGE: line 9: RangeError: Maximum call stack size exceeded.
 Tests hitting the recursion limit with equality comparisons. At one point this crashed due to lack of exception checking inside the engine.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
@@ -6,6 +5,7 @@
 
 If the test did not crash, it has passed.
 
+PASS test() threw exception RangeError: Maximum call stack size exceeded..
 PASS successfullyParsed is true
 
 TEST COMPLETE

Modified: trunk/LayoutTests/fast/js/script-tests/recursion-limit-equal.js (89841 => 89842)


--- trunk/LayoutTests/fast/js/script-tests/recursion-limit-equal.js	2011-06-27 18:36:06 UTC (rev 89841)
+++ trunk/LayoutTests/fast/js/script-tests/recursion-limit-equal.js	2011-06-27 18:40:18 UTC (rev 89842)
@@ -14,4 +14,5 @@
 debug('If the test did not crash, it has passed.');
 debug('');
 
-test();
+shouldThrow("test()");
+
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to