257427 – [popover] Implement autoPopoverList() as a separate list

Bug 257427 - [popover] Implement autoPopoverList() as a separate list

Summary: [popover] Implement autoPopoverList() as a separate list

Status:	RESOLVED FIXED

Alias:	None

Product:	WebKit
Classification:	Unclassified
Component:	DOM (show other bugs)
Version:	WebKit Nightly Build
Hardware:	Unspecified Unspecified

Importance:	P2 Normal
Assignee:	Tim Nguyen (:ntim)

URL:
Keywords:	InRadar

Depends on:
Blocks:	250171
	Show dependency tree / graph

Reported:	2023-05-27 12:45 PDT by Tim Nguyen (:ntim)
Modified:	2023-06-03 18:41 PDT (History)
CC List:	1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

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

Description Tim Nguyen (:ntim) 2023-05-27 12:45:12 PDT Comment hidden (obsolete)

In `hidePopoverInternal()` we call `Document::hideAllPopoversUntil` quite a lot. We should consider caching that list.

Comment 1 Tim Nguyen (:ntim) 2023-05-27 12:45:37 PDT

In `Document::hideAllPopoversUntil` we iterate through the top layer quite a lot. We should consider caching that list.

Comment 2 Radar WebKit Bug Importer 2023-05-27 12:45:58 PDT

<rdar://problem/109932621>

Comment 3 Tim Nguyen (:ntim) 2023-05-27 14:39:03 PDT

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

Comment 4 EWS 2023-06-01 15:29:47 PDT

Committed 264810@main (8141e23838a1): <https://commits.webkit.org/264810@main>

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