Skip to content

[Gecko Bug 1858362] Style invalidation after call to CSS.registerProperty.#42481

Merged
moz-wptsync-bot merged 1 commit intomasterfrom
gecko/1858362
Oct 13, 2023
Merged

[Gecko Bug 1858362] Style invalidation after call to CSS.registerProperty.#42481
moz-wptsync-bot merged 1 commit intomasterfrom
gecko/1858362

Conversation

@moz-wptsync-bot
Copy link
Copy Markdown
Collaborator

Call to CSS.registerProperty may result in rendering changes compared
to when variables were interpreted as unregistered custom properties.
Ensure we recascade style for the whole document tree.

This was already covered by WPTs but add a more direct check for
paint and reflow.

Differential Revision: https://phabricator.services.mozilla.com/D190656

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1858362
gecko-commit: 8dbb04b2762bb8887488b710517a201b2ad09d9d
gecko-reviewers: emilio

Copy link
Copy Markdown
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Firefox project.

Call to CSS.registerProperty may result in rendering changes compared
to when variables were interpreted as unregistered custom properties.
Ensure we recascade style for the whole document tree.

This was already covered by WPTs but add a more direct check for
paint and reflow.

Differential Revision: https://phabricator.services.mozilla.com/D190656

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1858362
gecko-commit: 1b47cd29b7c39df5f3bf62e090bef658e2d03664
gecko-reviewers: emilio
@moz-wptsync-bot moz-wptsync-bot merged commit 1a44fc9 into master Oct 13, 2023
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1858362 branch October 13, 2023 03:18
servo-wpt-sync pushed a commit to servo/wpt that referenced this pull request Feb 12, 2026
…ion (web-platform-tests#42481)

Most likely a regression introduced in web-platform-tests#42208 where the pending loads
counter has been reset. For successful request this wasn't an issue, but
for failed requests it was. Therefore, guard against these generation
request problems by bailing out as soon as we post-process the request
(which we do for all failed requests).

Fixes web-platform-tests#42477

Signed-off-by: Tim van der Lippe <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants