WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 238374
Reduce EventListenerVector's minimum capacity from 16 to 2
https://bugs.webkit.org/show_bug.cgi?id=238374
Summary
Reduce EventListenerVector's minimum capacity from 16 to 2
Chris Dumez
Reported
2022-03-25 07:43:12 PDT
Reduce EventListenerVector's minimum capacity from 16 to 2 to save memory and get a small speedup on Speedometer. Very few event listeners are registered for a given type in the common case so eagerly allocating enough memory for 16 is wasteful. This is a confirmed 0.4% progression on Speedometer according to A/B bots.
Attachments
Patch
(1.47 KB, patch)
2022-03-25 07:44 PDT
,
Chris Dumez
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Chris Dumez
Comment 1
2022-03-25 07:44:10 PDT
Created
attachment 455760
[details]
Patch
Geoffrey Garen
Comment 2
2022-03-25 09:54:09 PDT
Comment on
attachment 455760
[details]
Patch r=me
EWS
Comment 3
2022-03-25 10:40:17 PDT
Committed
r291866
(
248872@main
): <
https://commits.webkit.org/248872@main
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 455760
[details]
.
Radar WebKit Bug Importer
Comment 4
2022-03-25 10:41:18 PDT
<
rdar://problem/90845294
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug