Bug 91168 – REGRESSION: RenderInline boundingBox ignores relative position offset

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Kiran Muppala]

RenderInline::absoluteQuads was optimized to use a RenderGeometryMap for caching the transform for local to absolute coordinate conversion by https://bugs.webkit.org/show_bug.cgi?id=90715.  But since that change, the bounding box returned is not adding the relative position offset of the inline
element itself, if any.  This leads to assertion failures when opening several webpages, including https://developer.mozilla.org/en/DOM/element.getBoundingClientRect.

The root cause for this issue is that the GeometryMap expects the first mapping pushed to it be that of the RenderView, whereas RenderInline::absoluteQuads
is pushing it's own mapping first, leading to the observed discrepancy.

[Kiran Muppala]

<rdar://problem/11845860>

[Kiran Muppala]

Created attachment 152127 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 152127 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=152127&action=review

> Source/WebCore/rendering/RenderInline.cpp:660
> +            m_geometryMap.pushMappingsToAncestor(view, 0);
> +            m_geometryMap.pushMappingsToAncestor(renderer, view);
> +        }

Does m_geometryMap.pushMappingsToAncestor(renderer, 0); not work? If so, we should just fix RenderGeometryMap so that it does.

[Kiran Muppala]

(In reply to comment #3)
> (From update of attachment 152127 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=152127&action=review
> 
> > Source/WebCore/rendering/RenderInline.cpp:660
> > +            m_geometryMap.pushMappingsToAncestor(view, 0);
> > +            m_geometryMap.pushMappingsToAncestor(renderer, view);
> > +        }
> 
> Does m_geometryMap.pushMappingsToAncestor(renderer, 0); not work? If so, we should just fix RenderGeometryMap so that it does.

Yes, calling m_geometryMap.pushMappingsToAncestor(renderer, 0) results in the renderer's mapping pushed first, which is exactly what is happening currently.  In fact the assertion, ASSERT(!m_mapping.size()), in RenderGeometryMap::pushView fails for such a call.

[Kiran Muppala]

Created attachment 152351 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 152351 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=152351&action=review

> LayoutTests/fast/inline/inline-relative-offset-boundingbox.html:14
> +            parentRect = parent.getBoundingClientRect();
> +	    shouldBe("inlineRect.left", "parentRect.left + inlineLeftOffset");

Indentation looks wrong here.

[Kiran Muppala]

Created attachment 152364 [details]
Patch

[Kiran Muppala]

(In reply to comment #6)
> (From update of attachment 152351 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=152351&action=review
> 
> > LayoutTests/fast/inline/inline-relative-offset-boundingbox.html:14
> > +            parentRect = parent.getBoundingClientRect();
> > +	    shouldBe("inlineRect.left", "parentRect.left + inlineLeftOffset");
> 
> Indentation looks wrong here.

My bad, there was a tab in the second line.  I replaced it with spaces and that fixed the indentation.

Uploaded a new patch.  Only change is the fix for indentation.

[WebKit Review Bot]

Comment on attachment 152364 [details]
Patch

Clearing flags on attachment: 152364

Committed r122653: <http://trac.webkit.org/changeset/122653>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.