cc: Rename LayerTreeHost to LayerTreeHostInProcess.

cc: Rename LayerTreeHost to LayerTreeHostInProcess.

This is the final patch in the series that abstracts the LayerTreeHost
class to provide an implementation that run the impl thread components
in-process (single-threaded and threaded compositor). This renames that
existing implementation to the LayerTreeHostInProcess.

Next up, we add an implementation that handles remoting the content
provided to the compositor across a network boundary.

BUG=625283
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/6156779859b6c4978df742a2a09426ef554c9861
Cr-Commit-Position: refs/heads/master@{#419338}

[Khushal, 2016-09-14 23:58:50]

Description was changed from

==========
cc: Rename LayerTreeHost to LayerTreeHostInProcess.

This is the final patch in the series that abstracts the LayerTreeHost
class to provide an implementation that run the impl thread components
in-process (single-threaded and threaded compositor). This renames that
existing implementation to the LayerTreeHostInProcess.

Next up, we add an implementation that handles remoting the content
provided to the compositor across a network boundary.

BUG=625283
==========

to

==========
cc: Rename LayerTreeHost to LayerTreeHostInProcess.

This is the final patch in the series that abstracts the LayerTreeHost
class to provide an implementation that run the impl thread components
in-process (single-threaded and threaded compositor). This renames that
existing implementation to the LayerTreeHostInProcess.

Next up, we add an implementation that handles remoting the content
provided to the compositor across a network boundary.

BUG=625283
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[Khushal, 2016-09-15 00:00:46]

khushalsagar@chromium.org changed reviewers:
+ danakj@chromium.org

[Khushal, 2016-09-15 00:00:47]

Big rename patch is big. Since you suggested the name for this class, I figured
you would be the best reviewer. :)

I still have to update the external callers and tests.

[Khushal, 2016-09-15 00:56:20]

https://codereview.chromium.org/2340143002/diff/20001/cc/trees/layer_tree_hos...
File cc/trees/layer_tree_host_in_process.h (right):

https://codereview.chromium.org/2340143002/diff/20001/cc/trees/layer_tree_hos...
cc/trees/layer_tree_host_in_process.h:99: static
std::unique_ptr<LayerTreeHostInProcess> CreateThreaded(
I made these return the in process one itself. If the caller wants to keep the
interface, which only the renderer compositor would need to, then they can do
it.

[Khushal, 2016-09-15 01:17:44]

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

[commit-bot: I haz the power, 2016-09-15 01:18:19]

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-15 01:31:11]

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

[commit-bot: I haz the power, 2016-09-15 01:31:12]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Khushal, 2016-09-15 01:47:31]

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

[commit-bot: I haz the power, 2016-09-15 01:48:13]

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

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

[commit-bot: I haz the power, 2016-09-15 02:02:23]

Dry run: Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Khushal, 2016-09-15 20:30:47]

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

[commit-bot: I haz the power, 2016-09-15 20:31:14]

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-15 20:43:01]

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

[commit-bot: I haz the power, 2016-09-15 20:43:02]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[Khushal, 2016-09-15 20:56:59]

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

[commit-bot: I haz the power, 2016-09-15 20:57:44]

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-15 21:12:57]

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

[commit-bot: I haz the power, 2016-09-15 21:12:58]

Dry run: Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Khushal, 2016-09-15 21:49:57]

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

[commit-bot: I haz the power, 2016-09-15 21:50:29]

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-16 00:30:19]

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

[commit-bot: I haz the power, 2016-09-16 00:30:20]

Dry run: This issue passed the CQ dry run.

[Khushal, 2016-09-16 17:31:06]

friendly ping. Rebasing it once on the CompositorFrameSink patch was fun enough.
:P

[Khushal, 2016-09-16 18:08:13]

khushalsagar@chromium.org changed reviewers:
+ dtrainor@chromium.org, enne@chromium.org, sievers@chromium.org

[Khushal, 2016-09-16 18:08:14]

+enne for renderer compositor and Webkit
+dtrainor for blimp
+sievers for android UI compositor

[enne (OOO), 2016-09-16 18:19:41]

enne@chromium.org changed reviewers:
+ pdr@chromium.org

[enne (OOO), 2016-09-16 18:19:42]

cc and render_widget_compositor lgtm

I am not a Source/platform owner.  +pdr

https://codereview.chromium.org/2340143002/diff/140001/cc/trees/layer_tree_ho...
File cc/trees/layer_tree_host_in_process.h (right):

https://codereview.chromium.org/2340143002/diff/140001/cc/trees/layer_tree_ho...
cc/trees/layer_tree_host_in_process.h:339: // initialized a new output surface.
|new_compositor_frame_sink_| contains the
This paragraph looks like a bad rebase? There are other output surface
references as well that danakj needs to clean up, but this is the only one that
you changed back, I think.

[David Trainor- moved to gerrit, 2016-09-16 18:24:46]

blimp/ lgtm

[pdr., 2016-09-16 18:31:47]

On 2016/09/16 at 18:24:46, dtrainor wrote:
> blimp/ lgtm

platform lgtm

[Khushal, 2016-09-16 18:33:28]

khushalsagar@chromium.org changed reviewers:
+ piman@chromium.org

[Khushal, 2016-09-16 18:33:30]

+piman, may be you could take a look at rest of the content bits? Nothing
exciting, just small renames.

[Khushal, 2016-09-16 18:38:40]

https://codereview.chromium.org/2340143002/diff/140001/cc/trees/layer_tree_ho...
File cc/trees/layer_tree_host_in_process.h (right):

https://codereview.chromium.org/2340143002/diff/140001/cc/trees/layer_tree_ho...
cc/trees/layer_tree_host_in_process.h:339: // initialized a new output surface.
|new_compositor_frame_sink_| contains the
On 2016/09/16 18:19:42, enne wrote:
> This paragraph looks like a bad rebase? There are other output surface
> references as well that danakj needs to clean up, but this is the only one
that
> you changed back, I think.

Woops, thanks for catching that. Done.

[Khushal, 2016-09-16 18:41:50]

On 2016/09/16 18:33:30, Khushal wrote:
> +piman, may be you could take a look at rest of the content bits? Nothing
> exciting, just small renames.

Ooh, also ui.

[Khushal, 2016-09-16 20:37:59]

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

[commit-bot: I haz the power, 2016-09-16 20:38:24]

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

[piman, 2016-09-16 20:41:12]

lgtm

[Khushal, 2016-09-16 20:41:40]

The CQ bit was unchecked by khushalsagar@chromium.org

[Khushal, 2016-09-16 20:41:47]

The CQ bit was checked by khushalsagar@chromium.org

[Khushal, 2016-09-16 20:41:47]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org, enne@chromium.org, pdr@chromium.org
Link to the patchset: https://codereview.chromium.org/2340143002/#ps160001
(title: "comment fix")

[commit-bot: I haz the power, 2016-09-16 20:42: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-09-17 00:14:17]

Committed patchset #9 (id:160001)

[commit-bot: I haz the power, 2016-09-17 00:18:02]

Description was changed from

==========
cc: Rename LayerTreeHost to LayerTreeHostInProcess.

This is the final patch in the series that abstracts the LayerTreeHost
class to provide an implementation that run the impl thread components
in-process (single-threaded and threaded compositor). This renames that
existing implementation to the LayerTreeHostInProcess.

Next up, we add an implementation that handles remoting the content
provided to the compositor across a network boundary.

BUG=625283
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Rename LayerTreeHost to LayerTreeHostInProcess.

This is the final patch in the series that abstracts the LayerTreeHost
class to provide an implementation that run the impl thread components
in-process (single-threaded and threaded compositor). This renames that
existing implementation to the LayerTreeHostInProcess.

Next up, we add an implementation that handles remoting the content
provided to the compositor across a network boundary.

BUG=625283
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/6156779859b6c4978df742a2a09426ef554c9861
Cr-Commit-Position: refs/heads/master@{#419338}
==========

[commit-bot: I haz the power, 2016-09-17 00:18:03]

Patchset 9 (id:??) landed as
https://crrev.com/6156779859b6c4978df742a2a09426ef554c9861
Cr-Commit-Position: refs/heads/master@{#419338}