Changeset 94273 in webkit

Timestamp:

Aug 31, 2011 11:07:32 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

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

Patch by Tom Zakrajsek <​tomz@codeaurora.org> 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):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/canvas/CanvasRenderingContext2D.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-08-31  Tom Zakrajsek  <tomz@codeaurora.org>
+
+        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  <keishi@webkit.org>
 

trunk/Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp

@@ -835 +835 @@
 
     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;
     }