Created attachment 110242 [details] test case summary: margin-right is ignored with vertical writing mode See test case.
Created attachment 110247 [details] Patch
Passes all the fast/writing-mode tests. Will put r? once I've verified it passes all the tests.
You don't need the RenderBlock helpers. RenderStyle has helpers already for this.
RenderStyle has Length marginStartUsing(const RenderStyle* otherStyle) const; Length marginEndUsing(const RenderStyle* otherStyle) const; Pass in the RenderBlock's style instead of the child's style, and it will give you the right answer. This will make the patch much smaller/simpler.
(In reply to comment #4) > RenderStyle has > > Length marginStartUsing(const RenderStyle* otherStyle) const; > Length marginEndUsing(const RenderStyle* otherStyle) const; > > Pass in the RenderBlock's style instead of the child's style, and it will give you the right answer. This will make the patch much smaller/simpler. ooh. that's much nicer.
Created attachment 110408 [details] Patch
Comment on attachment 110408 [details] Patch Clearing flags on attachment: 110408 Committed r97222: <http://trac.webkit.org/changeset/97222>
All reviewed patches have been landed. Closing bug.