Attachment 345765 Details for Bug 185796 – Patch

[patch] Patch

bug-185796-20180725110636.patch (text/plain), 3.63 KB, created by Andy VanWagoner on 2018-07-25 10:06:37 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Andy VanWagoner

Created: 2018-07-25 10:06:37 PDT

Size: 3.63 KB

patch

obsolete

>Subversion Revision: 234197
>diff --git a/Source/JavaScriptCore/ChangeLog b/Source/JavaScriptCore/ChangeLog
>index 33ee8b51dbc1873e4db953990f2b4b44b1ea19fe..0b4f2def95478a7bd7b4e2c327d54d246efb61aa 100644
>--- a/Source/JavaScriptCore/ChangeLog
>+++ b/Source/JavaScriptCore/ChangeLog
>@@ -1,3 +1,16 @@
>+2018-07-25  Andy VanWagoner  <andy@vanwagoner.family>
>+
>+        [INTL] Call Typed Array elements toLocaleString with locale and options
>+        https://bugs.webkit.org/show_bug.cgi?id=185796
>+
>+        Reviewed by Keith Miller.
>+
>+        Improve ECMA 402 compliance of typed array toLocaleString, passing along
>+        the locale and options to element toLocaleString calls.
>+
>+        * builtins/TypedArrayPrototype.js:
>+        (toLocaleString):
>+
> 2018-07-24  Fujii Hironori  <Hironori.Fujii@sony.com>
> 
>         runJITThreadLimitTests is failing
>diff --git a/Source/JavaScriptCore/builtins/TypedArrayPrototype.js b/Source/JavaScriptCore/builtins/TypedArrayPrototype.js
>index 66dd6451591e23c28f56fb6b000337f75e290725..989aa606414e647e34cf46d24fffabb480423ead 100644
>--- a/Source/JavaScriptCore/builtins/TypedArrayPrototype.js
>+++ b/Source/JavaScriptCore/builtins/TypedArrayPrototype.js
>@@ -396,7 +396,7 @@ function filter(callback /*, thisArg */)
>     return result;
> }
> 
>-function toLocaleString()
>+function toLocaleString(/* locale, options */)
> {
>     "use strict";
> 
>@@ -405,9 +405,9 @@ function toLocaleString()
>     if (length == 0)
>         return "";
> 
>-    var string = this[0].toLocaleString();
>+    var string = this[0].toLocaleString(@argument(0), @argument(1));
>     for (var i = 1; i < length; i++)
>-        string += "," + this[i].toLocaleString();
>+        string += "," + this[i].toLocaleString(@argument(0), @argument(1));
> 
>     return string;
> }
>diff --git a/JSTests/ChangeLog b/JSTests/ChangeLog
>index a9b8e5edc09c1c425ecec9ecc9c3bd981b4d7d7f..2a3cbf132d8acefbde4f085f33c0c36c14c67e18 100644
>--- a/JSTests/ChangeLog
>+++ b/JSTests/ChangeLog
>@@ -1,3 +1,14 @@
>+2018-07-25  Andy VanWagoner  <andy@vanwagoner.family>
>+
>+        [INTL] Call Typed Array elements toLocaleString with locale and options
>+        https://bugs.webkit.org/show_bug.cgi?id=185796
>+
>+        Reviewed by Keith Miller.
>+
>+        Remove now passing TypedArray toLocaleString expectation.
>+
>+        * test262/expectations.yaml:
>+
> 2018-07-24  Commit Queue  <commit-queue@webkit.org>
> 
>         Unreviewed, rolling out r234183.
>diff --git a/JSTests/test262/expectations.yaml b/JSTests/test262/expectations.yaml
>index b462a2b342776fefb5a7b548db739034e7df416c..eba3ec9dd5567080e7ae7a51fc2561dcaf72b584 100644
>--- a/JSTests/test262/expectations.yaml
>+++ b/JSTests/test262/expectations.yaml
>@@ -1780,9 +1780,6 @@ test/intl402/PluralRules/length.js:
> test/intl402/PluralRules/prototype/resolvedOptions/pluralCategories.js:
>   default: 'Test262Error: Should have different arrays Expected SameValue(ÃÂ«undefinedÃÂ», ÃÂ«undefinedÃÂ») to be false'
>   strict mode: 'Test262Error: Should have different arrays Expected SameValue(ÃÂ«undefinedÃÂ», ÃÂ«undefinedÃÂ») to be false'
>-test/intl402/TypedArray/prototype/toLocaleString/calls-toLocaleString-number-elements.js:
>-  default: 'Test262Error: Expected SameValue(ÃÂ«0ÃÂ», ÃÂ«Ã Â¹Â.Ã Â¹ÂÃ Â¹ÂÃ Â¹ÂÃÂ») to be true (Testing with Float64Array.)'
>-  strict mode: 'Test262Error: Expected SameValue(ÃÂ«0ÃÂ», ÃÂ«Ã Â¹Â.Ã Â¹ÂÃ Â¹ÂÃ Â¹ÂÃÂ») to be true (Testing with Float64Array.)'
> test/language/arguments-object/mapped/nonconfigurable-nonenumerable-nonwritable-descriptors-set-by-arguments.js:
>   default: 'Test262Error: Expected obj[0] to have enumerable:false.'
> test/language/arguments-object/mapped/nonconfigurable-nonenumerable-nonwritable-descriptors-set-by-param.js:

Actions: View | Formatted Diff | Diff

Attachments on bug 185796: 340768 | 343721 | 345765