Changeset 292488 in webkit

Timestamp:

Apr 6, 2022 12:06:37 PM (2 years ago)

Author:

pvollan@apple.com

Message:

Avoid calling IOSurfaceAlignProperty in the WebContent process
​https://bugs.webkit.org/show_bug.cgi?id=238870
<rdar://91252359>

Reviewed by Geoffrey Garen.

Use cached values for bytes per row alignment.

• platform/graphics/cg/ImageBufferIOSurfaceBackend.cpp:

(WebCore::ImageBufferIOSurfaceBackend::calculateBytesPerRow):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/cg/ImageBufferIOSurfaceBackend.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2022-04-06  Per Arne Vollan  <pvollan@apple.com>
+
+        Avoid calling IOSurfaceAlignProperty in the WebContent process
+        https://bugs.webkit.org/show_bug.cgi?id=238870
+        <rdar://91252359>
+
+        Reviewed by Geoffrey Garen.
+
+        Use cached values for bytes per row alignment.
+
+        * platform/graphics/cg/ImageBufferIOSurfaceBackend.cpp:
+        (WebCore::ImageBufferIOSurfaceBackend::calculateBytesPerRow):
+
 2022-04-06  Chris Dumez  <cdumez@apple.com>
 

trunk/Source/WebCore/platform/graphics/cg/ImageBufferIOSurfaceBackend.cpp

@@ -30 +30 @@
 
 #include "GraphicsContextCG.h"
+#include "IOSurface.h"
 #include "IOSurfacePool.h"
 #include "IntRect.h"
@@ -58 +59 @@
 {
     unsigned bytesPerRow = ImageBufferCGBackend::calculateBytesPerRow(backendSize);
-    return IOSurfaceAlignProperty(kIOSurfaceBytesPerRow, bytesPerRow);
+    size_t alignmentMask = IOSurface::bytesPerRowAlignment() - 1;
+    return (bytesPerRow + alignmentMask) & ~alignmentMask;
 }