Bug 216448

Summary: [Apple Win] Migrate from FontPlatformDataWin.cpp to FontPlatformDataCoreText.cpp
Product: WebKit Reporter: Myles C. Maxfield <mmaxfield>
Component: TextAssignee: Myles C. Maxfield <mmaxfield>
Status: RESOLVED FIXED    
Severity: Normal CC: darin, dbarton, dino, ews-watchlist, fred.wang, Hironori.Fujii, jonlee, mmaxfield, simon.fraser, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 216353    
Attachments:
Description Flags
WIP
none
WIP
none
Patch
none
WIP
none
Patch
ews-feeder: commit-queue-
Patch
ews-feeder: commit-queue-
Patch
ews-feeder: commit-queue-
Patch
ews-feeder: commit-queue-
WIP
none
Patch
none
Patch darin: review+

Description Myles C. Maxfield 2020-09-12 15:55:02 PDT
This is one of the steps on the way to https://bugs.webkit.org/show_bug.cgi?id=216353 [Apple Win] Migrate the Apple Win port's font code to Core Text from Core Graphics
Comment 1 Myles C. Maxfield 2020-09-12 15:55:30 PDT
Created attachment 408620 [details]
WIP
Comment 2 Myles C. Maxfield 2020-09-12 15:59:09 PDT
Created attachment 408621 [details]
WIP
Comment 3 Myles C. Maxfield 2020-09-12 16:19:35 PDT
Created attachment 408623 [details]
Patch
Comment 4 Myles C. Maxfield 2020-09-12 16:24:32 PDT
Comment on attachment 408623 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=408623&action=review

> Source/WebCore/ChangeLog:8
> +        This is one of the steps toward migrating the Apple Win port off of Core Graphics and on to

Not off Core Graphics itself, but off of CGFont.
Comment 5 Myles C. Maxfield 2020-09-12 18:24:01 PDT
Looks like this requires createFontCustomPlatformData() to be migrated first.
Comment 6 Myles C. Maxfield 2020-09-12 20:32:30 PDT
Created attachment 408632 [details]
WIP
Comment 7 Myles C. Maxfield 2020-09-12 20:52:25 PDT
Created attachment 408635 [details]
Patch
Comment 8 Myles C. Maxfield 2020-09-12 20:56:25 PDT
Created attachment 408636 [details]
Patch
Comment 9 Myles C. Maxfield 2020-09-12 21:02:09 PDT
Created attachment 408637 [details]
Patch
Comment 10 Myles C. Maxfield 2020-09-12 21:38:52 PDT
Created attachment 408639 [details]
Patch
Comment 11 Myles C. Maxfield 2020-09-14 01:52:47 PDT
Created attachment 408690 [details]
WIP
Comment 12 Myles C. Maxfield 2020-09-14 01:56:42 PDT
Created attachment 408691 [details]
Patch
Comment 13 Myles C. Maxfield 2020-09-14 13:52:47 PDT
Created attachment 408744 [details]
Patch
Comment 14 Darin Adler 2020-09-14 17:31:37 PDT
Comment on attachment 408744 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=408744&action=review

> Source/WebCore/platform/graphics/win/FontCustomPlatformData.cpp:117
> +    auto bufferData = buffer.createCFData();
> +    result->fontDescriptor = adoptCF(CTFontManagerCreateFontDescriptorFromData(bufferData.get()));

Reads better without a local variable, I think.
Comment 15 Myles C. Maxfield 2020-09-14 21:19:31 PDT
Committed r267073: <https://trac.webkit.org/changeset/267073>
Comment 16 Radar WebKit Bug Importer 2020-09-14 21:20:15 PDT
<rdar://problem/68898486>
Comment 17 Fujii Hironori 2020-09-14 22:57:56 PDT
Committed r267077: <https://trac.webkit.org/changeset/267077>