Changeset 88321 in webkit

Timestamp:

Jun 7, 2011 8:31:26 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

2011-06-07 Ian Henderson <​ianh@apple.com>

Reviewed by Dan Bernstein.

(CG) Sub-image drawing clips unnecessarily
​https://bugs.webkit.org/show_bug.cgi?id=62258

If destRect completely encompasses the adjustedDestRect into which we
draw, clipping to destRect will do nothing. Skip the call to
CGContextClipToRect in this case.

• platform/graphics/cg/ImageCG.cpp: (WebCore::BitmapImage::draw):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/cg/ImageCG.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-06-07  Ian Henderson  <ianh@apple.com>
+
+        Reviewed by Dan Bernstein.
+
+        (CG) Sub-image drawing clips unnecessarily
+        https://bugs.webkit.org/show_bug.cgi?id=62258
+
+        If destRect completely encompasses the adjustedDestRect into which we
+        draw, clipping to destRect will do nothing.  Skip the call to
+        CGContextClipToRect in this case.
+
+        * platform/graphics/cg/ImageCG.cpp:
+        (WebCore::BitmapImage::draw):
+
 2011-06-07  Emil A Eklund  <eae@chromium.org>
 

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

@@ -242 +242 @@
         }
 
-        CGContextClipToRect(context, destRect);
+        if (!destRect.contains(adjustedDestRect))
+            CGContextClipToRect(context, destRect);
     }