RESOLVED FIXED 227575
[Cairo] Simplify GraphicsContextCairo creation 
https://bugs.webkit.org/show_bug.cgi?id=227575
Summary [Cairo] Simplify GraphicsContextCairo creation
Carlos Garcia Campos
Reported 2021-07-01 02:14:09 PDT
Remove the constructors taking a PlatformContextCairo and add two that receive a RefPtr<cairo_t>&& and cairo_surface_t*. In both cases the PlatformContextCairo is created, so it's now always owned and callers don't need to create it.
Attachments
Patch (18.37 KB, patch)
2021-07-01 02:16 PDT, Carlos Garcia Campos 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (18.69 KB, patch)
2021-07-01 02:41 PDT, Carlos Garcia Campos 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (18.61 KB, patch)
2021-07-01 02:50 PDT, Carlos Garcia Campos 		zdobersek: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Carlos Garcia Campos
Comment 1 2021-07-01 02:16:42 PDT
Created attachment 432670 [details] Patch
Carlos Garcia Campos
Comment 2 2021-07-01 02:41:44 PDT
Created attachment 432672 [details] Patch
Carlos Garcia Campos
Comment 3 2021-07-01 02:50:48 PDT
Created attachment 432673 [details] Patch
Carlos Garcia Campos
Comment 4 2021-07-01 03:35:16 PDT
Committed r279460 (239316@main): <https://commits.webkit.org/239316@main>
Radar WebKit Bug Importer
Comment 5 2021-07-01 03:36:17 PDT
<rdar://problem/80015368>
Fujii Hironori
Comment 6 2021-07-05 22:55:12 PDT
Comment on attachment 432673 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=432673&action=review Very nice refactoring. We no longer need PlatformContextCairo. We can merge PlatformContextCairo into GraphicsContextCairo. > Source/WebCore/platform/graphics/win/GraphicsContextCairoWin.cpp:76 > + : GraphicsContextCairo(platformContext->cr()) Umm, this is not same with the original code. But, I think no problem.
Fujii Hironori
Comment 7 2021-07-06 14:30:54 PDT
(In reply to Fujii Hironori from comment #6) > Very nice refactoring. We no longer need PlatformContextCairo. We can merge > PlatformContextCairo into GraphicsContextCairo. Filed: Bug 227721 – [Cairo] Merge PlatformContextCairo into GraphicsContextCairo
Note You need to log in before you can comment on or make changes to this bug.