Skip to content
  • eric@webkit.org's avatar
    Reviewed by George Staikos. · d404a6ef
    eric@webkit.org authored
            Remove broken absoluteTransform() code from RenderSVGInlineText
            https://bugs.webkit.org/show_bug.cgi?id=25532
    
            This improved SVG text focus rings.
            This also improved SVG.getScreenCTM() results.
    
            SVGSVGElement::getScreenCTM was passing "true" for the second argument in
            localToAbsolute, which is "fixed" it meant to pass "true" for the
            third argument (useTransforms).  That change alone caused no layout test changes, however
            before I fixed this, it was hitting ASSERT(!fixed) in RenderSVGRoot::mapLocalToContainer
    
            mapLocalToContainer implementations really should be shared,
            but I'll do that in a later patch.
    
            * rendering/RenderSVGInlineText.cpp:
            (WebCore::RenderSVGInlineText::styleDidChange):
            (WebCore::RenderSVGInlineText::absoluteQuads):
            (WebCore::RenderSVGInlineText::computeRepaintRectForRange):
            (WebCore::RenderSVGInlineText::computeRepaintQuadForRange):
            * rendering/RenderSVGInlineText.h:
            * rendering/RenderSVGRoot.cpp:
            (WebCore::RenderSVGRoot::mapLocalToContainer):
            * rendering/RenderSVGRoot.h:
            * rendering/RenderSVGText.cpp:
            (WebCore::RenderSVGText::mapLocalToContainer):
            * rendering/RenderSVGText.h:
            * svg/SVGSVGElement.cpp:
            (WebCore::SVGSVGElement::getScreenCTM):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@43177 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    d404a6ef