diff options
author | Aaron O'Mullan <aaron.omullan@gmail.com> | 2021-08-19 17:36:14 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-08-19 17:36:14 +0200 |
commit | 91051047cc82e6074348d17414854efb906019d2 (patch) | |
tree | a588b45d4de89d0f28f89de25a1b5950275e7c62 /ext/url/00_url.js | |
parent | bfc197f33e7df6f3c1a882d70c48ae742e04e288 (diff) |
perf(ext/url): use DOMString instead of USVString as webidl converter for URL parsing (#11775)
A 20% decrease in url parsing:
- before: `~2450ns/parse`
- after: `~1950ns/parse`
Diffstat (limited to 'ext/url/00_url.js')
-rw-r--r-- | ext/url/00_url.js | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/ext/url/00_url.js b/ext/url/00_url.js index 805a61329..e63d70bf8 100644 --- a/ext/url/00_url.js +++ b/ext/url/00_url.js @@ -313,9 +313,9 @@ */ constructor(url, base = undefined) { const prefix = "Failed to construct 'URL'"; - url = webidl.converters.USVString(url, { prefix, context: "Argument 1" }); + url = webidl.converters.DOMString(url, { prefix, context: "Argument 1" }); if (base !== undefined) { - base = webidl.converters.USVString(base, { + base = webidl.converters.DOMString(base, { prefix, context: "Argument 2", }); @@ -363,7 +363,7 @@ webidl.assertBranded(this, URL); const prefix = "Failed to set 'hash' on 'URL'"; webidl.requiredArguments(arguments.length, 1, { prefix }); - value = webidl.converters.USVString(value, { + value = webidl.converters.DOMString(value, { prefix, context: "Argument 1", }); @@ -385,7 +385,7 @@ webidl.assertBranded(this, URL); const prefix = "Failed to set 'host' on 'URL'"; webidl.requiredArguments(arguments.length, 1, { prefix }); - value = webidl.converters.USVString(value, { + value = webidl.converters.DOMString(value, { prefix, context: "Argument 1", }); @@ -407,7 +407,7 @@ webidl.assertBranded(this, URL); const prefix = "Failed to set 'hostname' on 'URL'"; webidl.requiredArguments(arguments.length, 1, { prefix }); - value = webidl.converters.USVString(value, { + value = webidl.converters.DOMString(value, { prefix, context: "Argument 1", }); @@ -429,7 +429,7 @@ webidl.assertBranded(this, URL); const prefix = "Failed to set 'href' on 'URL'"; webidl.requiredArguments(arguments.length, 1, { prefix }); - value = webidl.converters.USVString(value, { + value = webidl.converters.DOMString(value, { prefix, context: "Argument 1", }); @@ -454,7 +454,7 @@ webidl.assertBranded(this, URL); const prefix = "Failed to set 'password' on 'URL'"; webidl.requiredArguments(arguments.length, 1, { prefix }); - value = webidl.converters.USVString(value, { + value = webidl.converters.DOMString(value, { prefix, context: "Argument 1", }); @@ -476,7 +476,7 @@ webidl.assertBranded(this, URL); const prefix = "Failed to set 'pathname' on 'URL'"; webidl.requiredArguments(arguments.length, 1, { prefix }); - value = webidl.converters.USVString(value, { + value = webidl.converters.DOMString(value, { prefix, context: "Argument 1", }); @@ -498,7 +498,7 @@ webidl.assertBranded(this, URL); const prefix = "Failed to set 'port' on 'URL'"; webidl.requiredArguments(arguments.length, 1, { prefix }); - value = webidl.converters.USVString(value, { + value = webidl.converters.DOMString(value, { prefix, context: "Argument 1", }); @@ -520,7 +520,7 @@ webidl.assertBranded(this, URL); const prefix = "Failed to set 'protocol' on 'URL'"; webidl.requiredArguments(arguments.length, 1, { prefix }); - value = webidl.converters.USVString(value, { + value = webidl.converters.DOMString(value, { prefix, context: "Argument 1", }); @@ -542,7 +542,7 @@ webidl.assertBranded(this, URL); const prefix = "Failed to set 'search' on 'URL'"; webidl.requiredArguments(arguments.length, 1, { prefix }); - value = webidl.converters.USVString(value, { + value = webidl.converters.DOMString(value, { prefix, context: "Argument 1", }); @@ -565,7 +565,7 @@ webidl.assertBranded(this, URL); const prefix = "Failed to set 'username' on 'URL'"; webidl.requiredArguments(arguments.length, 1, { prefix }); - value = webidl.converters.USVString(value, { + value = webidl.converters.DOMString(value, { prefix, context: "Argument 1", }); |