Title: [220636] releases/WebKitGTK/webkit-2.18
Revision
220636
Author
[email protected]
Date
2017-08-13 23:00:26 -0700 (Sun, 13 Aug 2017)

Log Message

Merge r220449 - REGRESSION: 2 test262/test/language/statements/async-function failures
https://bugs.webkit.org/show_bug.cgi?id=175334

Reviewed by Yusuke Suzuki.

JSTests:

Add @skip parameters to tests, and remove test for async iterator from
async await syntax test because it is already covered by async-iterator-syntax.js

* stress/async-await-syntax.js:
* stress/async-iteration-syntax.js:

Source/_javascript_Core:

Switch off useAsyncIterator by default

* runtime/Options.h:

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.18/JSTests/ChangeLog (220635 => 220636)


--- releases/WebKitGTK/webkit-2.18/JSTests/ChangeLog	2017-08-14 05:56:04 UTC (rev 220635)
+++ releases/WebKitGTK/webkit-2.18/JSTests/ChangeLog	2017-08-14 06:00:26 UTC (rev 220636)
@@ -1,3 +1,16 @@
+2017-08-09  Oleksandr Skachkov  <[email protected]>
+
+        REGRESSION: 2 test262/test/language/statements/async-function failures
+        https://bugs.webkit.org/show_bug.cgi?id=175334
+
+        Reviewed by Yusuke Suzuki.
+
+        Add @skip parameters to tests, and remove test for async iterator from 
+        async await syntax test because it is already covered by async-iterator-syntax.js
+
+        * stress/async-await-syntax.js:
+        * stress/async-iteration-syntax.js:
+
 2017-08-08  Yusuke Suzuki  <[email protected]>
 
         Unreviewed, gardening test262 for Promise resolve / reject function length

Modified: releases/WebKitGTK/webkit-2.18/JSTests/stress/async-await-syntax.js (220635 => 220636)


--- releases/WebKitGTK/webkit-2.18/JSTests/stress/async-await-syntax.js	2017-08-14 05:56:04 UTC (rev 220635)
+++ releases/WebKitGTK/webkit-2.18/JSTests/stress/async-await-syntax.js	2017-08-14 06:00:26 UTC (rev 220636)
@@ -193,10 +193,7 @@
 
     testSyntaxError(`var asyncArrowFn = async() => await;`);
 
-    testSyntax(`var asyncFn = async function*() {}`);
-    testSyntax(`async function* asyncGenerator() {}`);
     testSyntaxError(`var O = { *async asyncGeneratorMethod() {} };`);
-    testSyntax(`var O = { async *asyncGeneratorMethod() {} };`);
     testSyntaxError(`var O = { async asyncGeneratorMethod*() {} };`);
 
     testSyntaxError(`var asyncFn = async function(x = await 1) { return x; }`);

Modified: releases/WebKitGTK/webkit-2.18/JSTests/stress/async-iteration-syntax.js (220635 => 220636)


--- releases/WebKitGTK/webkit-2.18/JSTests/stress/async-iteration-syntax.js	2017-08-14 05:56:04 UTC (rev 220635)
+++ releases/WebKitGTK/webkit-2.18/JSTests/stress/async-iteration-syntax.js	2017-08-14 06:00:26 UTC (rev 220636)
@@ -1,3 +1,7 @@
+// This test requires enableAsyncIterator to be enabled at run time.
+//@ skip
+
+
 var assert = function (result, expected, message = "") {
   if (result !== expected) {
     throw new Error('Error in assert. Expected "' + expected + '" but was "' + result + '":' + message );

Modified: releases/WebKitGTK/webkit-2.18/Source/_javascript_Core/ChangeLog (220635 => 220636)


--- releases/WebKitGTK/webkit-2.18/Source/_javascript_Core/ChangeLog	2017-08-14 05:56:04 UTC (rev 220635)
+++ releases/WebKitGTK/webkit-2.18/Source/_javascript_Core/ChangeLog	2017-08-14 06:00:26 UTC (rev 220636)
@@ -1,3 +1,14 @@
+2017-08-09  Oleksandr Skachkov  <[email protected]>
+
+        REGRESSION: 2 test262/test/language/statements/async-function failures
+        https://bugs.webkit.org/show_bug.cgi?id=175334
+
+        Reviewed by Yusuke Suzuki.
+
+        Switch off useAsyncIterator by default
+
+        * runtime/Options.h:
+
 2017-08-08  Filip Pizlo  <[email protected]>
 
         ICs should do caging

Modified: releases/WebKitGTK/webkit-2.18/Source/_javascript_Core/runtime/Options.h (220635 => 220636)


--- releases/WebKitGTK/webkit-2.18/Source/_javascript_Core/runtime/Options.h	2017-08-14 05:56:04 UTC (rev 220635)
+++ releases/WebKitGTK/webkit-2.18/Source/_javascript_Core/runtime/Options.h	2017-08-14 06:00:26 UTC (rev 220636)
@@ -448,7 +448,7 @@
     \
     v(bool, useWebAssembly, true, Normal, "Expose the WebAssembly global object.") \
     \
-    v(bool, useAsyncIterator, true, Normal, "Allow to use Async Iterator in JS.") \
+    v(bool, useAsyncIterator, false, Normal, "Allow to use Async Iterator in JS.") \
     \
     v(bool, failToCompileWebAssemblyCode, false, Normal, "If true, no Wasm::Plan will sucessfully compile a function.") \
     v(size, webAssemblyPartialCompileLimit, 5000, Normal, "Limit on the number of bytes a Wasm::Plan::compile should attempt before checking for other work.") \
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to