cc: Split and comment methods for Display vs LayerTreeHostImpl

cc: Split and comment methods for Display vs LayerTreeHostImpl

This splits the methods on OutputSurface and OutputSurfaceClient based
on where they are used, to help with splitting these classes.

R=enne
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/5b20e75e681cb9aaa2d8d9696a9954417c9cc068
Cr-Commit-Position: refs/heads/master@{#418009}

[danakj, 2016-09-09 23:49:26]

Description was changed from

==========
cc: Split and comment methods for Display vs LayerTreeHostImpl

This splits the methods on OutputSurface and OutputSurfaceClient based
on where they are used, to help with splitting these classes.

R=enne
BUG=606056
==========

to

==========
cc: Split and comment methods for Display vs LayerTreeHostImpl

This splits the methods on OutputSurface and OutputSurfaceClient based
on where they are used, to help with splitting these classes.

R=enne
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[danakj, 2016-09-09 23:49:32]

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

[commit-bot: I haz the power, 2016-09-09 23:50:01]

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

[danakj, 2016-09-10 00:11:01]

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

[commit-bot: I haz the power, 2016-09-10 00:11:33]

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

[danakj, 2016-09-10 00:13:00]

Missed a few methods on OutputSurface, updated the patch.

[danakj, 2016-09-10 00:13:20]

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

[commit-bot: I haz the power, 2016-09-10 00:13:43]

Dry run: 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-09-10 01:30:07]

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

[commit-bot: I haz the power, 2016-09-10 01:30:08]

Dry run: This issue passed the CQ dry run.

[enne (OOO), 2016-09-12 18:25:20]

lgtm but

https://codereview.chromium.org/2331443003/diff/40001/cc/output/output_surface.h
File cc/output/output_surface.h (right):

https://codereview.chromium.org/2331443003/diff/40001/cc/output/output_surfac...
cc/output/output_surface.h:125: // ============== CONTENT COLLATOR ONLY
=========================
Maybe just call this ContentFrameSink only? I know I know, but content collator
maybe should just stay a joke and not enter the codebase for real.  Even content
compositor is slightly better.

<_<

>_>

[danakj, 2016-09-12 18:35:16]

Oh ok I said layer tree compositor instead.

[danakj, 2016-09-12 18:35:36]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-09-12 18:35:36]

The patchset sent to the CQ was uploaded after l-g-t-m from enne@chromium.org
Link to the patchset: https://codereview.chromium.org/2331443003/#ps60001
(title: "outputsurfacecomments")

[commit-bot: I haz the power, 2016-09-12 18:36:23]

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-09-12 19:46:39]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-09-12 19:49:30]

Description was changed from

==========
cc: Split and comment methods for Display vs LayerTreeHostImpl

This splits the methods on OutputSurface and OutputSurfaceClient based
on where they are used, to help with splitting these classes.

R=enne
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Split and comment methods for Display vs LayerTreeHostImpl

This splits the methods on OutputSurface and OutputSurfaceClient based
on where they are used, to help with splitting these classes.

R=enne
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/5b20e75e681cb9aaa2d8d9696a9954417c9cc068
Cr-Commit-Position: refs/heads/master@{#418009}
==========

[commit-bot: I haz the power, 2016-09-12 19:49:31]

Patchset 4 (id:??) landed as
https://crrev.com/5b20e75e681cb9aaa2d8d9696a9954417c9cc068
Cr-Commit-Position: refs/heads/master@{#418009}