2008-06-07 Cameron Zwarich <cwzwarich@uwaterloo.ca>

        Reviewed by Oliver.

        Bug 17663: Wrong sequencing of argument list evaluation
        <https://bugs.webkit.org/show_bug.cgi?id=17663>

        SquirrelFish fixed this bug, but we should add a test.

        LayoutTests:

        * fast/js/order-of-operations-expected.txt:
        * fast/js/resources/order-of-operations.js:

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

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index 8dff9fdd347e836ffb843accb79bf5a11ab9c358..96f9d56df5a1781dbcbcfd1aaf81c5cf728d9847 100644 (file)
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,15 @@
+2008-06-07  Cameron Zwarich  <cwzwarich@uwaterloo.ca>
+
+        Reviewed by Oliver.
+
+        Bug 17663: Wrong sequencing of argument list evaluation
+        <https://bugs.webkit.org/show_bug.cgi?id=17663>
+
+        SquirrelFish fixed this bug, but we should add a test.
+
+        * fast/js/order-of-operations-expected.txt:
+        * fast/js/resources/order-of-operations.js:
+
 2008-06-06  Robert Blaut  <webkit@blaut.biz>
 
         Reviewed by Darin Adler.

diff --git a/LayoutTests/fast/js/order-of-operations-expected.txt b/LayoutTests/fast/js/order-of-operations-expected.txt
index 13c7a836a0639a5182226664cb283aa1458d8f64..20f34de974c564f131449e40dee21e44f75407ee 100644 (file)
--- a/LayoutTests/fast/js/order-of-operations-expected.txt
+++ b/LayoutTests/fast/js/order-of-operations-expected.txt
@@ -15,6 +15,7 @@ PASS (function(){ leftRight = ""; left() && right(); return leftRight; })() is "
 PASS (function(){ leftRight = ""; left() & right(); return leftRight; })() is "LeftRight"
 PASS (function(){ leftRight = ""; left() | right(); return leftRight; })() is "LeftRight"
 PASS (function(){ leftRight = ""; left() ^ right(); return leftRight; })() is "LeftRight"
+PASS testEvaluationOfArguments() is true
 PASS successfullyParsed is true
 
 TEST COMPLETE

diff --git a/LayoutTests/fast/js/resources/order-of-operations.js b/LayoutTests/fast/js/resources/order-of-operations.js
index fdcef7f027840934b2f69833641902552d7035f0..bd8936e4020cc24f9e8523b0700878ff9da2d89f 100644 (file)
--- a/LayoutTests/fast/js/resources/order-of-operations.js
+++ b/LayoutTests/fast/js/resources/order-of-operations.js
@@ -25,4 +25,22 @@ shouldBe('(function(){ leftRight = ""; left() & right(); return leftRight; })()'
 shouldBe('(function(){ leftRight = ""; left() | right(); return leftRight; })()', '"LeftRight"');
 shouldBe('(function(){ leftRight = ""; left() ^ right(); return leftRight; })()', '"LeftRight"');
 
+function testEvaluationOfArguments()
+{
+    function throwPass()
+    {
+        throw "PASS";
+    }
+    
+    var nonFunction = 42;
+    
+    try {
+        nonFunction(throwPass());
+    } catch (e) {
+        return e == "PASS";
+    }
+}
+
+shouldBeTrue("testEvaluationOfArguments()");
+
 var successfullyParsed = true;