[wpt-sync] Sync PR 42997 - Reland "Add argument to IsFocusable() for a11y use only"
Categories
(Core :: DOM: Core & HTML, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox122 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 42997 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/42997
Details from upstream follow.
Mason Freed <masonf@chromium.org> wrote:
Reland "Add argument to IsFocusable() for a11y use only"
This is a reland of commit 5d2f4b32390faebd8b8da13ad7856d482b2477a1
Original change's description:
Add argument to IsFocusable() for a11y use only
This CL adds this:
virtual bool IsFocusable(
bool disallow_layout_updates_for_accessibility_only = false);for use by a11y code, which sometimes hits the CHECK for clean
layout. When this bool is true (default false), layout will not
be updated to check focusability, and possibly-stale layout objects
will be used.Bug: 1485059, 1489580, 1492703
Change-Id: I7f8b73a74341e6dad04102c327b1b8ce99a2ef1d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4959753
Auto-Submit: Mason Freed \<masonf@chromium.org>
Commit-Queue: Mason Freed \<masonf@chromium.org>
Reviewed-by: Chris Harrelson \<chrishtr@chromium.org>
Commit-Queue: Chris Harrelson \<chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1213856}Bug: 1485059, 1489580, 1492703
Bug: 1499838
Change-Id: I4e18099fdd98bf92be1fc829a763298541fbe267
Reviewed-on: https://chromium-review.googlesource.com/5008754
WPT-Export-Revision: ffd429a3e4409fbda1afb054dc6decbf3f7a8d30
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 1•2 years ago
|
||
Assignee | ||
Comment 3•2 years ago
|
||
Comment 5•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/26b6fd27f1d7
https://hg.mozilla.org/mozilla-central/rev/18250de25260
Description
•