Add AddChildFrameSink() and RemoveChildFrameSink() To WindowAndroidCompositor

Add AddChildFrameSink() and RemoveChildFrameSink() To WindowAndroidCompositor

DelegatedFrameHostAndroid calls WindowAndroidCompositor::AddChildFrameSink()
and RemoveChildFrameSink() when it needs to attach to/detach from a compositor.

CompositorImpl manages an unordered set of pending FrameSinkIds that emulates
the message queue in mojo on desktop Chrome.

In the future, the FrameSinkId will be invalidated during gpu restart. This
change ensures that SurfaceManager::RegisterHierarchy() only gets called when
the root window has a valid FrameSinkId.

BUG=673543
Review-Url: https://codereview.chromium.org/2683663002
Cr-Commit-Position: refs/heads/master@{#448665}
Committed: https://chromium.googlesource.com/chromium/src/+/aa231112e50ded728119bba18dfd4d93d596f00f

[Alex Z., 2017-02-07 16:19:13]

The CQ bit was checked by staraz@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-07 16:19:40]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Alex Z., 2017-02-07 16:20:43]

staraz@chromium.org changed reviewers:
+ boliu@chromium.org, fsamuel@chromium.org

[Alex Z., 2017-02-07 16:20:44]

This CL contains the changes in Android code from my previous 
DirectCompositorFrameSink CL 
(https://codereview.chromium.org/2612083002/). 

PTAL.

[Fady Samuel, 2017-02-07 16:56:04]

lgtm

[commit-bot: I haz the power, 2017-02-07 16:56:30]

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

[commit-bot: I haz the power, 2017-02-07 16:56:31]

Dry run: This issue passed the CQ dry run.

[boliu, 2017-02-07 17:43:57]

lgtm

[Alex Z., 2017-02-07 17:48:46]

The CQ bit was checked by staraz@chromium.org

[commit-bot: I haz the power, 2017-02-07 17:48:56]

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, 2017-02-07 17:53:27]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486489726828320, "parent_rev":
"be1c7be7700037d82e0e92f66232795832f72101", "commit_rev":
"aa231112e50ded728119bba18dfd4d93d596f00f"}

[commit-bot: I haz the power, 2017-02-07 17:54:05]

Description was changed from

==========
Add AddChildFrameSink() and RemoveChildFrameSink() To WindowAndroidCompositor

DelegatedFrameHostAndroid calls WindowAndroidCompositor::AddChildFrameSink()
and RemoveChildFrameSink() when it needs to attach to/detach from a compositor.

CompositorImpl manages an unordered set of pending FrameSinkIds that emulates
the message queue in mojo on desktop Chrome.

In the future, the FrameSinkId will be invalidated during gpu restart. This
change ensures that SurfaceManager::RegisterHierarchy() only gets called when
the root window has a valid FrameSinkId.

BUG=673543
==========

to

==========
Add AddChildFrameSink() and RemoveChildFrameSink() To WindowAndroidCompositor

DelegatedFrameHostAndroid calls WindowAndroidCompositor::AddChildFrameSink()
and RemoveChildFrameSink() when it needs to attach to/detach from a compositor.

CompositorImpl manages an unordered set of pending FrameSinkIds that emulates
the message queue in mojo on desktop Chrome.

In the future, the FrameSinkId will be invalidated during gpu restart. This
change ensures that SurfaceManager::RegisterHierarchy() only gets called when
the root window has a valid FrameSinkId.

BUG=673543

Review-Url: https://codereview.chromium.org/2683663002
Cr-Commit-Position: refs/heads/master@{#448665}
Committed:
https://chromium.googlesource.com/chromium/src/+/aa231112e50ded728119bba18dfd...
==========

[commit-bot: I haz the power, 2017-02-07 17:54:07]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/aa231112e50ded728119bba18dfd...