From d00a52558ac10fcd9e886cffe0fd5fe65fb28dfe Mon Sep 17 00:00:00 2001 From: Mestery Date: Sun, 6 Feb 2022 20:16:51 +0100 Subject: [PATCH] lib: use kEnumerableProperty from internal/util --- lib/internal/abort_controller.js | 7 +-- lib/internal/encoding.js | 11 ++-- lib/internal/event_target.js | 6 +-- lib/internal/url.js | 53 ++++++++++---------- lib/internal/webstreams/compression.js | 6 +-- lib/internal/webstreams/encoding.js | 8 ++- lib/internal/webstreams/queuingstrategies.js | 2 +- lib/internal/webstreams/readablestream.js | 2 +- lib/internal/webstreams/transformstream.js | 2 +- lib/internal/webstreams/util.js | 4 -- lib/internal/webstreams/writablestream.js | 2 +- lib/internal/worker/io.js | 5 +- 12 files changed, 49 insertions(+), 59 deletions(-) diff --git a/lib/internal/abort_controller.js b/lib/internal/abort_controller.js index 076670248fcdce..e0c0d63899972c 100644 --- a/lib/internal/abort_controller.js +++ b/lib/internal/abort_controller.js @@ -25,6 +25,7 @@ const { } = require('internal/event_target'); const { customInspectSymbol, + kEnumerableProperty, } = require('internal/util'); const { inspect } = require('internal/util/inspect'); const { @@ -260,7 +261,7 @@ function ClonedAbortSignal() { ClonedAbortSignal.prototype[kDeserialize] = () => {}; ObjectDefineProperties(AbortSignal.prototype, { - aborted: { enumerable: true } + aborted: kEnumerableProperty, }); ObjectDefineProperty(AbortSignal.prototype, SymbolToStringTag, { @@ -329,8 +330,8 @@ class AbortController { } ObjectDefineProperties(AbortController.prototype, { - signal: { enumerable: true }, - abort: { enumerable: true } + signal: kEnumerableProperty, + abort: kEnumerableProperty, }); ObjectDefineProperty(AbortController.prototype, SymbolToStringTag, { diff --git a/lib/internal/encoding.js b/lib/internal/encoding.js index 2cdd453b15ba91..def0e9223b84c7 100644 --- a/lib/internal/encoding.js +++ b/lib/internal/encoding.js @@ -30,7 +30,8 @@ const kEncoder = Symbol('encoder'); const { getConstructorOf, - customInspectSymbol: inspect + customInspectSymbol: inspect, + kEnumerableProperty, } = require('internal/util'); const { @@ -358,9 +359,9 @@ class TextEncoder { ObjectDefineProperties( TextEncoder.prototype, { - 'encode': { enumerable: true }, - 'encodeInto': { enumerable: true }, - 'encoding': { enumerable: true }, + 'encode': kEnumerableProperty, + 'encodeInto': kEnumerableProperty, + 'encoding': kEnumerableProperty, [SymbolToStringTag]: { configurable: true, value: 'TextEncoder' }, }); @@ -569,7 +570,7 @@ ObjectDefineProperties( ); ObjectDefineProperties(TextDecoder.prototype, { - decode: { enumerable: true }, + decode: kEnumerableProperty, [inspect]: { enumerable: false }, [SymbolToStringTag]: { configurable: true, diff --git a/lib/internal/event_target.js b/lib/internal/event_target.js index 3bc2fbb81cb20c..bc6f91fbe82159 100644 --- a/lib/internal/event_target.js +++ b/lib/internal/event_target.js @@ -8,7 +8,6 @@ const { FunctionPrototypeCall, NumberIsInteger, ObjectAssign, - ObjectCreate, ObjectDefineProperties, ObjectDefineProperty, ObjectGetOwnPropertyDescriptor, @@ -36,7 +35,7 @@ const { } = require('internal/errors'); const { validateObject, validateString } = require('internal/validators'); -const { customInspectSymbol } = require('internal/util'); +const { customInspectSymbol, kEnumerableProperty } = require('internal/util'); const { inspect } = require('util'); const kIsEventTarget = SymbolFor('nodejs.event_target'); @@ -298,9 +297,6 @@ class Event { static BUBBLING_PHASE = 3; } -const kEnumerableProperty = ObjectCreate(null); -kEnumerableProperty.enumerable = true; - ObjectDefineProperties( Event.prototype, { [SymbolToStringTag]: { diff --git a/lib/internal/url.js b/lib/internal/url.js index 284ffe15fb3eb0..41ae1b1a8be429 100644 --- a/lib/internal/url.js +++ b/lib/internal/url.js @@ -43,6 +43,7 @@ const { getConstructorOf, removeColors, toUSVString, + kEnumerableProperty, } = require('internal/util'); const { @@ -517,18 +518,18 @@ class URLSearchParams { } ObjectDefineProperties(URLSearchParams.prototype, { - append: { enumerable: true }, - delete: { enumerable: true }, - get: { enumerable: true }, - getAll: { enumerable: true }, - has: { enumerable: true }, - set: { enumerable: true }, - sort: { enumerable: true }, - entries: { enumerable: true }, - forEach: { enumerable: true }, - keys: { enumerable: true }, - values: { enumerable: true }, - toString: { enumerable: true }, + append: kEnumerableProperty, + delete: kEnumerableProperty, + get: kEnumerableProperty, + getAll: kEnumerableProperty, + has: kEnumerableProperty, + set: kEnumerableProperty, + sort: kEnumerableProperty, + entries: kEnumerableProperty, + forEach: kEnumerableProperty, + keys: kEnumerableProperty, + values: kEnumerableProperty, + toString: kEnumerableProperty, [SymbolToStringTag]: { configurable: true, value: 'URLSearchParams' }, // https://heycam.github.io/webidl/#es-iterable-entries @@ -1044,20 +1045,20 @@ class URL { ObjectDefineProperties(URL.prototype, { [kFormat]: { configurable: false, writable: false }, [SymbolToStringTag]: { configurable: true, value: 'URL' }, - toString: { enumerable: true }, - href: { enumerable: true }, - origin: { enumerable: true }, - protocol: { enumerable: true }, - username: { enumerable: true }, - password: { enumerable: true }, - host: { enumerable: true }, - hostname: { enumerable: true }, - port: { enumerable: true }, - pathname: { enumerable: true }, - search: { enumerable: true }, - searchParams: { enumerable: true }, - hash: { enumerable: true }, - toJSON: { enumerable: true }, + toString: kEnumerableProperty, + href: kEnumerableProperty, + origin: kEnumerableProperty, + protocol: kEnumerableProperty, + username: kEnumerableProperty, + password: kEnumerableProperty, + host: kEnumerableProperty, + hostname: kEnumerableProperty, + port: kEnumerableProperty, + pathname: kEnumerableProperty, + search: kEnumerableProperty, + searchParams: kEnumerableProperty, + hash: kEnumerableProperty, + toJSON: kEnumerableProperty, }); function update(url, params) { diff --git a/lib/internal/webstreams/compression.js b/lib/internal/webstreams/compression.js index 692f64af005af9..2ba8e53fa28e73 100644 --- a/lib/internal/webstreams/compression.js +++ b/lib/internal/webstreams/compression.js @@ -16,13 +16,11 @@ const { newReadableWritablePairFromDuplex, } = require('internal/webstreams/adapters'); -const { - customInspect, - kEnumerableProperty, -} = require('internal/webstreams/util'); +const { customInspect } = require('internal/webstreams/util'); const { customInspectSymbol: kInspect, + kEnumerableProperty, } = require('internal/util'); let zlib; diff --git a/lib/internal/webstreams/encoding.js b/lib/internal/webstreams/encoding.js index 5af59bc9f4a502..9cc76fae2ce10d 100644 --- a/lib/internal/webstreams/encoding.js +++ b/lib/internal/webstreams/encoding.js @@ -14,10 +14,7 @@ const { TransformStream, } = require('internal/webstreams/transformstream'); -const { - customInspect, - kEnumerableProperty, -} = require('internal/webstreams/util'); +const { customInspect } = require('internal/webstreams/util'); const { codes: { @@ -26,7 +23,8 @@ const { } = require('internal/errors'); const { - customInspectSymbol: kInspect + customInspectSymbol: kInspect, + kEnumerableProperty, } = require('internal/util'); const kHandle = Symbol('kHandle'); diff --git a/lib/internal/webstreams/queuingstrategies.js b/lib/internal/webstreams/queuingstrategies.js index cd8ec2bf35bfd0..78cec0c35559d3 100644 --- a/lib/internal/webstreams/queuingstrategies.js +++ b/lib/internal/webstreams/queuingstrategies.js @@ -14,6 +14,7 @@ const { const { customInspectSymbol: kInspect, + kEnumerableProperty, } = require('internal/util'); const { @@ -21,7 +22,6 @@ const { isBrandCheck, kType, kState, - kEnumerableProperty, } = require('internal/webstreams/util'); const { diff --git a/lib/internal/webstreams/readablestream.js b/lib/internal/webstreams/readablestream.js index 76200cbc1e876c..9853bd8b2cf2a3 100644 --- a/lib/internal/webstreams/readablestream.js +++ b/lib/internal/webstreams/readablestream.js @@ -50,6 +50,7 @@ const { const { createDeferredPromise, customInspectSymbol: kInspect, + kEnumerableProperty, } = require('internal/util'); const { @@ -109,7 +110,6 @@ const { nonOpStart, kType, kState, - kEnumerableProperty, } = require('internal/webstreams/util'); const { diff --git a/lib/internal/webstreams/transformstream.js b/lib/internal/webstreams/transformstream.js index e3cfd0362a26a5..457c9eb8fb338a 100644 --- a/lib/internal/webstreams/transformstream.js +++ b/lib/internal/webstreams/transformstream.js @@ -27,6 +27,7 @@ const { const { createDeferredPromise, customInspectSymbol: kInspect, + kEnumerableProperty, } = require('internal/util'); const { @@ -45,7 +46,6 @@ const { nonOpFlush, kType, kState, - kEnumerableProperty, } = require('internal/webstreams/util'); const { diff --git a/lib/internal/webstreams/util.js b/lib/internal/webstreams/util.js index 58f191cf07ff5a..305064ba05a490 100644 --- a/lib/internal/webstreams/util.js +++ b/lib/internal/webstreams/util.js @@ -207,9 +207,6 @@ function lazyTransfer() { return transfer; } -const kEnumerableProperty = ObjectCreate(null); -kEnumerableProperty.enumerable = true; - module.exports = { ArrayBufferViewGetBuffer, ArrayBufferViewGetByteLength, @@ -237,5 +234,4 @@ module.exports = { nonOpWrite, kType, kState, - kEnumerableProperty, }; diff --git a/lib/internal/webstreams/writablestream.js b/lib/internal/webstreams/writablestream.js index fa0d3b78e3e1ea..8c25cc33c15f5f 100644 --- a/lib/internal/webstreams/writablestream.js +++ b/lib/internal/webstreams/writablestream.js @@ -33,6 +33,7 @@ const { const { createDeferredPromise, customInspectSymbol: kInspect, + kEnumerableProperty, } = require('internal/util'); const { @@ -64,7 +65,6 @@ const { nonOpWrite, kType, kState, - kEnumerableProperty, } = require('internal/webstreams/util'); const { diff --git a/lib/internal/worker/io.js b/lib/internal/worker/io.js index 0f9f2d777a695c..822b8672525da3 100644 --- a/lib/internal/worker/io.js +++ b/lib/internal/worker/io.js @@ -18,6 +18,8 @@ const { SymbolFor, } = primordials; +const { kEnumerableProperty } = require('internal/util'); + const { handle_onclose: handleOnCloseSymbol, oninit: onInitSymbol, @@ -501,9 +503,6 @@ class BroadcastChannel extends EventTarget { } } -const kEnumerableProperty = ObjectCreate(null); -kEnumerableProperty.enumerable = true; - ObjectDefineProperties(BroadcastChannel.prototype, { name: kEnumerableProperty, close: kEnumerableProperty,