Implement Object.fromEntries
authoryusukesuzuki@slowstart.org <yusukesuzuki@slowstart.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 2 Sep 2018 16:41:45 +0000 (16:41 +0000)
committeryusukesuzuki@slowstart.org <yusukesuzuki@slowstart.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 2 Sep 2018 16:41:45 +0000 (16:41 +0000)
https://bugs.webkit.org/show_bug.cgi?id=188481

Reviewed by Darin Adler.

JSTests:

* stress/object-from-entries.js: Added.
(shouldBe):
(shouldThrow):
(shouldBe.JSON.stringify.Object.getOwnPropertyDescriptor):
(shouldBe.set get shouldThrow):

Source/JavaScriptCore:

Object.fromEntries becomes stage 3[1]. This patch implements it by using builtin JS.

[1]: https://tc39.github.io/proposal-object-from-entries/

* builtins/ObjectConstructor.js:
(fromEntries):
* runtime/ObjectConstructor.cpp:

LayoutTests:

* js/Object-getOwnPropertyNames-expected.txt:
* js/script-tests/Object-getOwnPropertyNames.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@235589 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JSTests/ChangeLog
JSTests/stress/object-from-entries.js [new file with mode: 0644]
LayoutTests/ChangeLog
LayoutTests/js/Object-getOwnPropertyNames-expected.txt
LayoutTests/js/script-tests/Object-getOwnPropertyNames.js
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/builtins/ObjectConstructor.js
Source/JavaScriptCore/runtime/ObjectConstructor.cpp

index 55e27c8..cecac86 100644 (file)
@@ -1,3 +1,16 @@
+2018-09-02  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
+
+        Implement Object.fromEntries
+        https://bugs.webkit.org/show_bug.cgi?id=188481
+
+        Reviewed by Darin Adler.
+
+        * stress/object-from-entries.js: Added.
+        (shouldBe):
+        (shouldThrow):
+        (shouldBe.JSON.stringify.Object.getOwnPropertyDescriptor):
+        (shouldBe.set get shouldThrow):
+
 2018-08-24  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
 
         Function object should convert params to string before throw a parsing error
diff --git a/JSTests/stress/object-from-entries.js b/JSTests/stress/object-from-entries.js
new file mode 100644 (file)
index 0000000..9e3befc
--- /dev/null
@@ -0,0 +1,196 @@
+function shouldBe(actual, expected) {
+    if (actual !== expected)
+        throw new Error('bad value: ' + actual);
+}
+
+function shouldThrow(func, errorMessage) {
+    var errorThrown = false;
+    var error = null;
+    try {
+        func();
+    } catch (e) {
+        errorThrown = true;
+        error = e;
+    }
+    if (!errorThrown)
+        throw new Error('not thrown');
+    if (String(error) !== errorMessage)
+        throw new Error(`bad error: ${String(error)}`);
+}
+
+shouldBe(JSON.stringify(Object.getOwnPropertyDescriptor(Object, "fromEntries")), `{"writable":true,"enumerable":false,"configurable":true}`);
+shouldBe(Object.fromEntries.length, 1);
+
+shouldThrow(() => Object.fromEntries(null), `TypeError: null is not an object`);
+shouldThrow(() => Object.fromEntries(undefined), `TypeError: undefined is not an object`);
+shouldThrow(() => Object.fromEntries(0), `TypeError: undefined is not a function`);
+shouldThrow(() => Object.fromEntries(true), `TypeError: undefined is not a function`);
+shouldThrow(() => Object.fromEntries(Symbol("Cocoa")), `TypeError: undefined is not a function`);
+shouldThrow(() => Object.fromEntries("Cocoa"), `TypeError: Object.fromEntries requires the first iterable parameter yields objects`);
+shouldThrow(() => Object.fromEntries([0]), `TypeError: Object.fromEntries requires the first iterable parameter yields objects`);
+shouldThrow(() => Object.fromEntries([["Cocoa", "Cappuccino"], 0]), `TypeError: Object.fromEntries requires the first iterable parameter yields objects`);
+
+{
+    let object = Object.fromEntries([]);
+    shouldBe(JSON.stringify(object), `{}`);
+}
+{
+    let object = Object.fromEntries([["Cocoa", "Cappuccino"]]);
+    shouldBe(JSON.stringify(object), `{"Cocoa":"Cappuccino"}`);
+    shouldBe(JSON.stringify(Object.getOwnPropertyDescriptor(object, "Cocoa")), `{"value":"Cappuccino","writable":true,"enumerable":true,"configurable":true}`);
+}
+{
+    let obj = { abc: 1, def: 2, ghij: 3 };
+    let res = Object.fromEntries(
+        Object.entries(obj)
+            .filter(([ key, val ]) => key.length === 3)
+            .map(([ key, val ]) => [ key, val * 2 ])
+        );
+    shouldBe(JSON.stringify(res), `{"abc":2,"def":4}`);
+}
+{
+    let map = new Map([ [ 'a', 1 ], [ 'b', 2 ], [ 'c', 3 ] ]);
+    let obj = Object.fromEntries(map);
+    shouldBe(JSON.stringify(obj), `{"a":1,"b":2,"c":3}`);
+}
+{
+    let arr = [ { name: 'Alice', age: 40 }, { name: 'Bob', age: 36 } ];
+    let obj = Object.fromEntries(arr.map(({ name, age }) => [ name, age ]));
+    shouldBe(JSON.stringify(obj), `{"Alice":40,"Bob":36}`);
+}
+{
+    Object.defineProperty(Object.prototype, "bad", {
+        get() { throw new Error("out"); },
+        set(v) { throw new Error("out"); }
+    });
+    shouldThrow(() => {
+        let object = {};
+        object.bad;
+    }, `Error: out`);
+    shouldThrow(() => {
+        let object = {};
+        object.bad = 42;
+    }, `Error: out`);
+    let object = Object.fromEntries([["bad", "value"]]);
+    shouldBe(JSON.stringify(object), `{"bad":"value"}`);
+}
+{
+    var counter = 0;
+    class Recorder {
+        constructor(first, second)
+        {
+            this.first = first;
+            this.second = second;
+        }
+
+        get 0()
+        {
+            shouldBe(counter++, this.first);
+            return this.first;
+        }
+
+        get 1()
+        {
+            shouldBe(counter++, this.second);
+            return this.second;
+        }
+    }
+    var result = Object.fromEntries([new Recorder(0, 1), new Recorder(2, 3)]);
+    shouldBe(result[0], 1);
+    shouldBe(result[2], 3);
+    shouldBe(counter, 4);
+}
+{
+    class Iterable {
+        constructor()
+        {
+        }
+
+        *[Symbol.iterator]()
+        {
+            yield [0, 1];
+            yield [1, 2];
+        }
+    }
+
+    var result = Object.fromEntries(new Iterable);
+    shouldBe(result[0], 1);
+    shouldBe(result[1], 2);
+}
+{
+    class Iterator {
+        constructor()
+        {
+            this.index = 0;
+        }
+
+        next()
+        {
+            if (this.index === 4)
+                throw new Error("out");
+
+            this.index++;
+            return {
+                value: [0, 1],
+                done: false
+            };
+        }
+    }
+
+    class Iterable {
+        constructor()
+        {
+        }
+
+        [Symbol.iterator]()
+        {
+            return new Iterator;
+        }
+    }
+
+    try {
+        Object.fromEntries(new Iterable);
+    } catch (error) {
+        shouldBe(String(error), `Error: out`);
+    }
+}
+{
+    let array = [[], ['c', 'd']];
+    let object = Object.fromEntries(array);
+    shouldBe(JSON.stringify(Object.keys(object).sort()), `["c","undefined"]`);
+    shouldBe(object.c, 'd');
+    shouldBe(object.undefined, undefined);
+}
+{
+    let symbol = Symbol('Cocoa');
+    let array = [[symbol, 42]];
+    let object = Object.fromEntries(array);
+    shouldBe(Object.getOwnPropertySymbols(object).length, 1);
+    shouldBe(Object.getOwnPropertyNames(object).length, 0);
+    shouldBe(object.hasOwnProperty(symbol), true);
+    shouldBe(object[symbol], 42);
+}
+{
+    Object.defineProperty(Object.prototype, "hello", {
+        get() {
+            throw new Error("out");
+        },
+        set() {
+            throw new Error("out");
+        }
+    });
+    let result = Object.fromEntries([["hello", 42]]);
+    shouldBe(result.hello, 42);
+}
+{
+    let array = [['a', 'b'], ['c', 'd']];
+    Object.defineProperty(array, 0, {
+        get()
+        {
+            array.push(['e', 'f']);
+            return ['a', 'b'];
+        }
+    });
+    let object = Object.fromEntries(array);
+    shouldBe(JSON.stringify(object), `{"a":"b","c":"d","e":"f"}`);
+}
index 4517b11..2c2f481 100644 (file)
@@ -1,3 +1,13 @@
+2018-09-02  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
+
+        Implement Object.fromEntries
+        https://bugs.webkit.org/show_bug.cgi?id=188481
+
+        Reviewed by Darin Adler.
+
+        * js/Object-getOwnPropertyNames-expected.txt:
+        * js/script-tests/Object-getOwnPropertyNames.js:
+
 2018-08-31  Simon Fraser  <simon.fraser@apple.com>
 
         Clean up TestExpectations so that ./Tools/Scripts/lint-test-expectations
index 99c039a..565258e 100644 (file)
@@ -42,7 +42,7 @@ PASS getSortedOwnPropertyNames(decodeURI) is ['length', 'name']
 PASS getSortedOwnPropertyNames(decodeURIComponent) is ['length', 'name']
 PASS getSortedOwnPropertyNames(encodeURI) is ['length', 'name']
 PASS getSortedOwnPropertyNames(encodeURIComponent) is ['length', 'name']
-PASS getSortedOwnPropertyNames(Object) is ['assign', 'create', 'defineProperties', 'defineProperty', 'entries', 'freeze', 'getOwnPropertyDescriptor', 'getOwnPropertyDescriptors', 'getOwnPropertyNames', 'getOwnPropertySymbols', 'getPrototypeOf', 'is', 'isExtensible', 'isFrozen', 'isSealed', 'keys', 'length', 'name', 'preventExtensions', 'prototype', 'seal', 'setPrototypeOf', 'values']
+PASS getSortedOwnPropertyNames(Object) is ['assign', 'create', 'defineProperties', 'defineProperty', 'entries', 'freeze', 'fromEntries', 'getOwnPropertyDescriptor', 'getOwnPropertyDescriptors', 'getOwnPropertyNames', 'getOwnPropertySymbols', 'getPrototypeOf', 'is', 'isExtensible', 'isFrozen', 'isSealed', 'keys', 'length', 'name', 'preventExtensions', 'prototype', 'seal', 'setPrototypeOf', 'values']
 PASS getSortedOwnPropertyNames(Object.prototype) is ['__defineGetter__', '__defineSetter__', '__lookupGetter__', '__lookupSetter__', '__proto__', 'constructor', 'hasOwnProperty', 'isPrototypeOf', 'propertyIsEnumerable', 'toLocaleString', 'toString', 'valueOf']
 PASS getSortedOwnPropertyNames(Function) is ['length', 'name', 'prototype']
 PASS getSortedOwnPropertyNames(Function.prototype) is ['apply', 'arguments', 'bind', 'call', 'caller', 'constructor', 'length', 'name', 'toString']
index 7e1ab5a..32a38c3 100644 (file)
@@ -51,7 +51,7 @@ var expectedPropertyNamesSet = {
     "encodeURI": "['length', 'name']",
     "encodeURIComponent": "['length', 'name']",
 // Built-in ECMA objects
-    "Object": "['assign', 'create', 'defineProperties', 'defineProperty', 'entries', 'freeze', 'getOwnPropertyDescriptor', 'getOwnPropertyDescriptors', 'getOwnPropertyNames', 'getOwnPropertySymbols', 'getPrototypeOf', 'is', 'isExtensible', 'isFrozen', 'isSealed', 'keys', 'length', 'name', 'preventExtensions', 'prototype', 'seal', 'setPrototypeOf', 'values']",
+    "Object": "['assign', 'create', 'defineProperties', 'defineProperty', 'entries', 'freeze', 'fromEntries', 'getOwnPropertyDescriptor', 'getOwnPropertyDescriptors', 'getOwnPropertyNames', 'getOwnPropertySymbols', 'getPrototypeOf', 'is', 'isExtensible', 'isFrozen', 'isSealed', 'keys', 'length', 'name', 'preventExtensions', 'prototype', 'seal', 'setPrototypeOf', 'values']",
     "Object.prototype": "['__defineGetter__', '__defineSetter__', '__lookupGetter__', '__lookupSetter__', '__proto__', 'constructor', 'hasOwnProperty', 'isPrototypeOf', 'propertyIsEnumerable', 'toLocaleString', 'toString', 'valueOf']",
     "Function": "['length', 'name', 'prototype']",
     "Function.prototype": "['apply', 'arguments', 'bind', 'call', 'caller', 'constructor', 'length', 'name', 'toString']",
index 18d19d6..7c1abdc 100644 (file)
@@ -1,3 +1,18 @@
+2018-09-02  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
+
+        Implement Object.fromEntries
+        https://bugs.webkit.org/show_bug.cgi?id=188481
+
+        Reviewed by Darin Adler.
+
+        Object.fromEntries becomes stage 3[1]. This patch implements it by using builtin JS.
+
+        [1]: https://tc39.github.io/proposal-object-from-entries/
+
+        * builtins/ObjectConstructor.js:
+        (fromEntries):
+        * runtime/ObjectConstructor.cpp:
+
 2018-08-24  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
 
         Function object should convert params to string before throw a parsing error
index ada0eca..5a76f48 100644 (file)
@@ -1,6 +1,7 @@
 /*
  * Copyright (C) 2016 Oleksandr Skachkov <gskachkov@gmail.com>.
  * Copyright (C) 2015 Jordan Harband. All rights reserved.
+ * Copyright (C) 2018 Yusuke Suzuki <yusukesuzuki@slowstart.org>.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -39,3 +40,20 @@ function entries(object)
 
     return properties;
 }
+
+function fromEntries(iterable)
+{
+    "use strict";
+
+    let object = {};
+
+    for (let entry of iterable) {
+        if (!@isObject(entry))
+            @throwTypeError("Object.fromEntries requires the first iterable parameter yields objects");
+        let key = entry[0];
+        let value = entry[1];
+        @putByValDirect(object, key, value);
+    }
+
+    return object;
+}
index b3a2964..3f9f81c 100644 (file)
@@ -87,6 +87,7 @@ const ClassInfo ObjectConstructor::s_info = { "Function", &InternalFunction::s_i
   assign                    objectConstructorAssign                     DontEnum|Function 2
   values                    objectConstructorValues                     DontEnum|Function 1
   entries                   JSBuiltin                                   DontEnum|Function 1
+  fromEntries               JSBuiltin                                   DontEnum|Function 1
 @end
 */