Replace layout with updateLayout during dumping LayoutTree

Replace layout with updateLayout during dumping LayoutTree

BUG=None

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=193144

[qiankun, 2015-04-03 16:30:06]

qiankun.miao@intel.com changed reviewers:
+ esprehn@chromium.org, leviw@chromium.org

[qiankun, 2015-04-03 16:30:07]

This CL is driven by https://codereview.chromium.org/999033002/#msg24. For
fast/css/resize-corner-tracking.html, it would trigger a layout when doing
LayoutTreeAsText::writeLayers which may be not needed. This CL always checks if
needLayout() before really do layout.

[esprehn, 2015-04-03 16:45:19]

You want to do view->document()->updateLayout().

The previous unconditional call also updated style, which may mark the
layout dirty bit.

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[qiankun, 2015-04-03 17:12:16]

On 2015/04/03 16:45:19, esprehn wrote:
> You want to do view->document()->updateLayout().
> 
> The previous unconditional call also updated style, which may mark the
> layout dirty bit.
> 
Done.

> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:blink-reviews+unsubscribe@chromium.org.

[leviw_travelin_and_unemployed, 2015-04-03 18:27:26]

The CQ bit was checked by leviw@chromium.org

[leviw_travelin_and_unemployed, 2015-04-03 18:27:27]

lgtm.

[commit-bot: I haz the power, 2015-04-03 18:27:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1054423002/20001

[commit-bot: I haz the power, 2015-04-03 22:11:10]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=193144