Work on getting things built with -Wimplicit-fallthrough. This CL was uploaded by git cl split.

Bug: 177475
R=dalecurtis@chromium.org

Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;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
Change-Id: I5536e4854edf66a0838fe0a89f37bdedfc6bfc7e
Reviewed-on: https://chromium-review.googlesource.com/889721Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532565}

media/cdm/ppapi/clear_key_cdm/ffmpeg_cdm_audio_decoder.cc

@@ -247,9 +247,11 @@ cdm::Status FFmpegCdmAudioDecoder::DecodeBuffer(
       return cdm::kDecodeError;
     case FFmpegDecodingLoop::DecodeStatus::kFrameProcessingFailed:
       NOTREACHED();
+      FALLTHROUGH;
     case FFmpegDecodingLoop::DecodeStatus::kDecodeFrameFailed:
       DLOG(WARNING) << " failed to decode an audio buffer: "
                     << timestamp.InMicroseconds();
+      break;
     case FFmpegDecodingLoop::DecodeStatus::kOkay:
       break;
   }

media/filters/aom_video_decoder_unittest.cc

@@ -96,6 +96,7 @@ class AomVideoDecoderTest : public testing::Test {
           break;
         case DecodeStatus::ABORTED:
           NOTREACHED();
+          FALLTHROUGH;
         case DecodeStatus::DECODE_ERROR:
           DCHECK(output_frames_.empty());
           return status;

media/filters/ffmpeg_audio_decoder.cc

@@ -193,6 +193,7 @@ bool FFmpegAudioDecoder::FFmpegDecode(
           << GetDisplayName() << " failed to decode an audio buffer: "
           << AVErrorToString(decoding_loop_->last_averror_code()) << ", at "
           << buffer->AsHumanReadableString();
+      break;
     case FFmpegDecodingLoop::DecodeStatus::kOkay:
       break;
   }

media/filters/ffmpeg_video_decoder_unittest.cc

@@ -136,6 +136,7 @@ class FFmpegVideoDecoderTest : public testing::Test {
           break;
         case DecodeStatus::ABORTED:
           NOTREACHED();
+          FALLTHROUGH;
         case DecodeStatus::DECODE_ERROR:
           DCHECK(output_frames_.empty());
           return status;

media/filters/video_frame_stream_unittest.cc

@@ -369,6 +369,7 @@ class VideoFrameStreamTest
       case DEMUXER_READ_CONFIG_CHANGE:
         EXPECT_MEDIA_LOG(HasSubstr("decoder config changed"))
             .Times(testing::AtLeast(1));
+        FALLTHROUGH;
       case DEMUXER_READ_NORMAL:
         demuxer_stream_->SatisfyRead();
         break;

media/filters/vpx_video_decoder_unittest.cc

@@ -91,6 +91,7 @@ class VpxVideoDecoderTest : public testing::Test {
           break;
         case DecodeStatus::ABORTED:
           NOTREACHED();
+          FALLTHROUGH;
         case DecodeStatus::DECODE_ERROR:
           DCHECK(output_frames_.empty());
           return status;