[Chromecast] Fix CastMediaClient::IsSupportedAudioConfig() for (E)AC3 on Android

[Chromecast] Fix CastMediaClient::IsSupportedAudioConfig() for (E)AC3 on Android

Before we have the chance to fully implement the function, we need
a quick fix for (E)AC3.

No ATV we know of that supports (E)AC3 decoder, so it relies on the
audio sink device.

BUG=Internal b/36896359

Review-Url: https://codereview.chromium.org/2845173002
Cr-Commit-Position: refs/heads/master@{#467784}
Committed: https://chromium.googlesource.com/chromium/src/+/d0e6a4dc1cf50fbc387786abbb2379a2b2066aff

[AndyWu, 2017-04-27 18:15:20]

Description was changed from

==========
[Chromecast] Fix CastMediaClient::IsSupportedAudioConfig() for (E)AC3 on Android

Before we have the chance to fully implement the function, we need
a quick fix for (E)AC3.

No ATV we know of that supports (E)AC3 decoder, so it relies on the
audio sink device.

BUG=Internal b/36896359
==========

to

==========
[Chromecast] Fix CastMediaClient::IsSupportedAudioConfig() for (E)AC3 on Android

Before we have the chance to fully implement the function, we need
a quick fix for (E)AC3.

No ATV we know of that supports (E)AC3 decoder, so it relies on the
audio sink device.

BUG=Internal b/36896359
==========

[AndyWu, 2017-04-27 18:15:20]

tsunghung@chromium.org changed reviewers:
+ halliwell@chromium.org, sanfin@chromium.org, servolk@chromium.org

[AndyWu, 2017-04-27 18:19:30]

The previous change in chromecast/common/media/cast_media_client.cc was wiped
out; need to reimplement it again in
chromecast/renderer/cast_content_renderer_client.cc.

[Simeon, 2017-04-27 18:22:03]

On 2017/04/27 18:19:30, AndyWu wrote:
> The previous change in chromecast/common/media/cast_media_client.cc was wiped
> out; need to reimplement it again in
> chromecast/renderer/cast_content_renderer_client.cc.

lgtm

[halliwell, 2017-04-27 19:22:08]

On 2017/04/27 18:22:03, Simeon wrote:
> On 2017/04/27 18:19:30, AndyWu wrote:
> > The previous change in chromecast/common/media/cast_media_client.cc was
wiped
> > out; need to reimplement it again in
> > chromecast/renderer/cast_content_renderer_client.cc.
> 
> lgtm

lgtm

[AndyWu, 2017-04-27 19:59:10]

The CQ bit was checked by tsunghung@chromium.org

[commit-bot: I haz the power, 2017-04-27 19:59:43]

CQ is trying da patch.

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

[servolk, 2017-04-27 20:02:08]

On 2017/04/27 19:59:43, commit-bot: I haz the power wrote:
> CQ is trying da patch.
> 
> Follow status at:
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

lgtm

[commit-bot: I haz the power, 2017-04-27 20:50:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493323150384290, "parent_rev":
"bfd9716d05641238fc5e00e0f83ec8862928974a", "commit_rev":
"d0e6a4dc1cf50fbc387786abbb2379a2b2066aff"}

[commit-bot: I haz the power, 2017-04-27 20:50:14]

Description was changed from

==========
[Chromecast] Fix CastMediaClient::IsSupportedAudioConfig() for (E)AC3 on Android

Before we have the chance to fully implement the function, we need
a quick fix for (E)AC3.

No ATV we know of that supports (E)AC3 decoder, so it relies on the
audio sink device.

BUG=Internal b/36896359
==========

to

==========
[Chromecast] Fix CastMediaClient::IsSupportedAudioConfig() for (E)AC3 on Android

Before we have the chance to fully implement the function, we need
a quick fix for (E)AC3.

No ATV we know of that supports (E)AC3 decoder, so it relies on the
audio sink device.

BUG=Internal b/36896359

Review-Url: https://codereview.chromium.org/2845173002
Cr-Commit-Position: refs/heads/master@{#467784}
Committed:
https://chromium.googlesource.com/chromium/src/+/d0e6a4dc1cf50fbc387786abbb23...
==========

[commit-bot: I haz the power, 2017-04-27 20:50:15]

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