android: Remove in-process video path

android: Remove in-process video path

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel

Committed: https://crrev.com/30f1b26acd2c15f4782c1c15dd2fa16173c5f5ef
Cr-Commit-Position: refs/heads/master@{#388090}

[boliu, 2016-03-30 03:18:10]

Description was changed from

==========
android: Remove in-process video path

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

to

==========
android: Remove in-process video path

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[boliu, 2016-03-30 03:19:38]

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

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

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1844843002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1844843002/1

[boliu, 2016-03-30 04:12:00]

The CQ bit was unchecked by boliu@chromium.org

[boliu, 2016-03-30 04:15:16]

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

[commit-bot: I haz the power, 2016-03-30 04:15:55]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1844843002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1844843002/20001

[commit-bot: I haz the power, 2016-03-30 05:41:20]

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

[commit-bot: I haz the power, 2016-03-30 05:41:21]

Dry run: This issue passed the CQ dry run.

[boliu, 2016-04-13 23:52:41]

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

[boliu, 2016-04-13 23:52:42]

needed something to cheer me up, so sending this out

Assuming nothing goes wrong with m50, should be able to land this this time next
week

[boliu, 2016-04-14 02:04:04]

cc tobiasjs

[boliu, 2016-04-18 20:24:19]

Description was changed from

==========
android: Remove in-process video path

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
android: Remove in-process video path

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[boliu, 2016-04-18 20:25:46]

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

[commit-bot: I haz the power, 2016-04-18 20:26:12]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1844843002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1844843002/80001

[boliu, 2016-04-18 20:26:23]

Calling it! Want to land this now. PTAL

[commit-bot: I haz the power, 2016-04-18 20:29:53]

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

[commit-bot: I haz the power, 2016-04-18 20:29:55]

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

[no sievers, 2016-04-18 23:00:50]

LGTM!

https://codereview.chromium.org/1844843002/diff/80001/gpu/command_buffer/clie...
File gpu/command_buffer/client/gl_in_process_context.cc (right):

https://codereview.chromium.org/1844843002/diff/80001/gpu/command_buffer/clie...
gpu/command_buffer/client/gl_in_process_context.cc:205: #if defined(OS_ANDROID)
you forgot to remove this block after commenting it out

[boliu, 2016-04-18 23:05:06]

https://codereview.chromium.org/1844843002/diff/80001/gpu/command_buffer/clie...
File gpu/command_buffer/client/gl_in_process_context.cc (right):

https://codereview.chromium.org/1844843002/diff/80001/gpu/command_buffer/clie...
gpu/command_buffer/client/gl_in_process_context.cc:205: #if defined(OS_ANDROID)
On 2016/04/18 23:00:50, sievers wrote:
> you forgot to remove this block after commenting it out

Oops, removed

[boliu, 2016-04-18 23:05:36]

The CQ bit was checked by boliu@chromium.org

[boliu, 2016-04-18 23:05:37]

The patchset sent to the CQ was uploaded after l-g-t-m from sievers@chromium.org
Link to the patchset: https://codereview.chromium.org/1844843002/#ps100001
(title: "remove commented out code + rebase again")

[commit-bot: I haz the power, 2016-04-18 23:05:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1844843002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1844843002/100001

[commit-bot: I haz the power, 2016-04-19 00:12:46]

Description was changed from

==========
android: Remove in-process video path

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
android: Remove in-process video path

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[commit-bot: I haz the power, 2016-04-19 00:12:48]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-04-19 00:13:49]

Description was changed from

==========
android: Remove in-process video path

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
android: Remove in-process video path

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel

Committed: https://crrev.com/30f1b26acd2c15f4782c1c15dd2fa16173c5f5ef
Cr-Commit-Position: refs/heads/master@{#388090}
==========

[commit-bot: I haz the power, 2016-04-19 00:13:51]

Patchset 6 (id:??) landed as
https://crrev.com/30f1b26acd2c15f4782c1c15dd2fa16173c5f5ef
Cr-Commit-Position: refs/heads/master@{#388090}