Bug 33115 – PathElementAddQuadCurveToPoint not handled for <marker>

• Component: SVG
• Status: NEW

[Nikolas Zimmermann]

Marker elements don't support paths which contain PathElementAddQuadCurveToPoint elements. See SVGMarkerLayoutInfo, updateMarkerDataForElement() method.

[Ahmad Saleem]

Following Blink commit: https://chromium.googlesource.com/chromium/src.git/+/7003bed16de3861e47d6bb0287ba4d6bffcfea04

Removed this FIXME:

https://searchfox.org/wubkat/source/Source/WebCore/rendering/svg/SVGMarkerData.h#118

and changed it into:

case PathElement::Type::AddQuadCurveToPoint:
            m_inslopePoints[0] = points[0];
            m_inslopePoints[1] = points[1];
            m_origin = points[1];
            break;

[Ahmad Saleem]

case PathElement::Type::AddQuadCurveToPoint:
    // Compute tangents for quadratic bezier curve
    // Start tangent: from origin to control point
    // End tangent: from control point to end point
    {
        FloatSize startTangent = points[0] - m_origin;
        FloatSize endTangent = points[1] - points[0];
        
        if (startTangent.isZero()) {
            // If start tangent is zero, use the end tangent for both
            m_inslopePoints[0] = m_origin;
            m_inslopePoints[1] = points[1];
        } else if (endTangent.isZero()) {
            // If end tangent is zero, use the start tangent for both
            m_inslopePoints[0] = m_origin;
            m_inslopePoints[1] = points[0];
        } else {
            // Normal case: use control point for in-slope calculation
            m_inslopePoints[0] = points[0];
            m_inslopePoints[1] = points[1];
        }
        
        m_origin = points[1];
    }
    break;