cc: Remove client/engine LayerTreeHostInProcess.

cc: Remove client/engine LayerTreeHostInProcess.

Remove all unused code in LayerTreeHostInProcess for the remote mode.
All code for this is now in cc/blimp, so this removes the code for the
deprecated path. Clean up for proto conversion code is going to be in
a subsequent patch.

BUG=648442
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/c7eae9db07da0e2198fd95936046d719c7e78753
Cr-Commit-Position: refs/heads/master@{#431772}

[Khushal, 2016-11-10 00:30:49]

Description was changed from

==========
cc: Remove client/engine LayerTreeHostInProcess.

BUG=
==========

to

==========
cc: Remove client/engine LayerTreeHostInProcess.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Khushal, 2016-11-10 01:24:57]

Description was changed from

==========
cc: Remove client/engine LayerTreeHostInProcess.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Remove client/engine LayerTreeHostInProcess.

Remove all unused code in LayerTreeHostInProcess for the remote mode.
All code for this is now in cc/blimp, so this removes the code for the
deprecated path. Clean up for proto conversion code is going to be in
a subsequent patch.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Khushal, 2016-11-10 01:28:41]

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

[Khushal, 2016-11-10 01:31:50]

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

[commit-bot: I haz the power, 2016-11-10 01:32:26]

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-11-10 01:56:39]

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

[commit-bot: I haz the power, 2016-11-10 01:56:40]

Dry run: 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...)

[Khushal, 2016-11-10 02:39:30]

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

[commit-bot: I haz the power, 2016-11-10 02:39:57]

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-11-10 03:19:18]

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

[commit-bot: I haz the power, 2016-11-10 03:19:19]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[Khushal, 2016-11-10 06:04:01]

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

[commit-bot: I haz the power, 2016-11-10 06:04:15]

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-11-10 06:08:51]

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

[commit-bot: I haz the power, 2016-11-10 06:08:52]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[Khushal, 2016-11-10 06:48:55]

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

[commit-bot: I haz the power, 2016-11-10 06:49:11]

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-11-10 07:53:43]

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

[commit-bot: I haz the power, 2016-11-10 07:53:44]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[ajuma, 2016-11-10 15:42:13]

Thanks! lgtm

[Khushal, 2016-11-10 20:36:39]

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

[commit-bot: I haz the power, 2016-11-10 20:37:01]

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

[Khushal, 2016-11-10 20:38:47]

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

[Khushal, 2016-11-10 20:38:48]

+piman. Could you take a look at the RemoteProtoChannel move. It's not used in
cc anymore, and is only for renderer<->browser plumbing so content looked like a
better place for it. Its still temporary, I'm going to replace it with mojo.

[commit-bot: I haz the power, 2016-11-10 20:59:01]

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

[commit-bot: I haz the power, 2016-11-10 20:59:02]

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

[Khushal, 2016-11-10 21:45:31]

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

[commit-bot: I haz the power, 2016-11-10 21:46:02]

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-11-10 23:47:46]

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

[commit-bot: I haz the power, 2016-11-10 23:47:47]

Dry run: Try jobs failed on following builders:
  linux_trusty_blink_rel on master.tryserver.blink (JOB_TIMED_OUT, no build URL)

[Khushal, 2016-11-10 23:59:57]

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

[commit-bot: I haz the power, 2016-11-11 00:00:34]

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-11-11 00:11:48]

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

[commit-bot: I haz the power, 2016-11-11 00:11:49]

Dry run: 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...)

[Khushal, 2016-11-11 03:42:51]

Description was changed from

==========
cc: Remove client/engine LayerTreeHostInProcess.

Remove all unused code in LayerTreeHostInProcess for the remote mode.
All code for this is now in cc/blimp, so this removes the code for the
deprecated path. Clean up for proto conversion code is going to be in
a subsequent patch.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Remove client/engine LayerTreeHostInProcess.

Remove all unused code in LayerTreeHostInProcess for the remote mode.
All code for this is now in cc/blimp, so this removes the code for the
deprecated path. Clean up for proto conversion code is going to be in
a subsequent patch.

BUG=648442
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Khushal, 2016-11-11 18:17:51]

On 2016/11/10 20:38:48, Khushal wrote:
> +piman. Could you take a look at the RemoteProtoChannel move. It's not used in
> cc anymore, and is only for renderer<->browser plumbing so content looked like
a
> better place for it. Its still temporary, I'm going to replace it with mojo.

pingy to piman. :)

[piman, 2016-11-12 00:01:21]

sorry for delay, LGTM

[Khushal, 2016-11-12 00:09:07]

On 2016/11/12 00:01:21, piman wrote:
> sorry for delay, LGTM

Thanks!

[Khushal, 2016-11-12 00:09:25]

The CQ bit was checked by khushalsagar@chromium.org

[Khushal, 2016-11-12 00:09:25]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/2494623002/#ps140001
(title: "blimp test")

[commit-bot: I haz the power, 2016-11-12 00:10:07]

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-11-12 00:15:05]

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

[commit-bot: I haz the power, 2016-11-12 00:15:06]

Try jobs failed on following builders:
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  blimp_linux_dbg on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/blimp_linux_dbg...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Khushal, 2016-11-12 00:32:49]

The CQ bit was checked by khushalsagar@chromium.org

[Khushal, 2016-11-12 00:32:50]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org,
piman@chromium.org
Link to the patchset: https://codereview.chromium.org/2494623002/#ps200001
(title: "..")

[commit-bot: I haz the power, 2016-11-12 00:33:18]

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-11-12 02:34:27]

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

[commit-bot: I haz the power, 2016-11-12 02:34:28]

Try jobs failed on following builders:
  linux_trusty_blink_rel on master.tryserver.blink (JOB_TIMED_OUT, no build URL)

[Khushal, 2016-11-12 02:36:52]

The CQ bit was checked by khushalsagar@chromium.org

[commit-bot: I haz the power, 2016-11-12 02:37:08]

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-11-12 04:44:35]

Description was changed from

==========
cc: Remove client/engine LayerTreeHostInProcess.

Remove all unused code in LayerTreeHostInProcess for the remote mode.
All code for this is now in cc/blimp, so this removes the code for the
deprecated path. Clean up for proto conversion code is going to be in
a subsequent patch.

BUG=648442
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Remove client/engine LayerTreeHostInProcess.

Remove all unused code in LayerTreeHostInProcess for the remote mode.
All code for this is now in cc/blimp, so this removes the code for the
deprecated path. Clean up for proto conversion code is going to be in
a subsequent patch.

BUG=648442
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[commit-bot: I haz the power, 2016-11-12 04:44:36]

Committed patchset #11 (id:200001)

[commit-bot: I haz the power, 2016-11-12 04:46:43]

Description was changed from

==========
cc: Remove client/engine LayerTreeHostInProcess.

Remove all unused code in LayerTreeHostInProcess for the remote mode.
All code for this is now in cc/blimp, so this removes the code for the
deprecated path. Clean up for proto conversion code is going to be in
a subsequent patch.

BUG=648442
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Remove client/engine LayerTreeHostInProcess.

Remove all unused code in LayerTreeHostInProcess for the remote mode.
All code for this is now in cc/blimp, so this removes the code for the
deprecated path. Clean up for proto conversion code is going to be in
a subsequent patch.

BUG=648442
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/c7eae9db07da0e2198fd95936046d719c7e78753
Cr-Commit-Position: refs/heads/master@{#431772}
==========

[commit-bot: I haz the power, 2016-11-12 04:46:44]

Patchset 11 (id:??) landed as
https://crrev.com/c7eae9db07da0e2198fd95936046d719c7e78753
Cr-Commit-Position: refs/heads/master@{#431772}