Fix line height when changing multiplier.

Update line height when changing multiplier.

Line height is cached in the RenderBlock; when FTA updates a multiplier it needs to force recomputation of the line height.

(The old autosizer avoids this issue by using setStyle instead of setStyleInternal.  This triggers style recalc, which invalidates the line height.)

BUG=353309, 354032
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169975

[pdr., 2014-03-25 04:28:04]

On 2014/03/24 23:53:14, skobes wrote:

Oh wow, this is a fantastic change.

LGTM, but can you add a test?

[skobes, 2014-03-25 17:35:38]

On 2014/03/25 04:28:04, pdr wrote:
> On 2014/03/24 23:53:14, skobes wrote:
> 
> Oh wow, this is a fantastic change.
> 
> LGTM, but can you add a test?

This is hard to test without fixing the resize/setting change logic.  (To
trigger the bug you have to have a layout without a style recalc.)  Mind if I
test this in http://crrev.com/209353003?

[skobes, 2014-03-25 17:35:38]

On 2014/03/25 04:28:04, pdr wrote:
> On 2014/03/24 23:53:14, skobes wrote:
> 
> Oh wow, this is a fantastic change.
> 
> LGTM, but can you add a test?

This is hard to test without fixing the resize/setting change logic.  (To
trigger the bug you have to have a layout without a style recalc.)  Mind if I
test this in http://crrev.com/209353003?

[pdr, 2014-03-25 17:50:00]

On 2014/03/25 17:35:38, skobes wrote:
> On 2014/03/25 04:28:04, pdr wrote:
> > On 2014/03/24 23:53:14, skobes wrote:
> > 
> > Oh wow, this is a fantastic change.
> > 
> > LGTM, but can you add a test?
> 
> This is hard to test without fixing the resize/setting change logic.  (To
> trigger the bug you have to have a layout without a style recalc.)  Mind if I
> test this in http://crrev.com/209353003?

SGTM!

[pdr, 2014-03-25 17:50:31]

SGTM! :)

[skobes, 2014-03-25 18:10:03]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2014-03-25 18:10:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/skobes@chromium.org/209853007/40001

[skobes, 2014-03-25 18:23:46]

The CQ bit was unchecked by skobes@chromium.org

[skobes, 2014-03-25 18:24:29]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2014-03-25 18:24:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/skobes@chromium.org/209853007/120001

[commit-bot: I haz the power, 2014-03-25 19:31:02]

Change committed as 169975

[vsevik, 2014-03-26 12:21:17]

A revert of this CL has been created in
https://codereview.chromium.org/212163006/ by vsevik@chromium.org.

The reason for reverting is: This might have caused android tests failure: 
http://build.chromium.org/p/chromium.webkit/builders/Android%20Tests%20%28dbg...
Speculatively reverting it..