[Webkit-unassigned] [Bug 78713] Absolute positioned elements with Inline Relative Positioned Container are not layout correctly

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Feb 16 16:06:20 PST 2012


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


Julien Chaffraix <jchaffraix at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #127186|review?, commit-queue?      |review-
               Flag|                            |




--- Comment #6 from Julien Chaffraix <jchaffraix at webkit.org>  2012-02-16 16:06:20 PST ---
(From update of attachment 127186)
View in context: https://bugs.webkit.org/attachment.cgi?id=127186&action=review

This is the wrong fix. RenderObject::container() should be patched not markContainingBlockForLayout().

The right fix (TM) is to find a way to merge container() and containingBlock() so that such bugs don't happen down the road again.

> Source/WebCore/ChangeLog:12
> +        This is a regression. r104183 changes containingBlock() so that it returns the container
> +        of an anonymous block for positioned objects, not the anonymous block itself. We should
> +        change markContainingBlocksForLayout() to match the change in containingBlock().

Your analysis is good but the conclusion is wrong. container() and containingBlock() should return the same result and here they don't.

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



More information about the webkit-unassigned mailing list