Split pixel buffer format data out into a new PixelBufferFormat struct
Created attachment 428396 [details] Patch
Looks great, waiting to review until we have a version that’s a little more green on EWS.
Created attachment 428419 [details] Patch
Comment on attachment 428419 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=428419&action=review > Source/WebCore/platform/graphics/PixelBufferFormat.cpp:35 > + return ts << pixelBuffer.alphaFormat << " " << pixelBuffer.pixelFormat << " " << pixelBuffer.colorSpace; Faster if you use ' ' instead of " "? > Source/WebCore/platform/graphics/PixelBufferFormat.h:50 > + encoder << alphaFormat; > + encoder << pixelFormat; > + encoder << colorSpace; Use the one-line form? > Source/WebCore/platform/graphics/PixelFormat.h:40 > +WEBCORE_EXPORT WTF::TextStream& operator<<(WTF::TextStream&, PixelFormat); Don’t need WTF prefix.
Created attachment 428542 [details] Patch
Created attachment 428544 [details] Patch
Committed r277450 (237698@main): <https://commits.webkit.org/237698@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 428544 [details].
<rdar://problem/77985115>