Add a few FALLTHROUGH annotations, and fix a missing break in video_color_space.cc

TBR=danakj

Bug: 177475
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;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: I4278dac80badb5768e725986f6a32b11ca244507
Reviewed-on: https://chromium-review.googlesource.com/886882
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531996}

cc/paint/paint_shader.cc

@@ -376,7 +376,7 @@ bool PaintShader::IsValid() const {
           start_degrees_ >= end_degrees_) {
         return false;
       }
-    // Fallthrough.
+      FALLTHROUGH;
     case Type::kLinearGradient:
     case Type::kRadialGradient:
     case Type::kTwoPointConicalGradient:
@@ -428,7 +428,7 @@ bool PaintShader::operator==(const PaintShader& other) const {
         return false;
       if (!PaintOp::AreEqualEvenIfNaN(end_degrees_, other.end_degrees_))
         return false;
-    // Fallthrough.
+      FALLTHROUGH;
     case Type::kLinearGradient:
     case Type::kRadialGradient:
     case Type::kTwoPointConicalGradient:

media/base/mime_util_internal.cc

@@ -575,6 +575,7 @@ bool MimeUtil::IsCodecSupportedOnAndroid(
       // valid codecs to be used with HLS mime types.
       DCHECK(!base::EndsWith(mime_type_lower_case, "mpegurl",
                              base::CompareCase::SENSITIVE));
+      FALLTHROUGH;
     case PCM:
     case MP3:
     case MPEG4_AAC:

media/base/video_color_space.cc

@@ -104,6 +104,7 @@ gfx::ColorSpace VideoColorSpace::ToGfxColorSpace() const {
       break;
     case PrimaryID::SMPTEST432_1:
       primary_id = gfx::ColorSpace::PrimaryID::SMPTEST432_1;
+      break;
     case PrimaryID::EBU_3213_E:
       // TODO(uzair.jaleel) Need to check this once.
       primary_id = gfx::ColorSpace::PrimaryID::INVALID;

media/base/video_frame.cc

@@ -830,7 +830,7 @@ size_t VideoFrame::BitDepth() const {
   switch (format_) {
     case media::PIXEL_FORMAT_UNKNOWN:
       NOTREACHED();
-    // Fall through!
+      FALLTHROUGH;
     case media::PIXEL_FORMAT_I420:
     case media::PIXEL_FORMAT_YV12:
     case media::PIXEL_FORMAT_I422:

media/filters/aom_video_decoder.cc

@@ -101,6 +101,7 @@ static VideoPixelFormat AomImgFmtToVideoPixelFormat(const aom_image_t* img) {
       // TODO(dalecurtis): We'll need to add support for these to handle the
       // full range of expected AOM content.
       NOTIMPLEMENTED();
+      break;
 
     default:
       break;

media/filters/gpu_video_decoder.cc

@@ -407,7 +407,7 @@ void GpuVideoDecoder::Decode(const scoped_refptr<DecoderBuffer>& buffer,
   switch (state_) {
     case kDecoderDrained:
       state_ = kNormal;
-      // Fall-through.
+      FALLTHROUGH;
     case kNormal:
       break;
     case kDrainingDecoder:

ui/events/gesture_detection/gesture_provider.cc

@@ -203,6 +203,7 @@ class GestureProvider::GestureListenerImpl : public ScaleGestureListener,
         // which case the press should simply be dropped.
         if (pinch_event_sent_ || scroll_event_sent_)
           return;
+        break;
       default:
         break;
     };

ui/events/gesture_event_details.cc

@@ -79,6 +79,7 @@ GestureEventDetails::GestureEventDetails(ui::EventType type,
       // allowed as an exception.
       if (other.type() == ui::ET_GESTURE_PINCH_BEGIN)
         break;
+      FALLTHROUGH;
     case ui::ET_GESTURE_SCROLL_UPDATE:
     case ui::ET_SCROLL_FLING_START:
     case ui::ET_GESTURE_SWIPE:

ui/events/gestures/motion_event_aura.cc

@@ -96,6 +96,7 @@ bool MotionEventAura::OnTouch(const TouchEvent& touch) {
     case ET_TOUCH_PRESSED:
       if (!AddTouch(touch))
         return false;
+      FALLTHROUGH;
     case ET_TOUCH_RELEASED:
     case ET_TOUCH_CANCELLED:
       // Removing these touch points needs to be postponed until after the

ui/events/x/events_x_utils.cc

@@ -635,6 +635,7 @@ int GetChangedMouseButtonFlagsFromXEvent(const XEvent& xev) {
         default:
           break;
       }
+      break;
     }
     default:
       break;