sync compositor: Implement post metadata to UI

sync compositor: Implement post metadata to UI

For the async draw path. The frame is signaled on the IO thread, but the
metadata still needs to be posted back to the UI thread.

Host registers itself with Filter on UI thread, so Filter can keep a
routing_id to Host map. Note that Host and Filter lifetimes are still
independent.

With this CL, the async path is feature complete.

BUG=636164
Committed: https://crrev.com/524d4c0b9df4f2df8da612826a4bf539d4d839b1
Cr-Commit-Position: refs/heads/master@{#428277}

[boliu, 2016-10-26 22:40:40]

boliu@chromium.org changed reviewers:
+ sgurun@chromium.org

[boliu, 2016-10-26 22:40:52]

err, I mean.. ptal :p

[boliu, 2016-10-26 22:58:45]

Description was changed from

==========
sync compsitor: Implement post metadata to UI

For the async draw path. The frame is signaled on the IO thread, but the
metadata still needs to be posted back to the UI thread.

With this CL, the async path is feature complete.

BUG=636164
==========

to

==========
sync compsitor: Implement post metadata to UI

For the async draw path. The frame is signaled on the IO thread, but the
metadata still needs to be posted back to the UI thread.

Host registers itself with Filter on UI thread, so Filter can keep a
routing_id to Host map. Note that Host and Filter lifetimes are still
independent.

With this CL, the async path is feature complete.

BUG=636164
==========

[boliu, 2016-10-27 05:00:43]

Description was changed from

==========
sync compsitor: Implement post metadata to UI

For the async draw path. The frame is signaled on the IO thread, but the
metadata still needs to be posted back to the UI thread.

Host registers itself with Filter on UI thread, so Filter can keep a
routing_id to Host map. Note that Host and Filter lifetimes are still
independent.

With this CL, the async path is feature complete.

BUG=636164
==========

to

==========
sync compositor: Implement post metadata to UI

For the async draw path. The frame is signaled on the IO thread, but the
metadata still needs to be posted back to the UI thread.

Host registers itself with Filter on UI thread, so Filter can keep a
routing_id to Host map. Note that Host and Filter lifetimes are still
independent.

With this CL, the async path is feature complete.

BUG=636164
==========

[sgurun-gerrit only, 2016-10-27 23:14:48]

https://codereview.chromium.org/2456523004/diff/20001/content/browser/android...
File content/browser/android/synchronous_compositor_host.h (right):

https://codereview.chromium.org/2456523004/diff/20001/content/browser/android...
content/browser/android/synchronous_compositor_host.h:109: bool
registered_with_filter_ = false;
I don't know if such initialization is acceptable or not as a style. Please
confirm.

[sgurun-gerrit only, 2016-10-27 23:15:01]

On 2016/10/27 23:14:48, sgurun wrote:
>
https://codereview.chromium.org/2456523004/diff/20001/content/browser/android...
> File content/browser/android/synchronous_compositor_host.h (right):
> 
>
https://codereview.chromium.org/2456523004/diff/20001/content/browser/android...
> content/browser/android/synchronous_compositor_host.h:109: bool
> registered_with_filter_ = false;
> I don't know if such initialization is acceptable or not as a style. Please
> confirm.

lgtm with one minor coment.

[boliu, 2016-10-28 00:41:38]

https://codereview.chromium.org/2456523004/diff/20001/content/browser/android...
File content/browser/android/synchronous_compositor_host.h (right):

https://codereview.chromium.org/2456523004/diff/20001/content/browser/android...
content/browser/android/synchronous_compositor_host.h:109: bool
registered_with_filter_ = false;
On 2016/10/27 23:14:48, sgurun wrote:
> I don't know if such initialization is acceptable or not as a style. Please
> confirm.

See "Non-Static Class Member Initializers" as officially being allowed here:
https://chromium-cpp.appspot.com/

Here's an example:
https://cs.chromium.org/chromium/src/cc/output/gl_renderer.h?rcl=0&l=541

[boliu, 2016-10-28 01:10:29]

had to rebase on top of https://codereview.chromium.org/2458743002/

[boliu, 2016-10-28 01:10:37]

The CQ bit was checked by boliu@chromium.org

[boliu, 2016-10-28 01:10:37]

The patchset sent to the CQ was uploaded after l-g-t-m from sgurun@chromium.org
Link to the patchset: https://codereview.chromium.org/2456523004/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2016-10-28 01:10:57]

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-10-28 02:36:18]

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

[commit-bot: I haz the power, 2016-10-28 02:36:19]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[boliu, 2016-10-28 02:42:15]

The CQ bit was checked by boliu@chromium.org

[commit-bot: I haz the power, 2016-10-28 02:42:36]

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-10-28 04:04:31]

Description was changed from

==========
sync compositor: Implement post metadata to UI

For the async draw path. The frame is signaled on the IO thread, but the
metadata still needs to be posted back to the UI thread.

Host registers itself with Filter on UI thread, so Filter can keep a
routing_id to Host map. Note that Host and Filter lifetimes are still
independent.

With this CL, the async path is feature complete.

BUG=636164
==========

to

==========
sync compositor: Implement post metadata to UI

For the async draw path. The frame is signaled on the IO thread, but the
metadata still needs to be posted back to the UI thread.

Host registers itself with Filter on UI thread, so Filter can keep a
routing_id to Host map. Note that Host and Filter lifetimes are still
independent.

With this CL, the async path is feature complete.

BUG=636164
==========

[commit-bot: I haz the power, 2016-10-28 04:04:33]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-28 04:06:47]

Description was changed from

==========
sync compositor: Implement post metadata to UI

For the async draw path. The frame is signaled on the IO thread, but the
metadata still needs to be posted back to the UI thread.

Host registers itself with Filter on UI thread, so Filter can keep a
routing_id to Host map. Note that Host and Filter lifetimes are still
independent.

With this CL, the async path is feature complete.

BUG=636164
==========

to

==========
sync compositor: Implement post metadata to UI

For the async draw path. The frame is signaled on the IO thread, but the
metadata still needs to be posted back to the UI thread.

Host registers itself with Filter on UI thread, so Filter can keep a
routing_id to Host map. Note that Host and Filter lifetimes are still
independent.

With this CL, the async path is feature complete.

BUG=636164

Committed: https://crrev.com/524d4c0b9df4f2df8da612826a4bf539d4d839b1
Cr-Commit-Position: refs/heads/master@{#428277}
==========

[commit-bot: I haz the power, 2016-10-28 04:06:47]

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