Changeset 284007 in webkit

Timestamp:

Oct 12, 2021, 12:51:26 PM (5 years ago)

Author:

commit-queue@webkit.org

Message:

Cocoa GraphicsContextGLOpenGL uses depth and stencil for pbuffers on ANGLE Metal
​https://bugs.webkit.org/show_bug.cgi?id=231424

Patch by Kimmo Kinnunen <​kkinnunen@apple.com> on 2021-10-12
Reviewed by Dean Jackson.

ANGLE Metal allocates depth and stencil buffers when a pbuffer is bound to
a texture via eglBindTexImage.
To side-step this performance issue, allocate the gl context with EGLConfig
that disables depth and stencil. The config is used for the video texture
upload context, video texture upload pbuffers as well as other pbuffers.

• platform/graphics/cocoa/GraphicsContextGLOpenGLCocoa.mm:

(WebCore::GraphicsContextGLOpenGL::GraphicsContextGLOpenGL):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/cocoa/GraphicsContextGLOpenGLCocoa.mm (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2021-10-12  Kimmo Kinnunen  <kkinnunen@apple.com>
+
+        Cocoa GraphicsContextGLOpenGL uses depth and stencil for pbuffers on ANGLE Metal
+        https://bugs.webkit.org/show_bug.cgi?id=231424
+
+        Reviewed by Dean Jackson.
+
+        ANGLE Metal allocates depth and stencil buffers when a pbuffer is bound to
+        a texture via eglBindTexImage.
+        To side-step this performance issue, allocate the gl context with EGLConfig
+        that disables depth and stencil. The config is used for the video texture
+        upload context, video texture upload pbuffers as well as other pbuffers.
+
+        * platform/graphics/cocoa/GraphicsContextGLOpenGLCocoa.mm:
+        (WebCore::GraphicsContextGLOpenGL::GraphicsContextGLOpenGL):
+
 2021-10-12  Tim Horton  <timothy_horton@apple.com>
 

trunk/Source/WebCore/platform/graphics/cocoa/GraphicsContextGLOpenGLCocoa.mm

@@ -259 +259 @@
         EGL_BLUE_SIZE, 8,
         EGL_ALPHA_SIZE, 8,
+        EGL_DEPTH_SIZE, 0,
+        EGL_STENCIL_SIZE, 0,
         EGL_NONE
     };