media: Fix case ordering of switch statements.

media/base/audio_decoder_config.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/base/audio_decoder_config.h"
 
 #include "base/logging.h"
 #include "media/base/limits.h"
 
 namespace media {
@@ skipping 85 matching lines @@
 // These names come from src/third_party/ffmpeg/libavcodec/codec_desc.c
 std::string AudioDecoderConfig::GetHumanReadableCodecName() const {
   switch (codec()) {
     case kUnknownAudioCodec:
       return "unknown";
     case kCodecAAC:
       return "aac";
     case kCodecMP3:
       return "mp3";
     case kCodecPCM:
-    case kCodecPCM_S16BE:
-    case kCodecPCM_S24BE:
       return "pcm";
     case kCodecVorbis:
       return "vorbis";
     case kCodecFLAC:
       return "flac";
     case kCodecAMR_NB:
       return "amr_nb";
     case kCodecAMR_WB:
       return "amr_wb";
+    case kCodecPCM_MULAW:
+      return "pcm_mulaw";
     case kCodecGSM_MS:
       return "gsm_ms";
+    case kCodecPCM_S16BE:
+      return "pcm";

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

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.

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

Done.

+    case kCodecPCM_S24BE:
+      return "pcm";
+    case kCodecOpus:
+      return "opus";
     case kCodecPCM_ALAW:
       return "pcm_alaw";
-    case kCodecPCM_MULAW:
-      return "pcm_mulaw";
-    case kCodecOpus:
-      return "opus";
     case kCodecALAC:
       return "alac";
   }
   NOTREACHED();
   return "";
 }
 
 }  // namespace media