cc: Trigger BeginFrame when children needs it without SetNeedsCommit

cc: Trigger BeginFrame when children needs it without SetNeedsCommit

Browser scheduler should trigger BeginFrame when children needs even if
it doesn't have a pending commit.

R=brianderson@chromium.org
BUG=372086
TEST=cc_unittests

Committed: https://crrev.com/22c5f0f29d47128966ea3103f6860dfcffcd1bb5
Cr-Commit-Position: refs/heads/master@{#308467}

[simonhong, 2014-12-15 13:27:11]

PTAL!

[brianderson, 2014-12-15 20:25:41]

https://codereview.chromium.org/802353002/diff/1/cc/scheduler/scheduler.cc
File cc/scheduler/scheduler.cc (right):

https://codereview.chromium.org/802353002/diff/1/cc/scheduler/scheduler.cc#ne...
cc/scheduler/scheduler.cc:450: DCHECK_EQ(state_machine_.NextAction(),
SchedulerStateMachine::ACTION_NONE);
lgtm after you delete this DCHECK.

[simonhong, 2014-12-15 22:34:52]

https://codereview.chromium.org/802353002/diff/1/cc/scheduler/scheduler.cc
File cc/scheduler/scheduler.cc (right):

https://codereview.chromium.org/802353002/diff/1/cc/scheduler/scheduler.cc#ne...
cc/scheduler/scheduler.cc:450: DCHECK_EQ(state_machine_.NextAction(),
SchedulerStateMachine::ACTION_NONE);
On 2014/12/15 20:25:40, brianderson wrote:
> lgtm after you delete this DCHECK.

Done.

[simonhong, 2014-12-15 22:35:03]

The CQ bit was checked by simonhong@chromium.org

[commit-bot: I haz the power, 2014-12-15 22:36:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/802353002/20001

[commit-bot: I haz the power, 2014-12-16 00:06:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-12-16 00:08:37]

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