Refactor SMILTimeContainer can-schedule-frame predicate

Refactor SMILTimeContainer can-schedule-frame predicate

begin() and updateAnimationsAndScheduleFrameIfNeeded() use the same
predicate to check if they should schedule an animation frame - although
they phrase it slightly differently (because of local knowledge.)
Move the generic version to a canScheduleFrame() method and use that in
both cases.

BUG=631879
Committed: https://crrev.com/ebd6dde89d88e789ecc2751668506910acba165a
Cr-Commit-Position: refs/heads/master@{#413328}

[fs, 2016-08-18 12:52:08]

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

[commit-bot: I haz the power, 2016-08-18 12:52:16]

Dry run: 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-08-18 14:20:19]

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

[commit-bot: I haz the power, 2016-08-18 14:20:20]

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

[fs, 2016-08-19 11:44:00]

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

[commit-bot: I haz the power, 2016-08-19 11:44:26]

Dry run: 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-08-19 13:08:39]

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

[commit-bot: I haz the power, 2016-08-19 13:08:40]

Dry run: This issue passed the CQ dry run.

[fs, 2016-08-19 13:25:45]

fs@opera.com changed reviewers:
+ pdr@chromium.org

[pdr., 2016-08-19 21:41:13]

On 2016/08/19 at 13:25:46, fs wrote:
> 

LGTM

Touching the animation system frequently leads to fun edge-cases from
clusterfuzz. To make bisecting easier, you may want to wait a few hours between
these patches. Up to you though.

[fs, 2016-08-20 08:53:58]

On 2016/08/19 at 21:41:13, pdr wrote:
> On 2016/08/19 at 13:25:46, fs wrote:
> > 
> 
> LGTM
> 
> Touching the animation system frequently leads to fun edge-cases from
clusterfuzz. To make bisecting easier, you may want to wait a few hours between
these patches. Up to you though.

The try-bots seems to do their best to help in that regard...

[fs, 2016-08-20 08:54:07]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-08-20 08:54:12]

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-08-20 08:59:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-20 09:00:47]

Description was changed from

==========
Refactor SMILTimeContainer can-schedule-frame predicate

begin() and updateAnimationsAndScheduleFrameIfNeeded() use the same
predicate to check if they should schedule an animation frame - although
they phrase it slightly differently (because of local knowledge.)
Move the generic version to a canScheduleFrame() method and use that in
both cases.

BUG=631879
==========

to

==========
Refactor SMILTimeContainer can-schedule-frame predicate

begin() and updateAnimationsAndScheduleFrameIfNeeded() use the same
predicate to check if they should schedule an animation frame - although
they phrase it slightly differently (because of local knowledge.)
Move the generic version to a canScheduleFrame() method and use that in
both cases.

BUG=631879

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

[commit-bot: I haz the power, 2016-08-20 09:00:49]

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