Bug 262407

Summary:	Use `HashSet<CheckedRef<Range>>` instead of `WeakHashSet<Range>` for Document::m_ranges
Product:	WebKit	Reporter:	Chris Dumez <cdumez>
Component:	DOM	Assignee:	Chris Dumez <cdumez>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

Description Chris Dumez 2023-09-29 15:03:07 PDT

Use `HashSet<CheckedRef<Range>>` instead of `WeakHashSet<Range>` for Document::m_ranges.
The Range destructor takes care of removing itself from Document::m_ranges already so we don't need a WeakPtr. Using CheckedRef here is slightly more efficient.

Comment 1 Chris Dumez 2023-09-29 15:04:28 PDT

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

Comment 2 EWS 2023-09-29 17:23:12 PDT

Committed 268680@main (cbb5cdba4811): <https://commits.webkit.org/268680@main>

Reviewed commits have been landed. Closing PR #18441 and removing active labels.

Comment 3 Radar WebKit Bug Importer 2023-09-29 17:24:13 PDT

<rdar://problem/116263456>