Skip to content

script: Fix render blocking stylesheet load fire ordering#57306

Closed
servo-wpt-sync wants to merge 1 commit intoweb-platform-tests:masterfrom
servo:servo_export_41973
Closed

script: Fix render blocking stylesheet load fire ordering#57306
servo-wpt-sync wants to merge 1 commit intoweb-platform-tests:masterfrom
servo:servo_export_41973

Conversation

@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

While adding various spec comments to the parts of #the-end (which are scattered throughout the script crate), I stumbled upon the render blocking stylesheets implementation. There was a HTML PR to make clear when it should run and the corresponding WPT tests have been fixed.

Fixes #22715
Reviewed in servo/servo#41973

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 Servo project.

These should cause it to fail, regardless of the Ok status. To do so,
we remove the `bool` and use the `Option` to convey success/failure.

It also realigns some of the spec text to now only do it for link
elements, since for styles we shouldn't be checking the result of
the content type.

Part of web-platform-tests#22715

Signed-off-by: Tim van der Lippe <[email protected]>
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator Author

👋 Downstream pull request (servo/servo#41973) no longer contains any upstreamable changes. Closing pull request without merging.

@servo-wpt-sync servo-wpt-sync deleted the servo_export_41973 branch January 26, 2026 18:54
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