Recomposite transform animations when CSS zoom changes

Recomposite transform animations when CSS zoom changes

This change ensures we update composited transform animations in response
to the CSS zoom level changing so we aren't using outdated transform
compositor keyframe values.

Note: This scenario continues to be a bug for main thread transform animations.

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

[alancutter (OOO until 2018), 2015-09-15 11:53:58]

alancutter@chromium.org changed reviewers:
+ dstockwell@chromium.org

[Timothy Loh, 2015-09-15 11:58:59]

Can we test this with page zoom? window.internals has a setZoomFactor function.

[alancutter (OOO until 2018), 2015-09-16 03:10:59]

On 2015/09/15 at 11:58:59, timloh wrote:
> Can we test this with page zoom? window.internals has a setZoomFactor
function.

Done.

[alancutter (OOO until 2018), 2015-09-17 02:12:08]

dstockwell: Ping.

[dstockwell, 2015-09-17 04:07:13]

lgtm

https://codereview.chromium.org/1347493004/diff/60001/Source/core/animation/c...
File Source/core/animation/css/CSSAnimations.cpp (right):

https://codereview.chromium.org/1347493004/diff/60001/Source/core/animation/c...
Source/core/animation/css/CSSAnimations.cpp:245: if (oldStyle.effectiveZoom() !=
style.effectiveZoom()) {
early out if !oldStyle.isRunningTransformAnimationOnCompositor

[alancutter (OOO until 2018), 2015-09-17 04:21:45]

https://codereview.chromium.org/1347493004/diff/60001/Source/core/animation/c...
File Source/core/animation/css/CSSAnimations.cpp (right):

https://codereview.chromium.org/1347493004/diff/60001/Source/core/animation/c...
Source/core/animation/css/CSSAnimations.cpp:245: if (oldStyle.effectiveZoom() !=
style.effectiveZoom()) {
On 2015/09/17 at 04:07:13, dstockwell wrote:
> early out if !oldStyle.isRunningTransformAnimationOnCompositor

Done. Added early out of entire function for when no compositor animations are
present.

[alancutter (OOO until 2018), 2015-09-17 04:21:47]

The CQ bit was checked by alancutter@chromium.org

[alancutter (OOO until 2018), 2015-09-17 04:21:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
dstockwell@chromium.org
Link to the patchset: https://codereview.chromium.org/1347493004/#ps80001
(title: "Review changes")

[commit-bot: I haz the power, 2015-09-17 04:21:56]

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

[commit-bot: I haz the power, 2015-09-17 05:18:36]

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

[commit-bot: I haz the power, 2015-09-17 05:18:37]

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

[alancutter (OOO until 2018), 2015-09-17 07:51:15]

The CQ bit was checked by alancutter@chromium.org

[alancutter (OOO until 2018), 2015-09-17 07:51:16]

The patchset sent to the CQ was uploaded after l-g-t-m from
dstockwell@chromium.org
Link to the patchset: https://codereview.chromium.org/1347493004/#ps100001
(title: "Fix tests")

[commit-bot: I haz the power, 2015-09-17 07:51:24]

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

[commit-bot: I haz the power, 2015-09-17 09:16:28]

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

[commit-bot: I haz the power, 2015-09-17 09:16:29]

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

[alancutter (OOO until 2018), 2015-09-17 10:26:41]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2015-09-17 10:26:46]

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

[commit-bot: I haz the power, 2015-09-17 11:15:35]

Committed patchset #6 (id:100001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=202442