MEDIA_LOG FFmpegAudioDecoder::ConfigureDecoder() channel mismatch

MEDIA_LOG FFmpegAudioDecoder::ConfigureDecoder() channel mismatch

Bug 705038 encountered a case where such channel mismatch occurred, but
logs were insufficient in a release build. This change makes a previous
DLOG instead do a MEDIA_LOG(ERROR) for this specific case.

BUG=714730, 705038
Review-Url: https://codereview.chromium.org/2835043004
Cr-Commit-Position: refs/heads/master@{#466785}
Committed: https://chromium.googlesource.com/chromium/src/+/854dede8e4cb7885e803e9e7f3223f00e89a6d38

[wolenetz, 2017-04-24 19:50:26]

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

[wolenetz, 2017-04-24 19:50:26]

Please take a look.

With the repro that hits this log
(https://bugs.chromium.org/p/chromium/issues/detail?id=705038#c16), the produced
MediaError now contains better detail:
MediaError
code: 4
message: "DECODER_ERROR_NOT_SUPPORTED: Audio configuration specified 2 channels,
but FFmpeg thinks the file contains 1 channels"

[wolenetz, 2017-04-24 19:51:03]

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

[commit-bot: I haz the power, 2017-04-24 19:52:07]

Dry run: CQ is trying da patch.

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

[DaleCurtis, 2017-04-24 20:22:59]

lgtm

[wolenetz, 2017-04-24 20:42:15]

The CQ bit was unchecked by wolenetz@chromium.org

[wolenetz, 2017-04-24 20:42:18]

The CQ bit was checked by wolenetz@chromium.org

[commit-bot: I haz the power, 2017-04-24 20:42:39]

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-04-24 22:01:11]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493066538485370, "parent_rev":
"9ff73ab6d5782649fe7ca39c8729c5862228fe58", "commit_rev":
"854dede8e4cb7885e803e9e7f3223f00e89a6d38"}

[commit-bot: I haz the power, 2017-04-24 22:01:25]

Description was changed from

==========
MEDIA_LOG FFmpegAudioDecoder::ConfigureDecoder() channel mismatch

Bug 705038 encountered a case where such channel mismatch occurred, but
logs were insufficient in a release build. This change makes a previous
DLOG instead do a MEDIA_LOG(ERROR) for this specific case.

BUG=714730,705038
==========

to

==========
MEDIA_LOG FFmpegAudioDecoder::ConfigureDecoder() channel mismatch

Bug 705038 encountered a case where such channel mismatch occurred, but
logs were insufficient in a release build. This change makes a previous
DLOG instead do a MEDIA_LOG(ERROR) for this specific case.

BUG=714730,705038

Review-Url: https://codereview.chromium.org/2835043004
Cr-Commit-Position: refs/heads/master@{#466785}
Committed:
https://chromium.googlesource.com/chromium/src/+/854dede8e4cb7885e803e9e7f322...
==========

[commit-bot: I haz the power, 2017-04-24 22:01:26]

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