Skip to content

Commit

Permalink
url: make URL.parse enumerable
Browse files Browse the repository at this point in the history
PR-URL: #53720
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
  • Loading branch information
panva authored and aduh95 committed Jul 16, 2024
1 parent bc48090 commit 0f2b57d
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 8 deletions.
6 changes: 6 additions & 0 deletions lib/internal/url.js
Original file line number Diff line number Diff line change
Expand Up @@ -1167,6 +1167,12 @@ ObjectDefineProperties(URL, {
writable: true,
enumerable: true,
},
parse: {
__proto__: null,
configurable: true,
writable: true,
enumerable: true,
},
});

function installObjectURLMethods() {
Expand Down
8 changes: 0 additions & 8 deletions test/wpt/status/url.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,14 +13,6 @@
]
}
},
"idlharness.any.js": {
"fail": {
"note": "TODO(@panva)",
"expected": [
"URL interface: operation parse(USVString, optional USVString)"
]
}
},
"url-setters-a-area.window.js": {
"skip": "already tested in url-setters.any.js"
},
Expand Down

0 comments on commit 0f2b57d

Please sign in to comment.