[Webkit-unassigned] [Bug 57947] Split getBorderPaddingMargin into two functions

[bugzilla-daemon at webkit.org]

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





--- Comment #2 from Ryosuke Niwa <rniwa at webkit.org>  2011-04-06 08:12:52 PST ---
(From update of attachment 88425)
View in context: https://bugs.webkit.org/attachment.cgi?id=88425&action=review

> Source/WebCore/rendering/RenderBlockLineLayout.cpp:66
> +    // FIXME: Should we call marginStartForChild instead?
>      return child->marginStart() + child->paddingStart() + child->borderStart();

It's really odd that we don't call marginStartForChild here.  This is almost the only place we call marginStart instead.

Dave & Simon, this is correct?

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