Index: cc/output/filter_operations.cc |
diff --git a/cc/output/filter_operations.cc b/cc/output/filter_operations.cc |
index 24208418026390d2127183481018a7a9b6265eb3..e3b993b8cc00a8652a8733acf2b88e51b61112ad 100644 |
--- a/cc/output/filter_operations.cc |
+++ b/cc/output/filter_operations.cc |
@@ -86,10 +86,20 @@ bool FilterOperations::HasFilterThatMovesPixels() const { |
case FilterOperation::DROP_SHADOW: |
case FilterOperation::ZOOM: |
return true; |
- default: |
- break; |
+ case FilterOperation::OPACITY: |
+ case FilterOperation::COLOR_MATRIX: |
+ case FilterOperation::GRAYSCALE: |
+ case FilterOperation::SEPIA: |
+ case FilterOperation::SATURATE: |
+ case FilterOperation::HUE_ROTATE: |
+ case FilterOperation::INVERT: |
+ case FilterOperation::BRIGHTNESS: |
+ case FilterOperation::CONTRAST: |
+ case FilterOperation::SATURATING_BRIGHTNESS: |
+ return false; |
} |
} |
+ NOTREACHED(); |
return false; |
} |
@@ -104,13 +114,26 @@ bool FilterOperations::HasFilterThatAffectsOpacity() const { |
return true; |
case FilterOperation::COLOR_MATRIX: { |
const SkScalar* matrix = op.matrix(); |
- return matrix[15] || matrix[16] || matrix[17] || matrix[18] != 1 || |
- matrix[19]; |
- } |
- default: |
+ if (matrix[15] || |
+ matrix[16] || |
+ matrix[17] || |
+ matrix[18] != 1 || |
+ matrix[19]) |
+ return true; |
break; |
+ } |
+ case FilterOperation::GRAYSCALE: |
+ case FilterOperation::SEPIA: |
+ case FilterOperation::SATURATE: |
+ case FilterOperation::HUE_ROTATE: |
+ case FilterOperation::INVERT: |
+ case FilterOperation::BRIGHTNESS: |
+ case FilterOperation::CONTRAST: |
+ case FilterOperation::SATURATING_BRIGHTNESS: |
+ return false; |
} |
} |
+ NOTREACHED(); |
return false; |
} |