Change VideoFrameCompositor events to be async.

Change VideoFrameCompositor events from a start/stop pair to AsyncEvents.

BUG=catapult:2683
Committed: https://crrev.com/7de4d26d40e26626d70cf5cb2420d4e10ca0f620
Cr-Commit-Position: refs/heads/master@{#412677}

[alexandermont, 2016-08-16 22:14:52]

alexandermont@chromium.org changed reviewers:
+ kbr@chromium.org, wolenetz@chromium.org

[Ken Russell (switch to Gerrit), 2016-08-17 07:28:46]

LGTM if this has been tested.

Please reference a bug ID with more background information in the CL
description.

[alexandermont, 2016-08-17 18:41:39]

Description was changed from

==========
Add video IDs to VideoFrameCompositor events.
==========

to

==========
Change VideoFrameCompositor events from a start/stop pair to AsyncEvents.

BUG=catapult:2683
==========

[Ken Russell (switch to Gerrit), 2016-08-17 18:44:58]

Seems fine though I don't understand all the consequences of this change. Still
LGTM.

[alexandermont, 2016-08-17 18:54:11]

The CQ bit was checked by alexandermont@chromium.org

[commit-bot: I haz the power, 2016-08-17 18:54:35]

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-17 19:02:22]

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

[commit-bot: I haz the power, 2016-08-17 19:02:23]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Ken Russell (switch to Gerrit), 2016-08-17 20:27:48]

kbr@chromium.org changed reviewers:
+ dalecurtis@chromium.org, hubbe@chromium.org

[Ken Russell (switch to Gerrit), 2016-08-17 20:27:48]

+a couple more OWNERS

[DaleCurtis, 2016-08-17 20:29:16]

Why the name change?

[alexandermont1, 2016-08-17 20:40:14]

On 2016/08/17 at 20:29:16, dalecurtis wrote:
> Why the name change?

Because originally this was separate start and stop events, so the trace event
names were the same as the function name. Now it's just one async event, so I
made the name be something that describes what's going on through the whole
event (video playback).

[DaleCurtis, 2016-08-17 20:41:33]

Seems okay, I'm not strongly tied to the name. lgtm

[alexandermont, 2016-08-17 22:50:50]

The CQ bit was checked by alexandermont@chromium.org

[commit-bot: I haz the power, 2016-08-17 22:51:13]

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-17 22:55:57]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-17 22:58:39]

Description was changed from

==========
Change VideoFrameCompositor events from a start/stop pair to AsyncEvents.

BUG=catapult:2683
==========

to

==========
Change VideoFrameCompositor events from a start/stop pair to AsyncEvents.

BUG=catapult:2683

Committed: https://crrev.com/7de4d26d40e26626d70cf5cb2420d4e10ca0f620
Cr-Commit-Position: refs/heads/master@{#412677}
==========

[commit-bot: I haz the power, 2016-08-17 22:58:40]

Patchset 3 (id:??) landed as
https://crrev.com/7de4d26d40e26626d70cf5cb2420d4e10ca0f620
Cr-Commit-Position: refs/heads/master@{#412677}