[webkit-changes] [WebKit/WebKit] 8a4347: @font-face src tech() should parse valid unsupport...
Vitor Roriz
noreply at github.com
Wed Jul 12 12:48:31 PDT 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 8a43470a0beb1a5aaaf7f6f0b94f8fe0ebf40dfb
https://github.com/WebKit/WebKit/commit/8a43470a0beb1a5aaaf7f6f0b94f8fe0ebf40dfb
Author: Vitor Roriz <vitor.roriz at apple.com>
Date: 2023-07-12 (Wed, 12 Jul 2023)
Changed paths:
M LayoutTests/imported/w3c/web-platform-tests/css/css-fonts/parsing/font-face-src-tech-expected.txt
M Source/WebCore/css/CSSFontFaceSrcValue.cpp
M Source/WebCore/css/parser/CSSPropertyParserHelpers.cpp
M Source/WebCore/css/parser/CSSPropertyParserWorkerSafe.cpp
M Source/WebCore/css/parser/CSSSupportsParser.cpp
Log Message:
-----------
@font-face src tech() should parse valid unsupported keywords
https://bugs.webkit.org/show_bug.cgi?id=259145
rdar://112135896
Reviewed by Myles C. Maxfield.
A valid tech() keyword argument should be parsed even if the represented
techonology is not supported by the engine. The engine should reject loading
the font for unsupported techonologies but this should happen only at
loading time and not at parsing time.
The reason why we were rejecting it before at parsing time it is for some
inconsistency in different parts of the spec which suggested that. That
will be fixed soon by the CWWG, see: https://github.com/w3c/csswg-drafts/issues/8793
* LayoutTests/imported/w3c/web-platform-tests/css/css-fonts/parsing/font-face-src-tech-expected.txt:
* Source/WebCore/css/CSSFontFaceSrcValue.cpp:
(WebCore::CSSFontFaceSrcResourceValue::fontLoadRequest):
* Source/WebCore/css/parser/CSSPropertyParserHelpers.cpp:
(WebCore::CSSPropertyParserHelpers::consumeFontTech):
* Source/WebCore/css/parser/CSSPropertyParserWorkerSafe.cpp:
(WebCore::CSSPropertyParserHelpersWorkerSafe::consumeFontFaceSrcURI):
* Source/WebCore/css/parser/CSSSupportsParser.cpp:
(WebCore::CSSSupportsParser::consumeSupportsFontTechFunction):
Canonical link: https://commits.webkit.org/265999@main
More information about the webkit-changes
mailing list