Changeset 215360 in webkit

Timestamp:

Apr 14, 2017 2:08:51 AM (7 years ago)

Author:

gskachkov@gmail.com

Message:

Safari 10.1 JSON.parse returns incorrect object for numeric keys with decimal values
​https://bugs.webkit.org/show_bug.cgi?id=170442

Reviewed by Yusuke Suzuki.

• stress/object-number-properties.js:

Location:

trunk/JSTests

Files:

• ChangeLog (modified) (1 diff)
• stress/object-number-properties.js (modified) (1 diff)

trunk/JSTests/ChangeLog

@@ +1 @@
+2017-04-14  Oleksandr Skachkov  <gskachkov@gmail.com>
+
+        Safari 10.1 JSON.parse returns incorrect object for numeric keys with decimal values
+        https://bugs.webkit.org/show_bug.cgi?id=170442
+
+        Reviewed by Yusuke Suzuki.
+
+        * stress/object-number-properties.js:
+
 2017-04-13  Yusuke Suzuki  <utatane.tea@gmail.com>
 

trunk/JSTests/stress/object-number-properties.js

@@ -81 +81 @@
 assert(baz[250], "A");
 assert(baz[1001], "B");
+
+var bar = JSON.parse('{"0":97.1,"1000":96.5,"2000":96.1,"3000":97.4,"4000":90.4}');
+
+assert(Object.keys(bar).length, 5);
+assert(bar[0], 97.1);
+assert(bar[1000], 96.5);
+assert(bar[2000], 96.1);
+assert(bar[3000], 97.4);
+assert(bar[4000], 90.4);
+
+bar = JSON.parse('{"0":97.1, "250": 65.3, "1000":96.5,"2000":96.1,"3000":97.4,"4000":90.4}');
+
+assert(Object.keys(bar).length, 6);
+
+assert(bar[0], 97.1);
+assert(bar[250], 65.3);
+assert(bar[1000], 96.5);
+assert(bar[2000], 96.1);
+assert(bar[3000], 97.4);
+assert(bar[4000], 90.4);
+
+bar = JSON.parse('{"0":97.1, "250": null, "1000":96.5,"2000":96.1,"3000":97.4,"4000":90.4}');
+
+assert(Object.keys(bar).length, 6);
+
+assert(bar[0], 97.1);
+assert(bar[250], null);
+assert(bar[1000], 96.5);
+assert(bar[2000], 96.1);
+assert(bar[3000], 97.4);
+assert(bar[4000], 90.4);
+
+
+bar = eval('(()=>({"0":97.1, "250": 65.3, "1000":96.5,"2000":96.1,"3000":97.4,"4000":90.4}))();');
+
+assert(Object.keys(bar).length, 6);
+
+assert(bar[0], 97.1);
+assert(bar[250], 65.3);
+assert(bar[1000], 96.5);
+assert(bar[2000], 96.1);
+assert(bar[3000], 97.4);
+assert(bar[4000], 90.4);
+
+bar = eval('(()=>({"0":97.1, "250": null, "1000":96.5,"2000":96.1,"3000":97.4,"4000":90.4}))();');
+
+assert(Object.keys(bar).length, 6);
+
+assert(bar[0], 97.1);
+assert(bar[250], null);
+assert(bar[1000], 96.5);
+assert(bar[2000], 96.1);
+assert(bar[3000], 97.4);
+assert(bar[4000], 90.4);
+
+
+bar = eval('(()=>({"0":97.1, "250": NaN, "1000":96.5,"2000":96.1,"3000":97.4,"4000":90.4}))();');
+
+assert(Object.keys(bar).length, 6);
+
+assert(bar[0], 97.1);
+assert(isNaN(bar[250]), true);
+assert(bar[1000], 96.5);
+assert(bar[2000], 96.1);
+assert(bar[3000], 97.4);
+assert(bar[4000], 90.4);