Add IsSupportedBitstreamAudioCodec() function in MediaClient.

Add IsSupportedBitstreamAudioCodec() function in MediaClient.

This CL is part of enabling (E)AC3 passthrough feature.
Add IsSupportedBitstreamAudioCodec() function in MediaClient.

BUG=613385
Review-Url: https://codereview.chromium.org/2914273004
Cr-Commit-Position: refs/heads/master@{#477327}
Committed: https://chromium.googlesource.com/chromium/src/+/9cc516bf79d5adb8c2425d42b4d4baaf729fd21e

[AndyWu, 2017-06-02 22:26:04]

Description was changed from

==========
Add IsSupportedBitstreamAudio() function in MediaClient.

This CL is part of enabling (E)AC3 passthrough feature.
Add IsSupportedBitstreamAudio() function in MediaClient.

BUG=613385
==========

to

==========
Add IsSupportedBitstreamAudio() function in MediaClient.

This CL is part of enabling (E)AC3 passthrough feature.
Add IsSupportedBitstreamAudio() function in MediaClient.

BUG=613385
==========

[AndyWu, 2017-06-02 22:26:04]

tsunghung@chromium.org changed reviewers:
+ chcunningham@chromium.org, dalecurtis@google.com

[DaleCurtis, 2017-06-02 22:34:47]

dalecurtis@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[DaleCurtis, 2017-06-02 22:34:48]

lgtm % naming.

https://codereview.chromium.org/2914273004/diff/1/content/public/renderer/con...
File content/public/renderer/content_renderer_client.h (right):

https://codereview.chromium.org/2914273004/diff/1/content/public/renderer/con...
content/public/renderer/content_renderer_client.h:274: virtual bool
IsSupportedBitstreamAudio(media::AudioCodec codec);
Seems it should be IsSupportedBistreamAudioCodec?

[chcunningham, 2017-06-05 18:02:41]

lgtm

[AndyWu, 2017-06-05 18:45:22]

Description was changed from

==========
Add IsSupportedBitstreamAudio() function in MediaClient.

This CL is part of enabling (E)AC3 passthrough feature.
Add IsSupportedBitstreamAudio() function in MediaClient.

BUG=613385
==========

to

==========
Add IsSupportedBistreamAudioCodec() function in MediaClient.

This CL is part of enabling (E)AC3 passthrough feature.
Add IsSupportedBistreamAudioCodec() function in MediaClient.

BUG=613385
==========

[AndyWu, 2017-06-05 18:47:32]

The CQ bit was checked by tsunghung@chromium.org

[AndyWu, 2017-06-05 18:47:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
chcunningham@chromium.org, dalecurtis@chromium.org
Link to the patchset: https://codereview.chromium.org/2914273004/#ps20001
(title: "Rename")

[commit-bot: I haz the power, 2017-06-05 18:48:04]

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-06-05 18:57:50]

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

[commit-bot: I haz the power, 2017-06-05 18:57:51]

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...)

[halliwell, 2017-06-05 19:06:32]

halliwell@chromium.org changed reviewers:
+ halliwell@chromium.org

[halliwell, 2017-06-05 19:06:33]

chromecast/ lgtm % spelling
and you'll need a content owner.

https://codereview.chromium.org/2914273004/diff/20001/content/public/renderer...
File content/public/renderer/content_renderer_client.h (right):

https://codereview.chromium.org/2914273004/diff/20001/content/public/renderer...
content/public/renderer/content_renderer_client.h:279: virtual bool
IsSupportedBistreamAudioCodec(media::AudioCodec codec);
Bistream => Bitstream ?

[halliwell, 2017-06-05 19:06:34]

chromecast/ lgtm % spelling
and you'll need a content owner.

[AndyWu, 2017-06-05 19:23:50]

Description was changed from

==========
Add IsSupportedBistreamAudioCodec() function in MediaClient.

This CL is part of enabling (E)AC3 passthrough feature.
Add IsSupportedBistreamAudioCodec() function in MediaClient.

BUG=613385
==========

to

==========
Add IsSupportedBitstreamAudioCodec() function in MediaClient.

This CL is part of enabling (E)AC3 passthrough feature.
Add IsSupportedBitstreamAudioCodec() function in MediaClient.

BUG=613385
==========

[AndyWu, 2017-06-05 20:48:14]

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

[commit-bot: I haz the power, 2017-06-05 20:48:39]

Dry run: CQ is trying da patch.

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

[AndyWu, 2017-06-05 20:58:00]

tsunghung@chromium.org changed reviewers:
+ jochen@chromium.org
- dalecurtis@google.com

[commit-bot: I haz the power, 2017-06-05 20:59:30]

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

[commit-bot: I haz the power, 2017-06-05 20:59:31]

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...)

[AndyWu, 2017-06-05 21:05:39]

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

[commit-bot: I haz the power, 2017-06-05 21:06: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, 2017-06-05 23:22:40]

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

[commit-bot: I haz the power, 2017-06-05 23:22:41]

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...)

[AndyWu, 2017-06-05 23:53:33]

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

[commit-bot: I haz the power, 2017-06-05 23:54: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, 2017-06-06 03:40:48]

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

[commit-bot: I haz the power, 2017-06-06 03:40:50]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2017-06-06 14:47:24]

lgtm

[AndyWu, 2017-06-06 16:57:17]

The CQ bit was checked by tsunghung@chromium.org

[AndyWu, 2017-06-06 16:57:18]

The patchset sent to the CQ was uploaded after l-g-t-m from
chcunningham@chromium.org, halliwell@chromium.org, dalecurtis@chromium.org
Link to the patchset: https://codereview.chromium.org/2914273004/#ps80001
(title: "Fix one typo")

[commit-bot: I haz the power, 2017-06-06 16:57:46]

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-06-06 17:30:04]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1496768237796140,
"parent_rev": "77c67de3a506c39bf84fec3f92585c507f16f29f", "commit_rev":
"9cc516bf79d5adb8c2425d42b4d4baaf729fd21e"}

[commit-bot: I haz the power, 2017-06-06 17:30:11]

Description was changed from

==========
Add IsSupportedBitstreamAudioCodec() function in MediaClient.

This CL is part of enabling (E)AC3 passthrough feature.
Add IsSupportedBitstreamAudioCodec() function in MediaClient.

BUG=613385
==========

to

==========
Add IsSupportedBitstreamAudioCodec() function in MediaClient.

This CL is part of enabling (E)AC3 passthrough feature.
Add IsSupportedBitstreamAudioCodec() function in MediaClient.

BUG=613385

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

[commit-bot: I haz the power, 2017-06-06 17:30:14]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/9cc516bf79d5adb8c2425d42b4d4...