[webkit-changes] [WebKit/WebKit] dd90e2: Remote GraphicsContextGL buffer read pixels varian...
Kimmo Kinnunen
noreply at github.com
Fri May 26 00:12:48 PDT 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: dd90e25ef3b9630648b0203715d48c9c115a1cc4
https://github.com/WebKit/WebKit/commit/dd90e25ef3b9630648b0203715d48c9c115a1cc4
Author: Kimmo Kinnunen <kkinnunen at apple.com>
Date: 2023-05-26 (Fri, 26 May 2023)
Changed paths:
M Source/WebCore/html/canvas/WebGL2RenderingContext.cpp
M Source/WebCore/html/canvas/WebGLRenderingContextBase.cpp
M Source/WebCore/platform/graphics/GraphicsContextGL.h
M Source/WebCore/platform/graphics/angle/GraphicsContextGLANGLE.cpp
M Source/WebCore/platform/graphics/angle/GraphicsContextGLANGLE.h
M Source/WebKit/GPUProcess/graphics/RemoteGraphicsContextGL.cpp
M Source/WebKit/GPUProcess/graphics/RemoteGraphicsContextGL.h
M Source/WebKit/GPUProcess/graphics/RemoteGraphicsContextGL.messages.in
M Source/WebKit/GPUProcess/graphics/RemoteGraphicsContextGLFunctionsGenerated.h
M Source/WebKit/WebProcess/GPU/graphics/RemoteGraphicsContextGLProxy.cpp
M Source/WebKit/WebProcess/GPU/graphics/RemoteGraphicsContextGLProxy.h
M Source/WebKit/WebProcess/GPU/graphics/RemoteGraphicsContextGLProxyFunctionsGenerated.cpp
M Tools/Scripts/generate-gpup-webgl
M Tools/TestWebKitAPI/Tests/WebCore/cocoa/TestGraphicsContextGLCocoa.mm
Log Message:
-----------
Remote GraphicsContextGL buffer read pixels variant is not generated
https://bugs.webkit.org/show_bug.cgi?id=257319
rdar://109826254
Reviewed by Dan Glastonbury.
For legacy reasons, the read pixels to pixel buffer object was called
the same as read pixels to client memory. This prevented the
autogeneration.
Instead, rename the PBO variant as readPixelsBufferObject and let
the autogeneration generate it.
Rename other read pixels messages too.
Change the parameters to be more WebKit -like, e.g. int x,y,w,h should
be IntRect.
* Source/WebCore/html/canvas/WebGL2RenderingContext.cpp:
(WebCore::WebGL2RenderingContext::readPixels):
* Source/WebCore/html/canvas/WebGLRenderingContextBase.cpp:
(WebCore::WebGLRenderingContextBase::readPixels):
* Source/WebCore/platform/graphics/GraphicsContextGL.h:
* Source/WebCore/platform/graphics/angle/GraphicsContextGLANGLE.cpp:
(WebCore::GraphicsContextGLANGLE::readPixels):
(WebCore::GraphicsContextGLANGLE::readPixelsWithStatus):
(WebCore::GraphicsContextGLANGLE::readPixelsBufferObject):
(WebCore::GraphicsContextGLANGLE::readPixelsImpl):
(WebCore::GraphicsContextGLANGLE::readnPixels): Deleted.
(WebCore::GraphicsContextGLANGLE::readnPixelsWithStatus): Deleted.
(WebCore::GraphicsContextGLANGLE::readnPixelsImpl): Deleted.
* Source/WebCore/platform/graphics/angle/GraphicsContextGLANGLE.h:
* Source/WebKit/GPUProcess/graphics/RemoteGraphicsContextGL.cpp:
(WebKit::RemoteGraphicsContextGL::readPixelsInline):
(WebKit::RemoteGraphicsContextGL::readPixelsSharedMemory):
(WebKit::RemoteGraphicsContextGL::readnPixels0): Deleted.
(WebKit::RemoteGraphicsContextGL::readnPixels1): Deleted.
(WebKit::RemoteGraphicsContextGL::readnPixels2): Deleted.
* Source/WebKit/GPUProcess/graphics/RemoteGraphicsContextGL.h:
* Source/WebKit/GPUProcess/graphics/RemoteGraphicsContextGL.messages.in:
* Source/WebKit/GPUProcess/graphics/RemoteGraphicsContextGLFunctionsGenerated.h:
(readPixelsBufferObject):
* Source/WebKit/WebProcess/GPU/graphics/RemoteGraphicsContextGLProxy.cpp:
(WebKit::RemoteGraphicsContextGLProxy::readPixels):
(WebKit::RemoteGraphicsContextGLProxy::readPixelsSharedMemory):
(WebKit::RemoteGraphicsContextGLProxy::readnPixels): Deleted.
(WebKit::RemoteGraphicsContextGLProxy::readnPixelsSharedMemory): Deleted.
* Source/WebKit/WebProcess/GPU/graphics/RemoteGraphicsContextGLProxy.h:
* Source/WebKit/WebProcess/GPU/graphics/RemoteGraphicsContextGLProxyFunctionsGenerated.cpp:
(WebKit::RemoteGraphicsContextGLProxy::readPixelsBufferObject):
* Tools/Scripts/generate-gpup-webgl:
* Tools/TestWebKitAPI/Tests/WebCore/cocoa/TestGraphicsContextGLCocoa.mm:
(TestWebKitAPI::changeContextContents):
(TestWebKitAPI::TEST_P):
Canonical link: https://commits.webkit.org/264565@main
More information about the webkit-changes
mailing list