New wpt failures in /css/cssom-view/ [client-props-zoom.html, scroll-zoom.html]
Categories
(Core :: DOM: CSS Object Model, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox124 | --- | fixed |
People
(Reporter: wpt-sync, Assigned: emilio)
References
Details
(Whiteboard: [wpt], [wptsync upstream])
Attachments
(3 files)
Syncing wpt PR 43953 found new untriaged test failures in CI
Tests Affected
Firefox-only failures
- /css/cssom-view/client-props-zoom.html [wpt.fyi]
- Client properties for elements with css zoom 1:
FAIL
- Client properties for elements with css zoom 1:
- /css/cssom-view/scroll-zoom.html [wpt.fyi]
- scroll properties for elements with css zoom 1:
FAIL
- scroll properties for elements with css zoom 1:
CI Results
Missing results from treeherder
GitHub PR Head
Notes
These updates will be on mozilla-central once bug 1874310 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.
Comment 1•7 months ago
|
||
The pull request that added this WPT (upstream) references this google doc for more info (title "DOM APIs affected by CSS zoom"):
https://docs.google.com/document/d/1AcnDShjT-kEuRaMchZPm5uaIgNZ4OiYtM4JI9qiV8Po/edit
Assignee | ||
Comment 2•7 months ago
|
||
Yup, this is a known thing to fix before shipping. Thanks!
Assignee | ||
Comment 3•7 months ago
|
||
Also extend scroll-zoom to cover scroll{Top,Left,To,By}, since that was
completely untested.
Updated•7 months ago
|
Assignee | ||
Comment 4•6 months ago
|
||
This the "body propagated to root" case to also set aFrame.
This doesn't change behavior for current callers, but would be useful to
simplify the following patches.
Assignee | ||
Comment 5•6 months ago
|
||
This makes the next patch simpler too. No behavior change.
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/974683909782 Guarantee that Element::GetScrollFrame returns a frame if it returns a scrollframe. r=TYLin https://hg.mozilla.org/integration/autoland/rev/13da2be03857 Refactor some scroll APIs to use shared code. r=TYLin,webidl,smaug https://hg.mozilla.org/integration/autoland/rev/467c5213a79a Make Element.{client,scroll} APIs zoom aware. r=saschanaz,TYLin
Reporter | ||
Comment 7•6 months ago
|
||
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/44452 for changes under testing/web-platform/tests
Reporter | ||
Updated•6 months ago
|
Reporter | ||
Comment 9•6 months ago
|
||
Upstream PR was closed without merging
Assignee | ||
Updated•6 months ago
|
Comment 10•6 months ago
|
||
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6e353a8663db Guarantee that Element::GetScrollFrame returns a frame if it returns a scrollframe. r=TYLin https://hg.mozilla.org/integration/autoland/rev/1557fda26451 Refactor some scroll APIs to use shared code. r=TYLin,webidl,smaug https://hg.mozilla.org/integration/autoland/rev/fbf03aba7e9d Make Element.{client,scroll} APIs zoom aware. r=saschanaz,TYLin
Comment 11•6 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/6e353a8663db
https://hg.mozilla.org/mozilla-central/rev/1557fda26451
https://hg.mozilla.org/mozilla-central/rev/fbf03aba7e9d
Reporter | ||
Comment 12•6 months ago
|
||
Upstream PR merged by moz-wptsync-bot
Description
•