Reset multipliers correctly when autosizing is turned off.

Reset multipliers correctly when autosizing is turned off.

This lets us re-land http://crrev.com/208393008 without breaking
AwSettingsTest.testLayoutAlgorithmWithTwoViews.

Without http://crrev.com/208393008, style recalc would always reset the
autosizing multiplier to 1, so the bug in recalculateMultipliers was not
noticed.

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

[esprehn, 2014-03-27 23:10:16]

https://codereview.chromium.org/215063004/diff/1/Source/core/rendering/TextAu...
File Source/core/rendering/TextAutosizer.h (right):

https://codereview.chromium.org/215063004/diff/1/Source/core/rendering/TextAu...
Source/core/rendering/TextAutosizer.h:143: bool m_previouslyAutosized;
You don't initialize this in the constructor.

[pdr., 2014-03-27 23:12:04]

LGTM!

https://codereview.chromium.org/215063004/diff/1/Source/core/rendering/TextAu...
File Source/core/rendering/TextAutosizer.h (right):

https://codereview.chromium.org/215063004/diff/1/Source/core/rendering/TextAu...
Source/core/rendering/TextAutosizer.h:143: bool m_previouslyAutosized;
Do we need to initialize this?

[skobes, 2014-03-27 23:17:24]

https://codereview.chromium.org/215063004/diff/1/Source/core/rendering/TextAu...
File Source/core/rendering/TextAutosizer.h (right):

https://codereview.chromium.org/215063004/diff/1/Source/core/rendering/TextAu...
Source/core/rendering/TextAutosizer.h:143: bool m_previouslyAutosized;
On 2014/03/27 23:10:16, esprehn wrote:
> You don't initialize this in the constructor.

Done.

https://codereview.chromium.org/215063004/diff/1/Source/core/rendering/TextAu...
Source/core/rendering/TextAutosizer.h:143: bool m_previouslyAutosized;
On 2014/03/27 23:12:04, pdr wrote:
> Do we need to initialize this?

Done.

[skobes, 2014-03-27 23:17:40]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2014-03-27 23:17:50]

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

[commit-bot: I haz the power, 2014-03-27 23:21:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-27 23:21:10]

Try jobs failed on following builders:
  tryserver.blink on blink_presubmit

[skobes, 2014-03-27 23:21:45]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2014-03-27 23:21:47]

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

[commit-bot: I haz the power, 2014-03-27 23:24:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-27 23:24:17]

Try jobs failed on following builders:
  tryserver.blink on blink_presubmit

[skobes, 2014-03-27 23:25:30]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2014-03-27 23:25:34]

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

[commit-bot: I haz the power, 2014-03-28 00:25:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-28 00:25:36]

Try jobs failed on following builders:
  tryserver.blink on linux_blink_dbg

[pdr, 2014-03-28 00:36:01]

The CQ bit was checked by pdr@google.com

[commit-bot: I haz the power, 2014-03-28 00:36:12]

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

[skobes, 2014-03-28 00:49:51]

The CQ bit was unchecked by skobes@chromium.org

[skobes, 2014-03-28 00:52:16]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2014-03-28 00:52:23]

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

[commit-bot: I haz the power, 2014-03-28 01:45:13]

Change committed as 170241