Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c58996cc188d1ca5f11283c6c6d078ec7d4583fa
      
https://github.com/WebKit/WebKit/commit/c58996cc188d1ca5f11283c6c6d078ec7d4583fa
  Author: Vitor Roriz <[email protected]>
  Date:   2025-12-23 (Tue, 23 Dec 2025)

  Changed paths:
    M LayoutTests/fast/text/font-face-api-preinstalled.html
    M LayoutTests/fast/text/font-face-set-document.html
    M LayoutTests/fast/text/font-face-set-javascript.html
    M LayoutTests/fast/text/font-loading-multiple-sets.html
    R 
LayoutTests/imported/w3c/web-platform-tests/css/css-font-loading/fontfaceset-add-css-connected-expected.txt
    R 
LayoutTests/imported/w3c/web-platform-tests/css/css-font-loading/fontfaceset-add-css-connected.html
    M Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml
    M Source/WTF/wtf/cocoa/RuntimeApplicationChecksCocoa.h
    M Source/WebCore/css/FontFaceSet.idl
    M Source/WebKit/Shared/WebPreferencesDefaultValues.cpp
    M Source/WebKit/Shared/WebPreferencesDefaultValues.h
    M Source/WebKitLegacy/mac/WebView/WebPreferencesDefaultValues.h
    M Source/WebKitLegacy/mac/WebView/WebPreferencesDefaultValues.mm

  Log Message:
  -----------
  CSS Font Loading API: Remove FontFaceSet constructor
https://bugs.webkit.org/show_bug.cgi?id=276794
rdar://132031306

Reviewed by Simon Fraser.

The FontFaceSet constructor was considered not useful
and is to be removed from the FontFaceSet interface
as per [1] resolution.

[1] https://github.com/w3c/csswg-drafts/issues/10390

* LayoutTests/fast/text/font-face-api-preinstalled.html:
* LayoutTests/fast/text/font-face-set-document-expected.txt:
* LayoutTests/fast/text/font-face-set-document.html:
* LayoutTests/fast/text/font-face-set-javascript.html:
* LayoutTests/fast/text/font-loading-multiple-sets.html:
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-font-loading/fontfaceset-add-css-connected-expected.txt:
 Removed.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-font-loading/fontfaceset-add-css-connected.html:
 Removed.
  Remove fontfaceset-add-css-connected.html as it fundamentally requires the 
FontFaceSet constructor to function. The test verifies that CSS-connected 
FontFace objects cannot be added to a separate FontFaceSet instance, but 
without the constructor, there is no way to create such a separate instance in 
a document context. This test scenario becomes impossible once the constructor 
is removed per the CSSWG resolution mentioned above [1].

* Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml:
* Source/WTF/wtf/cocoa/RuntimeApplicationChecksCocoa.h:
* Source/WebCore/css/FontFaceSet.idl:
* Source/WebKit/Shared/WebPreferencesDefaultValues.cpp:
(WebKit::defaultFontFaceSetConstructorEnabled):
* Source/WebKit/Shared/WebPreferencesDefaultValues.h:
* Source/WebKitLegacy/mac/WebView/WebPreferencesDefaultValues.h:
* Source/WebKitLegacy/mac/WebView/WebPreferencesDefaultValues.mm:
(WebKit::defaultFontFaceSetConstructorEnabled):

Canonical link: https://commits.webkit.org/304912@main



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications

Reply via email to