Title: [94273] trunk/Source/WebCore
Revision
94273
Author
[email protected]
Date
2011-08-31 23:07:32 -0700 (Wed, 31 Aug 2011)

Log Message

Fix snow-leopard regression caused by r93982
https://bugs.webkit.org/show_bug.cgi?id=67301

Patch by Tom Zakrajsek <[email protected]> on 2011-08-31
Reviewed by Kenneth Russell.

Test: canvas/philip/tests/2d.path.stroke.prune.arc.html

* html/canvas/CanvasRenderingContext2D.cpp:
(WebCore::CanvasRenderingContext2D::arc):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (94272 => 94273)


--- trunk/Source/WebCore/ChangeLog	2011-09-01 05:56:41 UTC (rev 94272)
+++ trunk/Source/WebCore/ChangeLog	2011-09-01 06:07:32 UTC (rev 94273)
@@ -1,3 +1,15 @@
+2011-08-31  Tom Zakrajsek  <[email protected]>
+
+        Fix snow-leopard regression caused by r93982
+        https://bugs.webkit.org/show_bug.cgi?id=67301
+
+        Reviewed by Kenneth Russell.
+
+        Test: canvas/philip/tests/2d.path.stroke.prune.arc.html
+
+        * html/canvas/CanvasRenderingContext2D.cpp:
+        (WebCore::CanvasRenderingContext2D::arc):
+
 2011-08-31  Keishi Hattori  <[email protected]>
 
         Rename closeColorChooser to cleanupColorChooser

Modified: trunk/Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp (94272 => 94273)


--- trunk/Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp	2011-09-01 05:56:41 UTC (rev 94272)
+++ trunk/Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp	2011-09-01 06:07:32 UTC (rev 94273)
@@ -834,14 +834,8 @@
     }
 
     if (!r || sa == ea) {
-        // A subpath consisting of a single empty arc should draw nothing.
-        // However, an empty arc may comprise one point in a non-empty
-        // subpath, so do not ignore empty arcs entirely.
-        FloatPoint pt(x + r * cosf(sa), y + r * sinf(sa));
-        if (!m_path.hasCurrentPoint())
-            m_path.moveTo(pt);
-        else
-            m_path.addLineTo(pt);
+        // The arc is empty but we still need to draw the connecting line
+        lineTo(x + r * cosf(sa), y + r * sinf(sa));
         return;
     }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to