Always return CHANNEL_LAYOUT_UNSUPPORTED for unsupported layouts.

Always return CHANNEL_LAYOUT_UNSUPPORTED for unsupported layouts.

The old code would sometimes return CHANNEL_LAYOUT_NONE which we
don't handle fallback to guessing the layout for. Instead always
return CHANNEL_LAYOUT_UNSUPPORTED so that we will properly attempt
to guess the channel layout based on channel count.

This is a speculative fix for broken audio on Traktor 2 Audio.

BUG=651172
TEST=none
CQ_INCLUDE_TRYBOTS=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/2687873006
Cr-Commit-Position: refs/heads/master@{#449831}
Committed: https://chromium.googlesource.com/chromium/src/+/f062c42d082ebf9c2f2e39ac6683f7755d232ac2

[DaleCurtis, 2017-02-10 23:57:36]

Description was changed from

==========
Always return CHANNEL_LAYOUT_UNSUPPORTED for unsupported layouts.

The old code would sometimes return CHANNEL_LAYOUT_NONE which we
don't handle fallback to guessing the layout for. Instead always
return CHANNEL_LAYOUT_UNSUPPORTED so that we will properly attempt
to guess the channel layout based on channel count.

BUG=651172
TEST=none
==========

to

==========
Always return CHANNEL_LAYOUT_UNSUPPORTED for unsupported layouts.

The old code would sometimes return CHANNEL_LAYOUT_NONE which we
don't handle fallback to guessing the layout for. Instead always
return CHANNEL_LAYOUT_UNSUPPORTED so that we will properly attempt
to guess the channel layout based on channel count.

BUG=651172
TEST=none
CQ_INCLUDE_TRYBOTS=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
==========

[DaleCurtis, 2017-02-10 23:58:15]

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

[DaleCurtis, 2017-02-10 23:58:40]

Description was changed from

==========
Always return CHANNEL_LAYOUT_UNSUPPORTED for unsupported layouts.

The old code would sometimes return CHANNEL_LAYOUT_NONE which we
don't handle fallback to guessing the layout for. Instead always
return CHANNEL_LAYOUT_UNSUPPORTED so that we will properly attempt
to guess the channel layout based on channel count.

BUG=651172
TEST=none
CQ_INCLUDE_TRYBOTS=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

==========
Always return CHANNEL_LAYOUT_UNSUPPORTED for unsupported layouts.

The old code would sometimes return CHANNEL_LAYOUT_NONE which we
don't handle fallback to guessing the layout for. Instead always
return CHANNEL_LAYOUT_UNSUPPORTED so that we will properly attempt
to guess the channel layout based on channel count.

This is a speculative fix for broken audio on Traktor 2 Audio.

BUG=651172
TEST=none
CQ_INCLUDE_TRYBOTS=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
==========

[chcunningham, 2017-02-11 00:31:06]

LGTM

[DaleCurtis, 2017-02-11 00:31:36]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2017-02-11 00:32:34]

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-02-11 02:09:35]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486773096086840, "parent_rev":
"c3af50906850b5d9c9439edd80d996e236996c4d", "commit_rev":
"f062c42d082ebf9c2f2e39ac6683f7755d232ac2"}

[commit-bot: I haz the power, 2017-02-11 02:10:09]

Description was changed from

==========
Always return CHANNEL_LAYOUT_UNSUPPORTED for unsupported layouts.

The old code would sometimes return CHANNEL_LAYOUT_NONE which we
don't handle fallback to guessing the layout for. Instead always
return CHANNEL_LAYOUT_UNSUPPORTED so that we will properly attempt
to guess the channel layout based on channel count.

This is a speculative fix for broken audio on Traktor 2 Audio.

BUG=651172
TEST=none
CQ_INCLUDE_TRYBOTS=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

==========
Always return CHANNEL_LAYOUT_UNSUPPORTED for unsupported layouts.

The old code would sometimes return CHANNEL_LAYOUT_NONE which we
don't handle fallback to guessing the layout for. Instead always
return CHANNEL_LAYOUT_UNSUPPORTED so that we will properly attempt
to guess the channel layout based on channel count.

This is a speculative fix for broken audio on Traktor 2 Audio.

BUG=651172
TEST=none
CQ_INCLUDE_TRYBOTS=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/2687873006
Cr-Commit-Position: refs/heads/master@{#449831}
Committed:
https://chromium.googlesource.com/chromium/src/+/f062c42d082ebf9c2f2e39ac6683...
==========

[commit-bot: I haz the power, 2017-02-11 02:10:10]

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