[webkit-changes] [232319] trunk/JSTests

[keith_miller]

Title: [232319] trunk/JSTests

Revision

232319

Author

keith_mil...@apple.com

Date

2018-05-30 16:48:36 -0700 (Wed, 30 May 2018)

Log Message

Unreviewed, uncomment erroneously commented test code.

* stress/error-stack-trace-limit.js:

Modified Paths

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

Diff

Modified: trunk/JSTests/ChangeLog (232318 => 232319)

--- trunk/JSTests/ChangeLog	2018-05-30 23:42:36 UTC (rev 232318)
+++ trunk/JSTests/ChangeLog	2018-05-30 23:48:36 UTC (rev 232319)
@@ -1,5 +1,11 @@
 2018-05-30  Keith Miller  <keith_mil...@apple.com>
 
+        Unreviewed, uncomment erroneously commented test code.
+
+        * stress/error-stack-trace-limit.js:
+
+2018-05-30  Keith Miller  <keith_mil...@apple.com>
+
         LLInt get_by_id prototype caching doesn't properly handle changes
         https://bugs.webkit.org/show_bug.cgi?id=186112
 

Modified: trunk/JSTests/stress/error-stack-trace-limit.js (232318 => 232319)

--- trunk/JSTests/stress/error-stack-trace-limit.js	2018-05-30 23:42:36 UTC (rev 232318)
+++ trunk/JSTests/stress/error-stack-trace-limit.js	2018-05-30 23:48:36 UTC (rev 232319)
@@ -47,17 +47,17 @@
         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);

_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes