Clear baseComputedStyle when text-autosizing changes.

Clear baseComputedStyle when text-autosizing changes.

baseComputedStyle is an optimization for animations where the computed
style before animations are applied is cached and cloned to have
cheaper style recalcs for per-frame animation changes. An assumption is
that the computed style for the layout object only changes in
styleForElement or pseudoStyleForElement. That assumption is not true
for text autosizing as the computed style may be changed during layout.
Then, for the next animation frame, the text autosizing factor may be
different even though the style has not been marked for recalc, and the
sanity check for an unchanged baseComputedStyle will trigger an assert.

Make sure we clear the baseComputedStyle for an element when the text
autosizing factor changes.

R=pdr@chromium.org,drott@chromium.org
BUG=596018
Committed: https://crrev.com/32a7b19e92a9ff0fca3aba45c1fdd3896fdd5f1d
Cr-Commit-Position: refs/heads/master@{#382350}

[rune, 2016-03-21 14:23:03]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-03-21 14:23:19]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1816103002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1816103002/1

[rune, 2016-03-21 14:34:36]

ptal

[drott, 2016-03-21 15:19:52]

Non-owner LGTM, I would recommend pdr or someone with text-autosizing experience
to do a final review.
Looks nicely clean to me, too, but unfortunately I don't know much about code in
this area. Thanks very much for addressing this so quickly!

[commit-bot: I haz the power, 2016-03-21 16:11:36]

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

[commit-bot: I haz the power, 2016-03-21 16:11:37]

Dry run: Try jobs failed on following builders:
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[pdr., 2016-03-21 17:49:28]

On 2016/03/21 at 16:11:37, commit-bot wrote:
> Dry run: Try jobs failed on following builders:
>   win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

LGTM. Thanks!

[rune, 2016-03-21 18:35:43]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2016-03-21 18:36:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1816103002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1816103002/1

[commit-bot: I haz the power, 2016-03-21 19:17:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-21 19:20:08]

Description was changed from

==========
Clear baseComputedStyle when text-autosizing changes.

baseComputedStyle is an optimization for animations where the computed
style before animations are applied is cached and cloned to have
cheaper style recalcs for per-frame animation changes. An assumption is
that the computed style for the layout object only changes in
styleForElement or pseudoStyleForElement. That assumption is not true
for text autosizing as the computed style may be changed during layout.
Then, for the next animation frame, the text autosizing factor may be
different even though the style has not been marked for recalc, and the
sanity check for an unchanged baseComputedStyle will trigger an assert.

Make sure we clear the baseComputedStyle for an element when the text
autosizing factor changes.

R=pdr@chromium.org,drott@chromium.org
BUG=596018
==========

to

==========
Clear baseComputedStyle when text-autosizing changes.

baseComputedStyle is an optimization for animations where the computed
style before animations are applied is cached and cloned to have
cheaper style recalcs for per-frame animation changes. An assumption is
that the computed style for the layout object only changes in
styleForElement or pseudoStyleForElement. That assumption is not true
for text autosizing as the computed style may be changed during layout.
Then, for the next animation frame, the text autosizing factor may be
different even though the style has not been marked for recalc, and the
sanity check for an unchanged baseComputedStyle will trigger an assert.

Make sure we clear the baseComputedStyle for an element when the text
autosizing factor changes.

R=pdr@chromium.org,drott@chromium.org
BUG=596018

Committed: https://crrev.com/32a7b19e92a9ff0fca3aba45c1fdd3896fdd5f1d
Cr-Commit-Position: refs/heads/master@{#382350}
==========

[commit-bot: I haz the power, 2016-03-21 19:20:09]

Patchset 1 (id:??) landed as
https://crrev.com/32a7b19e92a9ff0fca3aba45c1fdd3896fdd5f1d
Cr-Commit-Position: refs/heads/master@{#382350}