Changeset 193390 in webkit

Timestamp:

Dec 3, 2015 4:49:48 PM (8 years ago)

Author:

commit-queue@webkit.org

Message:

Remove the use of GraphicsContextStateSaver from RenderLayer::paintLayerByApplyingTransform
​https://bugs.webkit.org/show_bug.cgi?id=151829

Patch by Said Abou-Hallawa <​sabouhallawa@apple.com> on 2015-12-03
Reviewed by Simon Fraser.

Set the CTM of the GraphicsContext to its original value before changing
it. This is a lot cheaper than using GraphicsContextStateSaver to save the
whole state of the GraphicsContext and restore it back;

• rendering/RenderLayer.cpp:

(WebCore::RenderLayer::paintLayerByApplyingTransform):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/RenderLayer.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2015-12-03  Said Abou-Hallawa  <sabouhallawa@apple.com>
+
+        Remove the use of GraphicsContextStateSaver from RenderLayer::paintLayerByApplyingTransform
+        https://bugs.webkit.org/show_bug.cgi?id=151829
+
+        Reviewed by Simon Fraser.
+
+        Set the CTM of the GraphicsContext to its original value before changing
+        it. This is a lot cheaper than using GraphicsContextStateSaver to save the
+        whole state of the GraphicsContext and restore it back;
+
+        * rendering/RenderLayer.cpp:
+        (WebCore::RenderLayer::paintLayerByApplyingTransform):
+
 2015-12-03  Eric Carlson  <eric.carlson@apple.com>
 

trunk/Source/WebCore/rendering/RenderLayer.cpp

@@ -4406 +4406 @@
     transform.translateRight(devicePixelSnappedOffsetForThisLayer.width(), devicePixelSnappedOffsetForThisLayer.height());
     // Apply the transform.
-    GraphicsContextStateSaver stateSaver(context);
+    AffineTransform oldTransfrom = context.getCTM();
     context.concatCTM(transform.toAffineTransform());
 
@@ -4414 +4414 @@
         paintingInfo.paintBehavior, adjustedSubpixelAccumulation, paintingInfo.subtreePaintRoot, paintingInfo.overlapTestRequests);
     paintLayerContentsAndReflection(context, transformedPaintingInfo, paintFlags);
+    context.setCTM(oldTransfrom);
 }