[SCM] WebKit Debian packaging branch, webkit-1.2, updated. upstream/1.1.90-6072-g9a69373

ariya at webkit.org ariya at webkit.org
Thu Apr 8 01:39:04 UTC 2010


The following commit has been merged in the webkit-1.2 branch:
commit 212502d098069e89ca37d9f8e38cbc72fffdf534
Author: ariya at webkit.org <ariya at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Thu Feb 4 17:43:17 2010 +0000

    2010-02-04  Ariya Hidayat  <ariya.hidayat at gmail.com>
    
            Reviewed by Simon Hausmann.
    
            [Qt] Unnecessary QBrush construction for doing a solid fill
            https://bugs.webkit.org/show_bug.cgi?id=34559
    
            Use the similar trick like r37421, i.e. use the special brush for
            solid color to avoid creating QBrush again and again.
    
            * platform/graphics/qt/GraphicsContextQt.cpp:
            (WebCore::GraphicsContext::setPlatformFillColor):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@54347 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index d466275..c6618e2 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,16 @@
+2010-02-04  Ariya Hidayat  <ariya.hidayat at gmail.com>
+
+        Reviewed by Simon Hausmann.
+
+        [Qt] Unnecessary QBrush construction for doing a solid fill
+        https://bugs.webkit.org/show_bug.cgi?id=34559
+
+        Use the similar trick like r37421, i.e. use the special brush for
+        solid color to avoid creating QBrush again and again.
+
+        * platform/graphics/qt/GraphicsContextQt.cpp:
+        (WebCore::GraphicsContext::setPlatformFillColor):
+
 2010-02-04  Dan Bernstein  <mitz at apple.com>
 
         Reviewed by Simon Fraser.
diff --git a/WebCore/platform/graphics/qt/GraphicsContextQt.cpp b/WebCore/platform/graphics/qt/GraphicsContextQt.cpp
index 9046449..938d7b2 100644
--- a/WebCore/platform/graphics/qt/GraphicsContextQt.cpp
+++ b/WebCore/platform/graphics/qt/GraphicsContextQt.cpp
@@ -1270,7 +1270,8 @@ void GraphicsContext::setPlatformFillColor(const Color& color, ColorSpace colorS
 {
     if (paintingDisabled())
         return;
-    m_data->p()->setBrush(QBrush(color));
+    m_data->solidColor.setColor(color);
+    m_data->p()->setBrush(m_data->solidColor);
 }
 
 void GraphicsContext::setPlatformShouldAntialias(bool enable)

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list