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

[Cocoa] CTFontCreateForCharactersWithLanguageAndOption() is available on all Cocoa platforms #8435

Conversation

litherum
Copy link
Contributor

@litherum litherum commented Jan 10, 2023

b81575a

[Cocoa] CTFontCreateForCharactersWithLanguageAndOption() is available on all Cocoa platforms
https://bugs.webkit.org/show_bug.cgi?id=250363
rdar://104063789

Reviewed by Tim Horton.

Simply unify the codepaths.

* Source/WTF/wtf/PlatformHave.h:
* Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:
(WebCore::preparePlatformFont):
(WebCore::createFontForCharacters):
(WebCore::FontCache::prewarm):
* Source/WebCore/platform/graphics/cocoa/FontDatabase.cpp:
(WebCore::FontDatabase::InstalledFont::InstalledFont):
(WebCore::FontDatabase::collectionForFamily):
(WebCore::FontDatabase::fontForPostScriptName):
* Source/WebCore/platform/graphics/cocoa/FontDatabase.h:

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

2f394a6

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios βœ… πŸ›  mac βœ… πŸ›  wpe   πŸ›  πŸ§ͺ win
βœ… πŸ§ͺ bindings βœ… πŸ›  ios-sim βœ… πŸ›  mac-AS-debug βœ… πŸ›  gtk βœ… πŸ›  wincairo
βœ… πŸ§ͺ webkitperl   πŸ§ͺ ios-wk2   πŸ§ͺ api-mac   πŸ§ͺ gtk-wk2
  πŸ§ͺ api-ios βœ… πŸ§ͺ mac-wk1 ⏳ πŸ§ͺ api-gtk
βœ… πŸ›  πŸ§ͺ jsc βœ… πŸ›  tv βœ… πŸ§ͺ mac-wk2 βœ… πŸ›  jsc-armv7
βœ… πŸ›  πŸ§ͺ jsc-arm64 βœ… πŸ›  tv-sim βœ… πŸ§ͺ mac-AS-debug-wk2 βœ… πŸ§ͺ jsc-armv7-tests
βœ… πŸ›  watch βœ… πŸ§ͺ mac-wk2-stress βœ… πŸ›  jsc-mips
βœ… πŸ›  πŸ§ͺ merge βœ… πŸ›  watch-sim βœ… πŸ§ͺ jsc-mips-tests

@litherum litherum self-assigned this Jan 10, 2023
@litherum litherum added the Text For bugs in text layout and rendering, including international text support. label Jan 10, 2023
Source/WTF/wtf/PlatformHave.h Outdated Show resolved Hide resolved
@litherum litherum force-pushed the eng/Cocoa-CTFontCreateForCharactersWithLanguageAndOption-is-available-on-all-Cocoa-platforms branch from d2e9fe3 to 2f394a6 Compare January 10, 2023 08:21
@litherum litherum added the merge-queue Applied to send a pull request to merge-queue label Jan 10, 2023
… on all Cocoa platforms

https://bugs.webkit.org/show_bug.cgi?id=250363
rdar://104063789

Reviewed by Tim Horton.

Simply unify the codepaths.

* Source/WTF/wtf/PlatformHave.h:
* Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:
(WebCore::preparePlatformFont):
(WebCore::createFontForCharacters):
(WebCore::FontCache::prewarm):
* Source/WebCore/platform/graphics/cocoa/FontDatabase.cpp:
(WebCore::FontDatabase::InstalledFont::InstalledFont):
(WebCore::FontDatabase::collectionForFamily):
(WebCore::FontDatabase::fontForPostScriptName):
* Source/WebCore/platform/graphics/cocoa/FontDatabase.h:

Canonical link: https://commits.webkit.org/258710@main
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/Cocoa-CTFontCreateForCharactersWithLanguageAndOption-is-available-on-all-Cocoa-platforms branch from 2f394a6 to b81575a Compare January 10, 2023 10:07
@webkit-commit-queue
Copy link
Collaborator

Committed 258710@main (b81575a): https://commits.webkit.org/258710@main

Reviewed commits have been landed. Closing PR #8435 and removing active labels.

@webkit-early-warning-system webkit-early-warning-system merged commit b81575a into WebKit:main Jan 10, 2023
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Text For bugs in text layout and rendering, including international text support.
Projects
None yet
4 participants