Add a bunch of vlogs to RAF related calls in chrome.

Add a bunch of vlogs to RAF related calls in chrome.
    
This is to figure out which link in RAF is broken to cause flakiness.
    
My current guess is the compositor scheduler issue, and these logs can confirm.
    
BUG=393331
TEST=bots
R=kbr@chromium.org
TBR=brianderson@chromium.org

Committed: https://crrev.com/1ca9422fc3ebbdc1370b5a75646924f0d02a8790
Cr-Commit-Position: refs/heads/master@{#292735}

[Zhenyao Mo, 2014-08-29 21:59:51]

zmo@chromium.org changed reviewers:
+ brianderson@chromium.org, kbr@chromium.org

[Zhenyao Mo, 2014-08-29 21:59:52]

kbr: please take a look.

[Ken Russell (switch to Gerrit), 2014-08-29 22:25:43]

LGTM. One question.

https://codereview.chromium.org/516663004/diff/20001/cc/scheduler/scheduler.cc
File cc/scheduler/scheduler.cc (right):

https://codereview.chromium.org/516663004/diff/20001/cc/scheduler/scheduler.c...
cc/scheduler/scheduler.cc:641: case SchedulerStateMachine::ACTION_ANIMATE:
I don't know this code. Is there any chance this action is related to
requestAnimationFrame? If so should it be instrumented too?

https://codereview.chromium.org/516663004/diff/20001/cc/trees/thread_proxy.cc
File cc/trees/thread_proxy.cc (right):

https://codereview.chromium.org/516663004/diff/20001/cc/trees/thread_proxy.cc...
cc/trees/thread_proxy.cc:826: if (main().animate_requested) {
Should this branch be instrumented too? As well as stuff in SetNeedsAnimate?

[Zhenyao Mo, 2014-08-29 22:40:26]

The CQ bit was checked by zmo@chromium.org

[Zhenyao Mo, 2014-08-29 22:40:27]

The CQ bit was unchecked by zmo@chromium.org

[Zhenyao Mo, 2014-08-29 23:11:11]

https://codereview.chromium.org/516663004/diff/20001/cc/scheduler/scheduler.cc
File cc/scheduler/scheduler.cc (right):

https://codereview.chromium.org/516663004/diff/20001/cc/scheduler/scheduler.c...
cc/scheduler/scheduler.cc:641: case SchedulerStateMachine::ACTION_ANIMATE:
On 2014/08/29 22:25:43, Ken Russell wrote:
> I don't know this code. Is there any chance this action is related to
> requestAnimationFrame? If so should it be instrumented too?

Same as explained in the other reply.

https://codereview.chromium.org/516663004/diff/20001/cc/trees/thread_proxy.cc
File cc/trees/thread_proxy.cc (right):

https://codereview.chromium.org/516663004/diff/20001/cc/trees/thread_proxy.cc...
cc/trees/thread_proxy.cc:826: if (main().animate_requested) {
On 2014/08/29 22:25:43, Ken Russell wrote:
> Should this branch be instrumented too? As well as stuff in SetNeedsAnimate?

The logging I added are on the direct call stack to serve RAF.  I think they are
enough to confirm is the root cause of the problem comes from scheduler.  Once
it is confirmed, we can add more logs to digger further.

[Zhenyao Mo, 2014-08-29 23:11:18]

The CQ bit was checked by zmo@chromium.org

[commit-bot: I haz the power, 2014-08-29 23:13:33]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/zmo@chromium.org/516663004/20001

[commit-bot: I haz the power, 2014-08-30 00:35:35]

Committed patchset #1 (id:20001) as 70cdaf4a721efdeaa5e905ab711bdbd6571bf75f

[commit-bot: I haz the power, 2014-09-10 03:12:24]

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