[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc

andreas.kling at nokia.com andreas.kling at nokia.com
Wed Dec 22 13:29:25 UTC 2010


The following commit has been merged in the debian/experimental branch:
commit d9cebc2e777d89b78c97c5627d22bc3beb9dba19
Author: andreas.kling at nokia.com <andreas.kling at nokia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Thu Sep 16 22:35:30 2010 +0000

    2010-09-16  Andreas Kling  <andreas.kling at nokia.com>
    
            Reviewed by Ariya Hidayat.
    
            [Qt] Remove unnecessary QPainter::clipPath() call in TransparencyLayer()
            https://bugs.webkit.org/show_bug.cgi?id=45923
    
            QPainter::clipPath() can be very expensive, so use QPainter::hasClipping()
            instead of QPainter::clipPath().isEmpty().
    
            * platform/graphics/qt/TransparencyLayer.h:
            (WebCore::TransparencyLayer::TransparencyLayer):
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@67675 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index d369cea..6d2243e 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,16 @@
+2010-09-16  Andreas Kling  <andreas.kling at nokia.com>
+
+        Reviewed by Ariya Hidayat.
+
+        [Qt] Remove unnecessary QPainter::clipPath() call in TransparencyLayer()
+        https://bugs.webkit.org/show_bug.cgi?id=45923
+
+        QPainter::clipPath() can be very expensive, so use QPainter::hasClipping()
+        instead of QPainter::clipPath().isEmpty().
+
+        * platform/graphics/qt/TransparencyLayer.h:
+        (WebCore::TransparencyLayer::TransparencyLayer):
+
 2010-09-16  David Hyatt  <hyatt at apple.com>
 
         Reviewed by Eric Seidel.
diff --git a/WebCore/platform/graphics/qt/TransparencyLayer.h b/WebCore/platform/graphics/qt/TransparencyLayer.h
index 1a614ac..73ad9c6 100644
--- a/WebCore/platform/graphics/qt/TransparencyLayer.h
+++ b/WebCore/platform/graphics/qt/TransparencyLayer.h
@@ -61,8 +61,7 @@ struct TransparencyLayer : FastAllocBase {
         painter.setFont(p->font());
         if (painter.paintEngine()->hasFeature(QPaintEngine::PorterDuff))
             painter.setCompositionMode(p->compositionMode());
-        // if the path is an empty region, this assignment disables all painting
-        if (!p->clipPath().isEmpty())
+        if (p->hasClipping())
             painter.setClipPath(p->clipPath());
     }
 

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list