Branch: refs/heads/main Home: https://github.com/WebKit/WebKit Commit: 8a43470a0beb1a5aaaf7f6f0b94f8fe0ebf40dfb https://github.com/WebKit/WebKit/commit/8a43470a0beb1a5aaaf7f6f0b94f8fe0ebf40dfb Author: Vitor Roriz <vitor.ro...@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 _______________________________________________ webkit-changes mailing list webkit-changes@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-changes