media: Add mojo_media_config to media/gpu configs

media: Add mojo_media_config to media/gpu configs

Otherwise, macros like ENABLE_MOJO_MEDIA_IN_GPU_PROCESS are not defined.
We should switch to use BUILDFLAG to prevent this from happening, that
is tracked by issue 596252.

BUG=596252, 721935
TEST=Manually tested encrypted playback.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2876123004
Cr-Commit-Position: refs/heads/master@{#471553}
Committed: https://chromium.googlesource.com/chromium/src/+/9d5f354616dea455ea08345c96b67081275a865d

[xhwang, 2017-05-12 21:21:16]

Description was changed from

==========
media: Add mojo_media_config to media/gpu configs

Otherwise, macros like ENABLE_MOJO_MEDIA_IN_GPU_PROCESS are not defined.
We should switch to use BUILDFLAG to prevent this from happening, that
is tracked by issue 596252.

BUG=596252
TEST=Manually tested encrypted playback.
==========

to

==========
media: Add mojo_media_config to media/gpu configs

Otherwise, macros like ENABLE_MOJO_MEDIA_IN_GPU_PROCESS are not defined.
We should switch to use BUILDFLAG to prevent this from happening, that
is tracked by issue 596252.

BUG=596252
TEST=Manually tested encrypted playback.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[xhwang, 2017-05-12 21:26:14]

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

[commit-bot: I haz the power, 2017-05-12 21:27:38]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[xhwang, 2017-05-12 21:30:56]

xhwang@chromium.org changed reviewers:
+ sandersd@chromium.org

[xhwang, 2017-05-12 21:30:56]

I thought our content_browsertests should catch this, but apparently something
went wrong. I'll double check.

The fix is still valid and I manually tested it. PTAL.

I'll also convert the macros to BUILDFLAGS next, to avoid similar issue in the
future.

[sandersd (OOO until July 31), 2017-05-12 22:06:29]

lgtm

[xhwang, 2017-05-12 22:33:28]

Ah, the content_browsertests test the case where we use DecryptingVideoDecoder +
MojoDecyrptor + CDM running in the GPU process, so it doesn't cover AVDA case.
This is because AVDA can only work with MediaDrm based CDM, and we haven't
enabled ClearKey support in MediaDrmBridge yet. I filed crbug.com/721935 to
track this work.

[xhwang, 2017-05-12 22:33:36]

Description was changed from

==========
media: Add mojo_media_config to media/gpu configs

Otherwise, macros like ENABLE_MOJO_MEDIA_IN_GPU_PROCESS are not defined.
We should switch to use BUILDFLAG to prevent this from happening, that
is tracked by issue 596252.

BUG=596252
TEST=Manually tested encrypted playback.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
media: Add mojo_media_config to media/gpu configs

Otherwise, macros like ENABLE_MOJO_MEDIA_IN_GPU_PROCESS are not defined.
We should switch to use BUILDFLAG to prevent this from happening, that
is tracked by issue 596252.

BUG=596252,721935
TEST=Manually tested encrypted playback.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[xhwang, 2017-05-12 22:33:40]

The CQ bit was unchecked by xhwang@chromium.org

[xhwang, 2017-05-12 22:33:40]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2017-05-12 22:34: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, 2017-05-13 03:57:44]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494628420497360, "parent_rev":
"f4d8a9411d27a591c8f9cf50648de32e49e8b889", "commit_rev":
"9d5f354616dea455ea08345c96b67081275a865d"}

[commit-bot: I haz the power, 2017-05-13 03:57:53]

Description was changed from

==========
media: Add mojo_media_config to media/gpu configs

Otherwise, macros like ENABLE_MOJO_MEDIA_IN_GPU_PROCESS are not defined.
We should switch to use BUILDFLAG to prevent this from happening, that
is tracked by issue 596252.

BUG=596252,721935
TEST=Manually tested encrypted playback.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
media: Add mojo_media_config to media/gpu configs

Otherwise, macros like ENABLE_MOJO_MEDIA_IN_GPU_PROCESS are not defined.
We should switch to use BUILDFLAG to prevent this from happening, that
is tracked by issue 596252.

BUG=596252,721935
TEST=Manually tested encrypted playback.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2876123004
Cr-Commit-Position: refs/heads/master@{#471553}
Committed:
https://chromium.googlesource.com/chromium/src/+/9d5f354616dea455ea08345c96b6...
==========

[commit-bot: I haz the power, 2017-05-13 03:57:54]

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