Unreviewed, uncomment erroneously commented test code.
authorkeith_miller@apple.com <keith_miller@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 May 2018 23:48:36 +0000 (23:48 +0000)
committerkeith_miller@apple.com <keith_miller@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 May 2018 23:48:36 +0000 (23:48 +0000)
* stress/error-stack-trace-limit.js:

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

JSTests/ChangeLog
JSTests/stress/error-stack-trace-limit.js

index b8487d1..aa28cad 100644 (file)
@@ -1,5 +1,11 @@
 2018-05-30  Keith Miller  <keith_miller@apple.com>
 
+        Unreviewed, uncomment erroneously commented test code.
+
+        * stress/error-stack-trace-limit.js:
+
+2018-05-30  Keith Miller  <keith_miller@apple.com>
+
         LLInt get_by_id prototype caching doesn't properly handle changes
         https://bugs.webkit.org/show_bug.cgi?id=186112
 
index 745ef0e..12f5b9f 100644 (file)
@@ -47,17 +47,17 @@ function testLimit(testID, updateLimit, reentryCount, expectedLimit, expectedNum
         assertEquals(testID + 3, numberOfFrames(exception.stack), expectedNumberOfFrames);
 }
 
-// testLimit(1000, () => { Error.stackTraceLimit = 0 }, 1000, 0, 0);
-// // note: Chrome always prints a header line. So, Chrome expects "Error" here.
-// assertEquals(1100, exception.stack, "");
+testLimit(1000, () => { Error.stackTraceLimit = 0 }, 1000, 0, 0);
+// note: Chrome always prints a header line. So, Chrome expects "Error" here.
+assertEquals(1100, exception.stack, "");
 
-// testLimit(2000, () => { Error.stackTraceLimit = 10 }, 1000, 10, 10);
-// testLimit(3000, () => { Error.stackTraceLimit = 100 }, 1000, 100, 100);
-// testLimit(4000, () => { Error.stackTraceLimit = 1000 }, 1000, 1000, 1000);
+testLimit(2000, () => { Error.stackTraceLimit = 10 }, 1000, 10, 10);
+testLimit(3000, () => { Error.stackTraceLimit = 100 }, 1000, 100, 100);
+testLimit(4000, () => { Error.stackTraceLimit = 1000 }, 1000, 1000, 1000);
 
-// // expectedNumberOfFrames includes (1) global + (2) testLimit + (3) 1000 recursion of
-// // recurse() + (4) recurse() which discovered x == 0 i.e. expectedNumberOfFrames == 1003.
-// testLimit(5000, () => { Error.stackTraceLimit = 2000 }, 1000, 2000, 1003);
+// expectedNumberOfFrames includes (1) global + (2) testLimit + (3) 1000 recursion of
+// recurse() + (4) recurse() which discovered x == 0 i.e. expectedNumberOfFrames == 1003.
+testLimit(5000, () => { Error.stackTraceLimit = 2000 }, 1000, 2000, 1003);
 
 var value = { };
 testLimit(6000, () => { Error.stackTraceLimit = value }, 1000, value, undefined);