Bug 242624

Summary:	Optimize AtomStringImpl::remove by skipping unnecessary string equality checks
Product:	WebKit	Reporter:	Darin Adler <darin>
Component:	Web Template Framework	Assignee:	Darin Adler <darin>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	webkit-bug-importer, ysuzuki
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	All
OS:	All

Darin Adler

Reported 2022-07-11 17:55:41 PDT

Noticed that when AtomStringImpl removes a string from the atom string table, it does an equality check on the string instead of doing a pointer comparison. Easy to fix.

Attachments
Add attachment proposed patch, testcase, etc.

Darin Adler

Comment 1 2022-07-11 22:22:00 PDT

Pull request: https://github.com/WebKit/WebKit/pull/2320

EWS

Comment 2 2022-07-12 07:32:47 PDT

Committed 252379@main (d781022ac569): <https://commits.webkit.org/252379@main> Reviewed commits have been landed. Closing PR #2320 and removing active labels.

Radar WebKit Bug Importer

Comment 3 2022-07-12 07:33:15 PDT

<rdar://problem/96884503>

Note You need to log in before you can comment on or make changes to this bug.