[wpt-sync] Sync PR 43268 - Reland: Put back the dialog SupportsFocus change
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 43268 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/43268
Details from upstream follow.
Mason Freed <masonf@chromium.org> wrote:
Reland: Put back the dialog SupportsFocus change
The [1] CL attempted to change SupportsFocus for dialogs to always
be true, but that caused some dialogs to become keyboard focusable
when they shouldn't be.This CL re-lands that behavior, but with the keyboard focusability
fixed: IsKeyboardFocusable needed to look explicitly at the things
that allow keyboard focusable dialogs, such as tabindex. Those are
handled by the base class Element::SupportsFocus.This also adds a DCHECK in focus_controller to make sure things
that are IsKeyboardFocusable() do not have negative tabindexes.[1] https://chromium-review.googlesource.com/c/chromium/src/+/4959753
[2] https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/web_tests/external/wpt/html/semantics/interactive-elements/the-dialog-element/dialog-focusability.htmlBug: 1499838
Change-Id: I7e87d1f707c35545e48e53bba58b5ba4cbce5124
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5034524
Auto-Submit: Mason Freed \<masonf@chromium.org>
Reviewed-by: Di Zhang \<dizhangg@chromium.org>
Commit-Queue: Mason Freed \<masonf@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1227721}
Assignee | ||
Comment 1•11 months ago
|
||
Assignee | ||
Updated•11 months ago
|
Assignee | ||
Comment 2•11 months ago
|
||
Assignee | ||
Updated•11 months ago
|
Assignee | ||
Updated•11 months ago
|
Assignee | ||
Comment 4•10 months ago
|
||
Assignee | ||
Comment 5•10 months ago
|
||
Assignee | ||
Comment 7•10 months ago
|
||
Comment 8•10 months ago
|
||
bugherder |
Description
•