RESOLVED FIXED Bug 233491
GraphicsContextGL implementations should have separate class declarations 
https://bugs.webkit.org/show_bug.cgi?id=233491
Summary GraphicsContextGL implementations should have separate class declarations
Kimmo Kinnunen
Reported 2021-11-25 03:29:54 PST
GraphicsContextGL implementations should have separate class definitions The extensive use of #ifdef and unmaintained code make the Cocoa variant hard to develop.
Attachments
Patch (138.77 KB, patch)
2021-11-25 03:38 PST, Kimmo Kinnunen 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (139.15 KB, patch)
2021-11-25 03:45 PST, Kimmo Kinnunen 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Kimmo Kinnunen
Comment 1 2021-11-25 03:38:39 PST
Created attachment 445132 [details] Patch
Kimmo Kinnunen
Comment 2 2021-11-25 03:45:19 PST
Created attachment 445133 [details] Patch
Antti Koivisto
Comment 3 2021-11-25 05:08:04 PST
Comment on attachment 445133 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=445133&action=review > Source/WebCore/platform/graphics/cocoa/GraphicsContextGLCocoa.mm:223 > +// FIXME: Below is functionality that should be moved to GraphicsContextGLCocoa to simplify the base class. > + > +GraphicsContextGLOpenGL::GraphicsContextGLOpenGL(GraphicsContextGLAttributes attrs) Hopefully soonish!
EWS
Comment 4 2021-11-25 05:22:49 PST
Committed r286167 (244550@main): <https://commits.webkit.org/244550@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 445133 [details].
Radar WebKit Bug Importer
Comment 5 2021-11-25 05:23:26 PST
<rdar://problem/85749766>
Note You need to log in before you can comment on or make changes to this bug.