Move fitBorderToLinesIfNeeded and most of deleteLineBoxTree into RenderBlockFlow.

Move fitBorderToLinesIfNeeded and most of deleteLineBoxTree into RenderBlockFlow.

fitBorderToLinesIfNeeded can only be called on block flows, so it can be moved
altogether.

deleteLineBoxTree can have its logic moved, leaving (for now) only an ASSERT
declaring that it wouldn't have done anything anyway for now.

BUG=393945, 302024
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=178577

[leviw_travelin_and_unemployed, 2014-07-21 18:36:45]

LGTM.

https://codereview.chromium.org/404993002/diff/20001/Source/core/rendering/Re...
File Source/core/rendering/RenderBlock.cpp (right):

https://codereview.chromium.org/404993002/diff/20001/Source/core/rendering/Re...
Source/core/rendering/RenderBlock.cpp:975: ASSERT(!m_lineBoxes.firstLineBox());
Your subject line led me to believe this was going to completely move. I assume
it will in a subsequent patch?

[jbroman, 2014-07-21 18:41:37]

On 2014/07/21 18:36:45, leviw wrote:
> LGTM.
> 
>
https://codereview.chromium.org/404993002/diff/20001/Source/core/rendering/Re...
> File Source/core/rendering/RenderBlock.cpp (right):
> 
>
https://codereview.chromium.org/404993002/diff/20001/Source/core/rendering/Re...
> Source/core/rendering/RenderBlock.cpp:975:
ASSERT(!m_lineBoxes.firstLineBox());
> Your subject line led me to believe this was going to completely move. I
assume
> it will in a subsequent patch?

I'm hoping it can, but at the moment it's easier to have it remain virtual (and
leaving the ASSERT gives some peace of mind).

Adjusting subject to be less misleading.

[jbroman, 2014-07-21 18:41:40]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2014-07-21 18:42:22]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jbroman@chromium.org/404993002/20001

[commit-bot: I haz the power, 2014-07-21 19:18:56]

Change committed as 178577