cc: Add SetNeedsOneBeginImplFrame to the Scheduler

cc: Add SetNeedsOneBeginImplFrame to the Scheduler

This requests another BeginImplFrame to happen. Once
the requests for new ones stop coming, the scheduler
will put the begin frame source to sleep.

This will be used to replace SetNeedsAnimate to
request new frames while an animation wants to tick.

BUG=522658
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/f10a644d2cbccad0d5b80598326420ff1b827f17
Cr-Commit-Position: refs/heads/master@{#357667}

[danakj, 2015-11-03 21:20:26]

Patchset #1 (id:1) has been deleted

[danakj, 2015-11-03 21:20:50]

Patchset #1 (id:20001) has been deleted

[danakj, 2015-11-03 22:05:34]

Description was changed from

==========
snanimate

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Add SetNeedsOneBeginImplFrame to the Scheduler

This requests another BeginImplFrame to happen. Once
the requests for new ones stop coming, the scheduler
will put the begin frame source to sleep.

BUG=522658
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[danakj, 2015-11-03 22:05:34]

danakj@chromium.org changed reviewers:
+ brianderson@chromium.org, sunnyps@chromium.org

[danakj, 2015-11-03 22:06:00]

Description was changed from

==========
cc: Add SetNeedsOneBeginImplFrame to the Scheduler

This requests another BeginImplFrame to happen. Once
the requests for new ones stop coming, the scheduler
will put the begin frame source to sleep.

BUG=522658
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Add SetNeedsOneBeginImplFrame to the Scheduler

This requests another BeginImplFrame to happen. Once
the requests for new ones stop coming, the scheduler
will put the begin frame source to sleep.

This will be used to replace SetNeedsAnimate to
request new frames while an animation wants to tick.

BUG=522658
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[brianderson, 2015-11-03 22:19:19]

lgtm

[sunnyps, 2015-11-03 22:25:55]

LGTM

nit: Can we call this SetNeedsBeginFrameForAnimation instead of
SetNeedsOneBeginImplFrame?

[danakj, 2015-11-03 22:26:51]

On 2015/11/03 22:25:55, sunnyps wrote:
> LGTM
> 
> nit: Can we call this SetNeedsBeginFrameForAnimation instead of
> SetNeedsOneBeginImplFrame?

FWIW this name is matchingish the SetNeedsOneBeginFrame over here:
https://codereview.chromium.org/1408213002/

WDYT?

[sunnyps, 2015-11-03 22:32:07]

On 2015/11/03 22:26:51, danakj wrote:
> On 2015/11/03 22:25:55, sunnyps wrote:
> > LGTM
> > 
> > nit: Can we call this SetNeedsBeginFrameForAnimation instead of
> > SetNeedsOneBeginImplFrame?
> 
> FWIW this name is matchingish the SetNeedsOneBeginFrame over here:
> https://codereview.chromium.org/1408213002/
> 
> WDYT?

Ok. We can rename it if it doesn't make sense later.

[danakj, 2015-11-03 22:46:39]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2015-11-03 22:48:45]

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

[commit-bot: I haz the power, 2015-11-03 23:14:34]

Committed patchset #1 (id:20002)

[commit-bot: I haz the power, 2015-11-03 23:15:15]

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