cc: Remove unnecessary "default" cases from switch statements.

cc/output/filter_operation.cc

Index: cc/output/filter_operation.cc
diff --git a/cc/output/filter_operation.cc b/cc/output/filter_operation.cc
index 6a778388ae5162ebe7a2ae6298414ff698ba7662..3870496992bad5aa291ae026711b33778d92a222 100644
--- a/cc/output/filter_operation.cc
+++ b/cc/output/filter_operation.cc
@@ -147,10 +147,9 @@ static FilterOperation CreateNoOpFilter(FilterOperation::FilterType type) {
       return FilterOperation::CreateZoomFilter(1.f, 0);
     case FilterOperation::SATURATING_BRIGHTNESS:
       return FilterOperation::CreateSaturatingBrightnessFilter(0.f);
-    default:
-      NOTREACHED();
-      return FilterOperation::CreateEmptyFilter();
   }
+  NOTREACHED();
+  return FilterOperation::CreateEmptyFilter();
 }
 
 static float ClampAmountForFilterType(float amount,
@@ -173,10 +172,10 @@ static float ClampAmountForFilterType(float amount,
     case FilterOperation::SATURATING_BRIGHTNESS:
       return amount;
     case FilterOperation::COLOR_MATRIX:
-    default:
-      NOTREACHED();
-      return amount;
+      break;

[danakj, 2013/08/26 15:55:39]

can you NOTREACHED() inside this case as well, just for clarity?

[reveman, 2013/08/26 17:16:57]

Done.

   }
+  NOTREACHED();
+  return amount;
 }
 
 // static