Blink Animation: Request serviceOnNextFrame on animation's finish and cancel.

Blink Animation: Request serviceOnNextFrame on animation's finish and cancel.

void AnimationTimeline::setOutdatedAnimation(Animation* animation) {
  DCHECK(animation->outdated());
  m_outdatedAnimationCount++;
  m_animationsNeedingUpdate.add(animation);
  if (isActive() && !m_document->page()->animator().isServicingAnimations())
    m_timing->serviceOnNextFrame();
}

AnimationTimeline::setOutdatedAnimation doesn't request
m_timing->serviceOnNextFrame() if animator().isServicingAnimations() (i.e. it is inside the requestAnimationFrame).

Request it explicitly.

BUG=620160
Committed: https://crrev.com/e796f22278e322c1b162deb5b6bbe1895b444c2e
Cr-Commit-Position: refs/heads/master@{#434614}

[loyso (OOO), 2016-11-24 05:53:27]

The CQ bit was checked by loyso@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-24 05:53:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[loyso (OOO), 2016-11-24 05:54:02]

Description was changed from

==========
Blink Animation: Request serviceOnNextFrame on animation's finish and cancel.

BUG=620160
==========

to

==========
Blink Animation: Request serviceOnNextFrame on animation's finish and cancel.

BUG=620160
==========

[loyso (OOO), 2016-11-24 05:54:02]

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

[loyso (OOO), 2016-11-24 05:56:06]

Description was changed from

==========
Blink Animation: Request serviceOnNextFrame on animation's finish and cancel.

BUG=620160
==========

to

==========
Blink Animation: Request serviceOnNextFrame on animation's finish and cancel.

void AnimationTimeline::setOutdatedAnimation(Animation* animation) {
  DCHECK(animation->outdated());
  m_outdatedAnimationCount++;
  m_animationsNeedingUpdate.add(animation);
  if (isActive() && !m_document->page()->animator().isServicingAnimations())
    m_timing->serviceOnNextFrame();
}

AnimationTimeline::setOutdatedAnimation doesn't request
m_timing->serviceOnNextFrame() if animator().isServicingAnimations() (i.e. it is
inside the requestAnimationFrame).

Request it explicitly.

BUG=620160
==========

[commit-bot: I haz the power, 2016-11-24 07:20:18]

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

[commit-bot: I haz the power, 2016-11-24 07:20:18]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[loyso (OOO), 2016-11-28 06:31:18]

PTAL!

[alancutter (OOO until 2018), 2016-11-28 06:46:36]

lgtm

[loyso (OOO), 2016-11-28 06:50:24]

The CQ bit was checked by loyso@chromium.org

[commit-bot: I haz the power, 2016-11-28 06:50:40]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-28 08:05:59]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480315824015480, "parent_rev":
"09f9fbe6abb97b84b130d934deb5798239e47fd0", "commit_rev":
"45d044f8102c01fa08bd17942538fa874b69de8d"}

[commit-bot: I haz the power, 2016-11-28 08:06:22]

Description was changed from

==========
Blink Animation: Request serviceOnNextFrame on animation's finish and cancel.

void AnimationTimeline::setOutdatedAnimation(Animation* animation) {
  DCHECK(animation->outdated());
  m_outdatedAnimationCount++;
  m_animationsNeedingUpdate.add(animation);
  if (isActive() && !m_document->page()->animator().isServicingAnimations())
    m_timing->serviceOnNextFrame();
}

AnimationTimeline::setOutdatedAnimation doesn't request
m_timing->serviceOnNextFrame() if animator().isServicingAnimations() (i.e. it is
inside the requestAnimationFrame).

Request it explicitly.

BUG=620160
==========

to

==========
Blink Animation: Request serviceOnNextFrame on animation's finish and cancel.

void AnimationTimeline::setOutdatedAnimation(Animation* animation) {
  DCHECK(animation->outdated());
  m_outdatedAnimationCount++;
  m_animationsNeedingUpdate.add(animation);
  if (isActive() && !m_document->page()->animator().isServicingAnimations())
    m_timing->serviceOnNextFrame();
}

AnimationTimeline::setOutdatedAnimation doesn't request
m_timing->serviceOnNextFrame() if animator().isServicingAnimations() (i.e. it is
inside the requestAnimationFrame).

Request it explicitly.

BUG=620160
==========

[commit-bot: I haz the power, 2016-11-28 08:06:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-28 08:08:11]

Description was changed from

==========
Blink Animation: Request serviceOnNextFrame on animation's finish and cancel.

void AnimationTimeline::setOutdatedAnimation(Animation* animation) {
  DCHECK(animation->outdated());
  m_outdatedAnimationCount++;
  m_animationsNeedingUpdate.add(animation);
  if (isActive() && !m_document->page()->animator().isServicingAnimations())
    m_timing->serviceOnNextFrame();
}

AnimationTimeline::setOutdatedAnimation doesn't request
m_timing->serviceOnNextFrame() if animator().isServicingAnimations() (i.e. it is
inside the requestAnimationFrame).

Request it explicitly.

BUG=620160
==========

to

==========
Blink Animation: Request serviceOnNextFrame on animation's finish and cancel.

void AnimationTimeline::setOutdatedAnimation(Animation* animation) {
  DCHECK(animation->outdated());
  m_outdatedAnimationCount++;
  m_animationsNeedingUpdate.add(animation);
  if (isActive() && !m_document->page()->animator().isServicingAnimations())
    m_timing->serviceOnNextFrame();
}

AnimationTimeline::setOutdatedAnimation doesn't request
m_timing->serviceOnNextFrame() if animator().isServicingAnimations() (i.e. it is
inside the requestAnimationFrame).

Request it explicitly.

BUG=620160

Committed: https://crrev.com/e796f22278e322c1b162deb5b6bbe1895b444c2e
Cr-Commit-Position: refs/heads/master@{#434614}
==========

[commit-bot: I haz the power, 2016-11-28 08:08:12]

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