Skip to content

url: don't update URL immediately on update to URLSearchParams #51520

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
nodejs-github-bot merged 18 commits into from
Jan 24, 2024
Merged

url: don't update URL immediately on update to URLSearchParams #51520

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
2b2d11b
url: remove #context from URLSearchParams
MattIPv4 Jan 19, 2024
129a1a9
url: rely on URLSearchParams if in use for search
MattIPv4 Jan 19, 2024
c2e8526
url: update URLSearchParams directly from URL search setter
MattIPv4 Jan 19, 2024
039380e
url: create href using searchParams without bindingUrl call
MattIPv4 Jan 19, 2024
e015de2
url: ensure ? is present when relying on URLSearchParams
MattIPv4 Jan 19, 2024
2487726
url: don't use public getter for toString/toJSON
MattIPv4 Jan 19, 2024
b0270bc
url: update searchParams tests to expect encoded results
MattIPv4 Jan 19, 2024
d9c5381
url: only use URLSearchParams as store when modified
MattIPv4 Jan 19, 2024
a05675c
url: store search in URL on first access/update after URLSearchParams…
MattIPv4 Jan 19, 2024
310d275
url: add benchmark for param append for URL and URLSearchParams
MattIPv4 Jan 19, 2024
b118a90
url: add benchmark to check impact of searchParams checking
MattIPv4 Jan 19, 2024
916374e
url: reduce diff for changes
MattIPv4 Jan 19, 2024
319eaa0
url: more comments to explain lazy searchParams updating
MattIPv4 Jan 19, 2024
0c44fb6
url: cleanup from review
MattIPv4 Jan 19, 2024
065c45c
url: update variable naming, add more comments, include hash in bench…
MattIPv4 Jan 19, 2024
0d0fef8
url: add tests to ensure lazy URLSearchParams is exercised
MattIPv4 Jan 19, 2024
e9c4da2
url: fix updating url hash etc. after searchParams update, add test
MattIPv4 Jan 21, 2024
04f53a7
url: update tests for invalid this
MattIPv4 Jan 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
url: add benchmark for param append for URL and URLSearchParams
  • Loading branch information
@MattIPv4
MattIPv4 committed Jan 19, 2024
commit 310d275a1c400e02b5cfff0a5ff49b9af221895b
19 changes: 19 additions & 0 deletions benchmark/url/url-searchparams-append.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
'use strict';
const common = require('../common.js');

const bench = common.createBenchmark(main, {
type: ['URL', 'URLSearchParams'],
n: [1e3, 1e6],
});

function main({ type, n }) {
const params = type === 'URL' ?
new URL('https://nodejs.org').searchParams :
new URLSearchParams();

bench.start();
for (let i = 0; i < n; i++) {
params.append('test', i);
}
bench.end(n);
}