Changeset 140094 in webkit

Timestamp:

Jan 17, 2013 7:44:03 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

Update GraphicsContext to support winding rule in clip operator for QT
​https://bugs.webkit.org/show_bug.cgi?id=106873

Patch by Rik Cabanier <​cabanier@adobe.com> on 2013-01-17
Reviewed by Dirk Schulze.

Added support for winding to clip() and canvasClip() functions.

No new tests, no change in behavior.

• platform/graphics/qt/GraphicsContextQt.cpp:

(WebCore::GraphicsContext::clip): Use passed in winding rule.
(WebCore::GraphicsContext::canvasClip): Rename variable.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/qt/GraphicsContextQt.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-01-17  Rik Cabanier  <cabanier@adobe.com>
+
+        Update GraphicsContext to support winding rule in clip operator for QT
+        https://bugs.webkit.org/show_bug.cgi?id=106873
+
+        Reviewed by Dirk Schulze.
+
+        Added support for winding to clip() and canvasClip() functions.
+
+        No new tests, no change in behavior.
+
+        * platform/graphics/qt/GraphicsContextQt.cpp:
+        (WebCore::GraphicsContext::clip): Use passed in winding rule.
+        (WebCore::GraphicsContext::canvasClip): Rename variable.
+
 2013-01-17  Rik Cabanier  <cabanier@adobe.com>
 

trunk/Source/WebCore/platform/graphics/qt/GraphicsContextQt.cpp

@@ -1252 +1252 @@
 }
 
-// FIXME: don't ignore the winding rule. https://bugs.webkit.org/show_bug.cgi?id=106873
-void GraphicsContext::clip(const Path& path, WindRule)
+void GraphicsContext::clip(const Path& path, WindRule windRule)
 {
     if (paintingDisabled())
@@ -1259 +1258 @@
 
     QPainterPath clipPath = path.platformPath();
-    clipPath.setFillRule(Qt::WindingFill);
+    clipPath.setFillRule(toQtFillRule(windRule));
     m_data->p()->setClipPath(clipPath, Qt::IntersectClip);
 }
 
-void GraphicsContext::canvasClip(const Path& path, WindRule fillRule)
-{
-    clip(path, fillRule);
+void GraphicsContext::canvasClip(const Path& path, WindRule windRule)
+{
+    clip(path, windRule);
 }