Closed Bug 1892334 Opened 7 months ago Closed 6 months ago

New wpt failures in /shadow-dom/focus-navigation/ [delegatesFocus-highlight-sibling.html, focus-navigation-slot-fallback-default-tabindex.html, focus-navigation-slot-fallback.html, and 15 others]

Categories

(Core :: DOM: Core & HTML, defect)

defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: wpt-sync, Unassigned)

References

Details

(Whiteboard: [wpt])

Syncing wpt PR 45772 found new untriaged test failures in CI

Tests Affected

New Tests That Don't Pass

CI Results

Gecko CI (Treeherder)
GitHub PR Head

Notes

These updates will be on mozilla-central once bug 1892119 lands.

Note: this bug is for tracking fixing the issues and is not
owned by the wpt sync bot.

This bug is linked to the relevant tests by an annotation in
https://github.com/web-platform-tests/wpt-metadata. These annotations
can be edited using the wpt interop dashboard
https://jgraham.github.io/wptdash/

If this bug is split into multiple bugs, please also update the
annotations, otherwise we are unable to track which wpt issues are
already triaged. Resolving as duplicate or closing this issue should
be cause the bot to automatically update or remove the annotation.

Hi Sean, maybe you're interested in taking a quick look at for these new wpt failures? Do they ring any bell?

Severity: -- → S3
Flags: needinfo?(sefeng)

Yeah, looks like some changes made to WPT caused us start to fail, we have been passing for those tests for a long time. I'll continue investigate to see why

So how the test works is there's a from element that gains the focus initially, then a tab key gets sent to document.documentElement, that supposes to move the focus to the next tabbable element.

However, this document.documentElement is new, it was document.body before the change. Since our marionette code would try to focus the target element, the focused element gets changed to <html> rather than remains at the from element. This is because document.body isn't a legit focusable element, so focusing document.body doesn't blur the existing focus.

I've asked the chrome folks to see if the change was intentional.

Flags: needinfo?(sefeng)

I think they are fixed now because Chrome folks have landed a fix https://chromium-review.googlesource.com/c/chromium/src/+/5519292.

Not sure why m-c doesn't have this change yet.

I believe this now has been fixed by the above WPT change.

Status: NEW → RESOLVED
Closed: 6 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.