Changeset 69462 in webkit

Timestamp:

Oct 9, 2010 4:56:31 PM (14 years ago)

Author:

andreas.kling@nokia.com

Message:

2010-10-09 Andreas Kling <​kling@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Fast-path in Path::clear() for already empty paths
​https://bugs.webkit.org/show_bug.cgi?id=47460

The only way to clear a QPainterPath is to assign QPainterPath() to it.
Avoid creating a temporary object etc if the path is already empty.

• platform/graphics/qt/PathQt.cpp: (WebCore::Path::clear):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/qt/PathQt.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-10-09  Andreas Kling  <kling@webkit.org>
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        [Qt] Fast-path in Path::clear() for already empty paths
+        https://bugs.webkit.org/show_bug.cgi?id=47460
+
+        The only way to clear a QPainterPath is to assign QPainterPath() to it.
+        Avoid creating a temporary object etc if the path is already empty.
+
+        * platform/graphics/qt/PathQt.cpp:
+        (WebCore::Path::clear):
+
 2010-10-08  Ryosuke Niwa  <rniwa@webkit.org>
 

trunk/WebCore/platform/graphics/qt/PathQt.cpp

@@ -336 +336 @@
 void Path::clear()
 {
+    if (!m_path.elementCount())
+        return;
     m_path = QPainterPath();
 }