media: Fix case ordering of switch statements.

media: Fix case ordering of switch statements.

Rearranged case order in IsCodecSupportedOnAndroid and
AudioDecoderConfig::GetHumanReadableCodecName as suggested
in https://codereview.chromium.org/812643005

BUG=none

Committed: https://crrev.com/f9133acc35cd3a1389eb60147a3e41e7233edd41
Cr-Commit-Position: refs/heads/master@{#368204}

[servolk, 2016-01-07 02:32:06]

Description was changed from

==========
Media code cleanup

Clean up nits pointed out in
https://codereview.chromium.org/812643005

BUG=none
==========

to

==========
Media code cleanup

Clean up nits pointed out in PS 23 at
https://codereview.chromium.org/812643005

BUG=none
==========

[servolk, 2016-01-07 02:32:07]

servolk@chromium.org changed reviewers:
+ ddorwin@chromium.org

[ddorwin, 2016-01-07 19:44:11]

Please be a little more specific in the description. For example, media: Fix
case ordering of switch statements.

https://codereview.chromium.org/1563893003/diff/20001/media/base/audio_decode...
File media/base/audio_decoder_config.cc (right):

https://codereview.chromium.org/1563893003/diff/20001/media/base/audio_decode...
media/base/audio_decoder_config.cc:120: return "pcm";
Even though they would be out of order, I think it makes sense to keep ones that
return the same string together. IOW, restore 106-7.

https://codereview.chromium.org/1563893003/diff/20001/media/base/mime_util.cc
File media/base/mime_util.cc (right):

https://codereview.chromium.org/1563893003/diff/20001/media/base/mime_util.cc...
media/base/mime_util.cc:147: case MimeUtil::VORBIS:
This should be before video (H264).

https://codereview.chromium.org/1563893003/diff/20001/media/base/mime_util.cc...
media/base/mime_util.cc:155: 
Opus should be here.
Then HEVC.

[servolk, 2016-01-07 22:07:47]

Description was changed from

==========
Media code cleanup

Clean up nits pointed out in PS 23 at
https://codereview.chromium.org/812643005

BUG=none
==========

to

==========
Media code cleanup

Rearranged case order in IsCodecSupportedOnAndroid and
AudioDecoderConfig::GetHumanReadableCodecName as suggested
in https://codereview.chromium.org/812643005

BUG=none
==========

[servolk, 2016-01-07 22:08:04]

Updated the CL description as well

https://codereview.chromium.org/1563893003/diff/20001/media/base/audio_decode...
File media/base/audio_decoder_config.cc (right):

https://codereview.chromium.org/1563893003/diff/20001/media/base/audio_decode...
media/base/audio_decoder_config.cc:120: return "pcm";
On 2016/01/07 19:44:11, ddorwin wrote:
> Even though they would be out of order, I think it makes sense to keep ones
that
> return the same string together. IOW, restore 106-7.

Done.

https://codereview.chromium.org/1563893003/diff/20001/media/base/mime_util.cc
File media/base/mime_util.cc (right):

https://codereview.chromium.org/1563893003/diff/20001/media/base/mime_util.cc...
media/base/mime_util.cc:147: case MimeUtil::VORBIS:
On 2016/01/07 19:44:11, ddorwin wrote:
> This should be before video (H264).

Done.

https://codereview.chromium.org/1563893003/diff/20001/media/base/mime_util.cc...
media/base/mime_util.cc:155: 
On 2016/01/07 19:44:11, ddorwin wrote:
> Opus should be here.
> Then HEVC.

Done.

[ddorwin, 2016-01-07 23:25:57]

Please update the first line of the description as I previously suggested.

Thanks!

LGTM

[servolk, 2016-01-07 23:30:08]

Description was changed from

==========
Media code cleanup

Rearranged case order in IsCodecSupportedOnAndroid and
AudioDecoderConfig::GetHumanReadableCodecName as suggested
in https://codereview.chromium.org/812643005

BUG=none
==========

to

==========
media: Fix case ordering of switch statements.

Rearranged case order in IsCodecSupportedOnAndroid and
AudioDecoderConfig::GetHumanReadableCodecName as suggested
in https://codereview.chromium.org/812643005

BUG=none
==========

[servolk, 2016-01-07 23:30:14]

The CQ bit was checked by servolk@chromium.org

[commit-bot: I haz the power, 2016-01-07 23:33:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1563893003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1563893003/40001

[commit-bot: I haz the power, 2016-01-07 23:55:13]

Description was changed from

==========
media: Fix case ordering of switch statements.

Rearranged case order in IsCodecSupportedOnAndroid and
AudioDecoderConfig::GetHumanReadableCodecName as suggested
in https://codereview.chromium.org/812643005

BUG=none
==========

to

==========
media: Fix case ordering of switch statements.

Rearranged case order in IsCodecSupportedOnAndroid and
AudioDecoderConfig::GetHumanReadableCodecName as suggested
in https://codereview.chromium.org/812643005

BUG=none
==========

[commit-bot: I haz the power, 2016-01-07 23:55:13]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-01-07 23:56:27]

Description was changed from

==========
media: Fix case ordering of switch statements.

Rearranged case order in IsCodecSupportedOnAndroid and
AudioDecoderConfig::GetHumanReadableCodecName as suggested
in https://codereview.chromium.org/812643005

BUG=none
==========

to

==========
media: Fix case ordering of switch statements.

Rearranged case order in IsCodecSupportedOnAndroid and
AudioDecoderConfig::GetHumanReadableCodecName as suggested
in https://codereview.chromium.org/812643005

BUG=none

Committed: https://crrev.com/f9133acc35cd3a1389eb60147a3e41e7233edd41
Cr-Commit-Position: refs/heads/master@{#368204}
==========

[commit-bot: I haz the power, 2016-01-07 23:56:27]

Patchset 3 (id:??) landed as
https://crrev.com/f9133acc35cd3a1389eb60147a3e41e7233edd41
Cr-Commit-Position: refs/heads/master@{#368204}