diff options
author | Timothy Gu <timothygu99@gmail.com> | 2018-11-13 14:34:07 -0800 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2018-12-01 08:52:56 -0800 |
commit | e1438025edaf18d490548ba0f75f7ba702493c22 (patch) | |
tree | 48701c2430ca21424a0341e61166b1c3967aaa9d /lib | |
parent | e9de43549843da9f4f081cce917945878967df7e (diff) | |
download | android-node-v8-e1438025edaf18d490548ba0f75f7ba702493c22.tar.gz android-node-v8-e1438025edaf18d490548ba0f75f7ba702493c22.tar.bz2 android-node-v8-e1438025edaf18d490548ba0f75f7ba702493c22.zip |
Revert "url: make the context non-enumerable"
This reverts commit 5e1bf058f4906c0a49c34c6a1353d0b34784c80a, as it
causes major performance regressions during object construction.
Refs: https://github.com/nodejs/node/pull/24218
PR-URL: https://github.com/nodejs/node/pull/24495
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/internal/url.js | 15 |
1 files changed, 2 insertions, 13 deletions
diff --git a/lib/internal/url.js b/lib/internal/url.js index 44b658b2d9..310e9ff9e2 100644 --- a/lib/internal/url.js +++ b/lib/internal/url.js @@ -246,14 +246,7 @@ function onParseError(flags, input) { // Reused by URL constructor and URL#href setter. function parse(url, input, base) { const base_context = base ? base[context] : undefined; - // In the URL#href setter - if (!url[context]) { - Object.defineProperty(url, context, { - enumerable: false, - configurable: false, - value: new URLContext() - }); - } + url[context] = new URLContext(); _parse(input.trim(), -1, base_context, undefined, onParseComplete.bind(url), onParseError); } @@ -1381,11 +1374,7 @@ function toPathIfFileURL(fileURLOrPath) { } function NativeURL(ctx) { - Object.defineProperty(this, context, { - enumerable: false, - configurable: false, - value: ctx - }); + this[context] = ctx; } NativeURL.prototype = URL.prototype; |