Closed Bug 1863562 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 42997 - Reland "Add argument to IsFocusable() for a11y use only"

Categories

(Core :: DOM: Core & HTML, task, P4)

task

Tracking

()

RESOLVED FIXED
122 Branch
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

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/70372e6c45fd [wpt PR 42997] - Reland "Add argument to IsFocusable() for a11y use only", a=testonly https://hg.mozilla.org/integration/autoland/rev/795d78e18e0e [wpt PR 42997] - Update wpt metadata, a=testonly
Test result changes from PR not available.
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/26b6fd27f1d7 [wpt PR 42997] - Reland "Add argument to IsFocusable() for a11y use only", a=testonly https://hg.mozilla.org/integration/autoland/rev/18250de25260 [wpt PR 42997] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch
You need to log in before you can comment on or make changes to this bug.