DescriptionMake mapLocalToContainer() work properly for content that follows a column spanner.
mapLocalToContainer() uses offsetFromContainer(), which uses columnOffset()
(which calls flowThreadTranslationAtOffset()), which didn't always return the
correct translation relatively to the flow thread.
This also improves the fix for bug 479155.
MultiColumnFragmentainerGroup::flowThreadTranslationAtOffset() needs to return
a translation relative to the flow thread, and the code we had basically just
happened to work by accident (flipping a flow thread portion rectangle for
writing mode around a column set is just bogus, since a column set isn't
sized in the flow thread coordinate space). This solution would cause problems
with the upcoming fix for bug 499706, because then the last column may be
shorter than the others (because it's not completely filled).
BUG=501772
R=dsinclair@chromium.org,jchaffraix@chromium.org
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=197373
Patch Set 1 #Messages
Total messages: 5 (1 generated)
|