DescriptionCleanup: Get rid of LayoutBlock::offsetFromLogicalTopOfFirstPage().
Fold its contents into LayoutBox::offsetFromLogicalTopOfFirstPage() and make
that one non-virtual. Some non-blocks (e.g. table rows and sections) do
establish layout states, so we cannot just unconditionally jump to the
containing block.
Currently offsetFromLogicalTopOfFirstPage() is never called on non-blocks (in
other words, the LayoutBox implementation of the method just sat there doing
nothing but contribute to the binary size), which explains why we had no bugs
because of this. I expect that non-blocks will call this method in the near
future, though.
R=jchaffraix@chromium.org,leviw@chromium.org
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=195709
Patch Set 1 #
Messages
Total messages: 5 (1 generated)
|