Bug 1251887 - Add break to unintentional switch fallthrough in GfxInfoBase.cpp to fix -Wimplicit-fallthrough warning. r=milan
authorChris Peterson <cpeterson@mozilla.com>
Fri, 26 Feb 2016 18:57:15 -0800
changeset 286213 65f7f2ffd30c41a291779d89ecd8ed07e6f19567
parent 286212 74a8cd912dcd154cdfd0c771d7d4ea73f016191e
child 286214 a1800ff23514f93ad5dbdb74ddfa33be9268cf26
push id17909
push usercbook@mozilla.com
push dateTue, 01 Mar 2016 14:26:33 +0000
treeherderfx-team@5e6d66db9b05 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmilan
bugs1251887
milestone47.0a1
Bug 1251887 - Add break to unintentional switch fallthrough in GfxInfoBase.cpp to fix -Wimplicit-fallthrough warning. r=milan widget/GfxInfoBase.cpp:158:5: warning: unannotated fall-through between switch labels [-Wimplicit-fallthrough]
widget/GfxInfoBase.cpp
--- a/widget/GfxInfoBase.cpp
+++ b/widget/GfxInfoBase.cpp
@@ -150,17 +150,19 @@ GetPrefNameForFeature(int32_t aFeature)
     case nsIGfxInfo::FEATURE_WEBRTC_HW_ACCELERATION_ENCODE:
       name = BLACKLIST_PREF_BRANCH "webrtc.hw.acceleration.encode";
       break;
     case nsIGfxInfo::FEATURE_WEBRTC_HW_ACCELERATION_DECODE:
       name = BLACKLIST_PREF_BRANCH "webrtc.hw.acceleration.decode";
       break;
     case nsIGfxInfo::FEATURE_CANVAS2D_ACCELERATION:
       name = BLACKLIST_PREF_BRANCH "canvas2d.acceleration";
+      break;
     default:
+      MOZ_ASSERT_UNREACHABLE("Unexpected nsIGfxInfo feature?!");
       break;
   }
 
   return name;
 }
 
 // Returns the value of the pref for the relevant feature in aValue.
 // If the pref doesn't exist, aValue is not touched, and returns false.