Changeset 78751 in webkit

Timestamp:

Feb 16, 2011 4:50:24 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

2011-02-16 Robin Cao <​robin.cao@torchmobile.com.cn>

Reviewed by James Robinson.

PlatformContextSkia::applyAntiAliasedClipPaths does not work for paths which have evenOdd property
​https://bugs.webkit.org/show_bug.cgi?id=54336

We need to take fill type of paths into account when drawing them.

No new tests, covered by svg/W3C-SVG-1.1/masking-path-05-f.svg.

• platform/graphics/skia/PlatformContextSkia.cpp: (WebCore::PlatformContextSkia::applyAntiAliasedClipPaths):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/skia/PlatformContextSkia.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-02-16  Robin Cao  <robin.cao@torchmobile.com.cn>
+
+        Reviewed by James Robinson.
+
+        PlatformContextSkia::applyAntiAliasedClipPaths does not work for paths which have evenOdd property
+        https://bugs.webkit.org/show_bug.cgi?id=54336
+
+        We need to take fill type of paths into account when drawing them.
+
+        No new tests, covered by svg/W3C-SVG-1.1/masking-path-05-f.svg.
+
+        * platform/graphics/skia/PlatformContextSkia.cpp:
+        (WebCore::PlatformContextSkia::applyAntiAliasedClipPaths):
+
 2011-02-16  Eric Seidel  <eric@webkit.org>
 

trunk/Source/WebCore/platform/graphics/skia/PlatformContextSkia.cpp

@@ -689 +689 @@
 
     for (size_t i = paths.size() - 1; i < paths.size(); --i) {
-        paths[i].setFillType(SkPath::kInverseWinding_FillType);
+        paths[i].toggleInverseFillType();
         m_canvas->drawPath(paths[i], paint);
     }