Commit 008c9ab1 authored by sashab@chromium.org's avatar sashab@chromium.org

Remove default case from CSSValue::equals

Remove the default case from the CSSValue::equals method, to enable the
compiler to check that every enum value is in the switch statement.

Review URL: https://codereview.chromium.org/1311843003

git-svn-id: svn://svn.chromium.org/blink/trunk@201114 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent edcbe126
......@@ -92,7 +92,7 @@ inline static bool compareCSSValues(const CSSValue& first, const CSSValue& secon
bool CSSValue::equals(const CSSValue& other) const
{
if (m_classType == other.m_classType) {
switch (m_classType) {
switch (classType()) {
case BorderImageSliceClass:
return compareCSSValues<CSSBorderImageSliceValue>(*this, other);
case CanvasClass:
......@@ -149,10 +149,9 @@ bool CSSValue::equals(const CSSValue& other) const
return compareCSSValues<CSSSVGDocumentValue>(*this, other);
case CSSContentDistributionClass:
return compareCSSValues<CSSContentDistributionValue>(*this, other);
default:
ASSERT_NOT_REACHED();
return false;
}
ASSERT_NOT_REACHED();
return false;
}
return false;
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment