[Webkit-unassigned] [Bug 34366] [OpenVG] Implement support for paths

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=34366


Dirk Schulze <krit at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #48904|review?                     |review-
               Flag|                            |




--- Comment #7 from Dirk Schulze <krit at webkit.org>  2010-02-25 12:52:06 PST ---
(From update of attachment 48904)
> +PlatformPathOpenVG::PlatformPathOpenVG(const PlatformPathOpenVG& other)
> +    : SharedResourceOpenVG()
> +    , m_currentPoint(other.m_currentPoint)
> +    , m_subpathStartPoint(other.m_subpathStartPoint)
> +{
> +    createPath();
> +    // makeCompatibleContextCurrent(); // called by createPath(), not necessary
> +    vgAppendPath(m_vgPath, other.m_vgPath);
> +    ASSERT_VG_NO_ERROR();
> +}

Either delete the comments or, if you think it is necessary for the
explanation, use two lines.


> +void Path::transform(const TransformationMatrix& transform)
> +{
> +    PlatformPathOpenVG* dst = new PlatformPathOpenVG();
> +    // dst->makeCompatibleContextCurrent(); // done by platform path constructor
> +    PainterOpenVG::transformPath(dst->vgPath(), m_path->vgPath(), transform);
> +    delete m_path;
> +    m_path = dst;
> +
> +    m_path->m_currentPoint = transform.mapPoint(m_path->m_currentPoint);
> +    m_path->m_subpathStartPoint = transform.mapPoint(m_path->m_subpathStartPoint);
> +}

same here

You should also make a explanation why Path::isEmpty is also empty for a
moveTo, either with a link to the HTML 5 spec or technical reasons. The current
implementation differs from other platforms.

I realy would like to see AffineTransform instead of TransformationMatrix.

r- mainly because of TransformationMatrix and Path:isEmpty.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.