Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fonts: Store web fonts in the per-Layout FontContext #46341

Merged

Conversation

servo-wpt-sync
Copy link
Collaborator

This moves mangement of web fonts to the per-Layout FontContext,
preventing web fonts from being available in different Documents.

Fixes #12920.

Co-authored-by: Mukilan Thiyagarajan mukilan@igalia.com
Signed-off-by: Martin Robinson mrobinson@igalia.com

Reviewed in servo/servo#32303

Copy link
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.

@servo-wpt-sync servo-wpt-sync force-pushed the servo_export_32303 branch 5 times, most recently from 79cca61 to f852e2d Compare May 20, 2024 12:08
This moves mangement of web fonts to the per-Layout `FontContext`,
preventing web fonts from being available in different Documents.

Fixes web-platform-tests#12920.

Co-authored-by: Mukilan Thiyagarajan <mukilan@igalia.com>
Signed-off-by: Martin Robinson <mrobinson@igalia.com>
@servo-wpt-sync servo-wpt-sync merged commit 13d894d into web-platform-tests:master May 20, 2024
19 checks passed
@servo-wpt-sync servo-wpt-sync deleted the servo_export_32303 branch May 20, 2024 15:16
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.

None yet

3 participants