Attachment 342968 Details for Bug 186786 – Patch

[patch] Patch

bug-186786-20180618164324.patch (text/plain), 13.31 KB, created by Leo Balter on 2018-06-18 13:43:25 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Leo Balter

Created: 2018-06-18 13:43:25 PDT

Size: 13.31 KB

patch

obsolete

>Subversion Revision: 232934
>diff --git a/JSTests/ChangeLog b/JSTests/ChangeLog
>index 2c8f26824852ac74dde25e4dea80dec220fd7204..aa9303f91e18f81209c91eea6e75f5dd89332f61 100644
>--- a/JSTests/ChangeLog
>+++ b/JSTests/ChangeLog
>@@ -1,3 +1,13 @@
>+2018-06-18  Leo Balter  <leonardo.balter@gmail.com>
>+
>+        [test262-runner] More updates to the skip list
>+        https://bugs.webkit.org/show_bug.cgi?id=186786
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * test262/config.yaml:
>+        * test262/expectations.yaml:
>+
> 2018-06-18  Michael Saboff  <msaboff@apple.com>
> 
>         Support Unicode 11 in RegExp
>diff --git a/JSTests/test262/config.yaml b/JSTests/test262/config.yaml
>index 10de6fcc42a0bd079c70aeabf92f61ff75798fed..2d2716c9efcd0968577a74a776ca78ae442085e4 100644
>--- a/JSTests/test262/config.yaml
>+++ b/JSTests/test262/config.yaml
>@@ -13,9 +13,13 @@ skip:
>     - async-iteration
>     # https://bugs.webkit.org/show_bug.cgi?id=186686
>     - Symbol.prototype.description
>-    # String.prototype.matchAll / Symbol.matchAll
>+    # https://bugs.webkit.org/show_bug.cgi?id=186694
>     - String.prototype.matchAll
>     - Symbol.matchAll
>+    # https://bugs.webkit.org/show_bug.cgi?id=174931
>+    - regexp-lookbehind
>+    # https://bugs.webkit.org/show_bug.cgi?id=126608
>+    - u180e
>   files:
>     - test/built-ins/Array/prototype/reverse/length-exceeding-integer-limit-with-object.js
>     - test/built-ins/Array/prototype/unshift/length-near-integer-limit.js
>@@ -46,3 +50,7 @@ skip:
>     # [[OwnPropertyKeys]] order with large integer index keys up to 2 ** 53 - 1
>     # https://bugs.webkit.org/show_bug.cgi?id=186691
>     - test/built-ins/Reflect/ownKeys/return-on-corresponding-order-large-index.js
>+
>+    # https://bugs.webkit.org/show_bug.cgi?id=186785
>+    - test/built-ins/global/global-object.js
>+    - test/built-ins/global/property-descriptor.js
>diff --git a/JSTests/test262/expectations.yaml b/JSTests/test262/expectations.yaml
>index 5cdd600aaa0087532c549123fa5283fd37b0b9d0..afa42de7932ed02f0967d175174e07b0dbb770f9 100644
>--- a/JSTests/test262/expectations.yaml
>+++ b/JSTests/test262/expectations.yaml
>@@ -5,9 +5,6 @@ test/annexB/built-ins/Function/createdynfn-html-close-comment-params.js:
> test/annexB/built-ins/Function/createdynfn-html-open-comment-params.js:
>   default: "SyntaxError: Unexpected token '}'. Expected a parameter pattern or a ')' in parameter list."
>   strict mode: "SyntaxError: Unexpected token '}'. Expected a parameter pattern or a ')' in parameter list."
>-test/annexB/built-ins/RegExp/named-groups/non-unicode-malformed.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
> test/annexB/built-ins/RegExp/prototype/Symbol.split/Symbol.match-getter-recompiles-source.js:
>   default: 'Test262Error: Expected SameValue(ÃÂ«ÃÂ», ÃÂ«aÃÂ») to be true'
>   strict mode: 'Test262Error: Expected SameValue(ÃÂ«ÃÂ», ÃÂ«aÃÂ») to be true'
>@@ -1046,15 +1043,6 @@ test/built-ins/Map/proto-from-ctor-realm.js:
> test/built-ins/Math/round/S15.8.2.15_A7.js:
>   default: "Test262Error: #4: '1 / Math.round(0.49999999999999994) !== 1 / 0'"
>   strict mode: "Test262Error: #4: '1 / Math.round(0.49999999999999994) !== 1 / 0'"
>-test/built-ins/Number/S9.3.1_A2_U180E.js:
>-  default: 'Test262Error: Number("\u180E") === NaN. Actual: 0 Expected SameValue(ÃÂ«0ÃÂ», ÃÂ«NaNÃÂ») to be true'
>-  strict mode: 'Test262Error: Number("\u180E") === NaN. Actual: 0 Expected SameValue(ÃÂ«0ÃÂ», ÃÂ«NaNÃÂ») to be true'
>-test/built-ins/Number/S9.3.1_A3_T1_U180E.js:
>-  default: 'Test262Error: Number("\u180E") === NaN Expected SameValue(ÃÂ«0ÃÂ», ÃÂ«NaNÃÂ») to be true'
>-  strict mode: 'Test262Error: Number("\u180E") === NaN Expected SameValue(ÃÂ«0ÃÂ», ÃÂ«NaNÃÂ») to be true'
>-test/built-ins/Number/S9.3.1_A3_T2_U180E.js:
>-  default: 'Test262Error: Number(dynaString("", "Ã¡Â Â")) === NaN Expected SameValue(ÃÂ«0ÃÂ», ÃÂ«NaNÃÂ») to be true'
>-  strict mode: 'Test262Error: Number(dynaString("", "Ã¡Â Â")) === NaN Expected SameValue(ÃÂ«0ÃÂ», ÃÂ«NaNÃÂ») to be true'
> test/built-ins/Number/proto-from-ctor-realm.js:
>   default: 'Test262Error: Expected SameValue(ÃÂ«0ÃÂ», ÃÂ«0ÃÂ») to be true'
>   strict mode: 'Test262Error: Expected SameValue(ÃÂ«0ÃÂ», ÃÂ«0ÃÂ») to be true'
>@@ -1134,57 +1122,6 @@ test/built-ins/Proxy/ownKeys/return-duplicate-symbol-entries-throws.js:
> test/built-ins/RegExp/S15.10.2.12_A2_T1.js:
>   default: 'Test262Error: #6: Error matching character class S between character 1800 and 1bff'
>   strict mode: 'Test262Error: #6: Error matching character class S between character 1800 and 1bff'
>-test/built-ins/RegExp/lookBehind/alternations.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/back-references-to-captures.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/back-references.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/captures-negative.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/captures.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/do-not-backtrack.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/greedy-loop.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/misc.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/mutual-recursive.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/negative.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/nested-lookaround.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/simple-fixed-length.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/sliced-strings.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/start-of-line.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/sticky.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/variable-length.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-test/built-ins/RegExp/lookBehind/word-boundary.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
> test/built-ins/RegExp/named-groups/groups-object-subclass-sans.js:
>   default: 'Test262Error: Expected SameValue(ÃÂ«bÃÂ», ÃÂ«$<a>ÃÂ») to be true'
>   strict mode: 'Test262Error: Expected SameValue(ÃÂ«bÃÂ», ÃÂ«$<a>ÃÂ») to be true'
>@@ -1197,9 +1134,6 @@ test/built-ins/RegExp/named-groups/groups-object-undefined.js:
> test/built-ins/RegExp/named-groups/groups-object.js:
>   default: 'Test262Error: Expected SameValue(ÃÂ«nullÃÂ», ÃÂ«[object Object]ÃÂ») to be true'
>   strict mode: 'Test262Error: Expected SameValue(ÃÂ«nullÃÂ», ÃÂ«[object Object]ÃÂ») to be true'
>-test/built-ins/RegExp/named-groups/lookbehind.js:
>-  default: 'SyntaxError: Invalid regular expression: invalid group specifier name'
>-  strict mode: 'SyntaxError: Invalid regular expression: invalid group specifier name'
> test/built-ins/RegExp/named-groups/non-unicode-malformed.js:
>   default: 'Test262Error: Expected a SyntaxError to be thrown but no exception was thrown at all'
>   strict mode: 'Test262Error: Expected a SyntaxError to be thrown but no exception was thrown at all'
>@@ -1260,9 +1194,6 @@ test/built-ins/RegExp/prototype/exec/u-lastindex-adv.js:
> test/built-ins/RegExp/prototype/test/S15.10.6.3_A1_T22.js:
>   default: 'Test262Error: #1: __re = /(?:ab|cd)\d?/g; __re.lastIndex=-1; __executed = __re.test("aacd22 "); __executed === true'
>   strict mode: 'Test262Error: #1: __re = /(?:ab|cd)\d?/g; __re.lastIndex=-1; __executed = __re.test("aacd22 "); __executed === true'
>-test/built-ins/RegExp/u180e.js:
>-  default: 'Test262Error: \s should not match U+180E Expected SameValue(ÃÂ«42ÃÂ», ÃÂ«Ã¡Â ÂÃÂ») to be true'
>-  strict mode: 'Test262Error: \s should not match U+180E Expected SameValue(ÃÂ«42ÃÂ», ÃÂ«Ã¡Â ÂÃÂ») to be true'
> test/built-ins/RegExp/unicode_restricted_brackets.js:
>   default: 'Test262Error: RegExp("]", "u"):  Expected a SyntaxError to be thrown but no exception was thrown at all'
>   strict mode: 'Test262Error: RegExp("]", "u"):  Expected a SyntaxError to be thrown but no exception was thrown at all'
>@@ -1293,9 +1224,6 @@ test/built-ins/Set/proto-from-ctor-realm.js:
> test/built-ins/String/proto-from-ctor-realm.js:
>   default: 'Test262Error: Expected SameValue(ÃÂ«ÃÂ», ÃÂ«ÃÂ») to be true'
>   strict mode: 'Test262Error: Expected SameValue(ÃÂ«ÃÂ», ÃÂ«ÃÂ») to be true'
>-test/built-ins/String/prototype/trim/u180e.js:
>-  default: 'Test262Error: Expected SameValue(ÃÂ«_ÃÂ», ÃÂ«_Ã¡Â ÂÃÂ») to be true'
>-  strict mode: 'Test262Error: Expected SameValue(ÃÂ«_ÃÂ», ÃÂ«_Ã¡Â ÂÃÂ») to be true'
> test/built-ins/ThrowTypeError/extensible.js:
>   default: 'Test262Error: Expected SameValue(ÃÂ«trueÃÂ», ÃÂ«falseÃÂ») to be true'
>   strict mode: 'Test262Error: Expected SameValue(ÃÂ«trueÃÂ», ÃÂ«falseÃÂ») to be true'
>@@ -1549,18 +1477,6 @@ test/built-ins/WeakMap/proto-from-ctor-realm.js:
> test/built-ins/WeakSet/proto-from-ctor-realm.js:
>   default: 'Test262Error: Expected SameValue(ÃÂ«[object WeakSet]ÃÂ», ÃÂ«[object WeakSet]ÃÂ») to be true'
>   strict mode: 'Test262Error: Expected SameValue(ÃÂ«[object WeakSet]ÃÂ», ÃÂ«[object WeakSet]ÃÂ») to be true'
>-test/built-ins/global/global-object.js:
>-  default: "ReferenceError: Can't find variable: global"
>-  strict mode: "ReferenceError: Can't find variable: global"
>-test/built-ins/global/property-descriptor.js:
>-  default: "TypeError: undefined is not an object (evaluating 'Object.getOwnPropertyDescriptor(obj, name).enumerable')"
>-  strict mode: "TypeError: undefined is not an object (evaluating 'Object.getOwnPropertyDescriptor(obj, name).enumerable')"
>-test/built-ins/parseFloat/S15.1.2.3_A2_T10_U180E.js:
>-  default: 'Test262Error: Single leading U+180E Expected SameValue(ÃÂ«1.1ÃÂ», ÃÂ«NaNÃÂ») to be true'
>-  strict mode: 'Test262Error: Single leading U+180E Expected SameValue(ÃÂ«1.1ÃÂ», ÃÂ«NaNÃÂ») to be true'
>-test/built-ins/parseInt/S15.1.2.2_A2_T10_U180E.js:
>-  default: 'Test262Error: Single leading U+180E Expected SameValue(ÃÂ«1ÃÂ», ÃÂ«NaNÃÂ») to be true'
>-  strict mode: 'Test262Error: Single leading U+180E Expected SameValue(ÃÂ«1ÃÂ», ÃÂ«NaNÃÂ») to be true'
> test/harness/assert-throws-early-referenceerror.js:
>   default: 'Test262Error: Function: 1 = 1; Expected a ReferenceError to be thrown but no exception was thrown at all'
>   strict mode: 'Test262Error: Function: 1 = 1; Expected a ReferenceError to be thrown but no exception was thrown at all'
>@@ -4020,9 +3936,3 @@ test/language/types/reference/put-value-prop-base-primitive-realm.js:
> test/language/types/reference/put-value-prop-base-primitive.js:
>   default: 'Test262Error: number Expected SameValue(ÃÂ«0ÃÂ», ÃÂ«1ÃÂ») to be true'
>   strict mode: 'TypeError: Attempted to assign to readonly property.'
>-test/language/white-space/mongolian-vowel-separator-eval.js:
>-  default: 'Test262Error: Expected a SyntaxError to be thrown but no exception was thrown at all'
>-  strict mode: 'Test262Error: Expected a SyntaxError to be thrown but no exception was thrown at all'
>-test/language/white-space/mongolian-vowel-separator.js:
>-  default: 'Test262: This statement should not be evaluated.'
>-  strict mode: 'Test262: This statement should not be evaluated.'

Actions: View | Formatted Diff | Diff

Attachments on bug 186786: 342968 | 342982