Chromium Code Reviews| Index: Source/core/svg/SVGFEColorMatrixElement.cpp |
| diff --git a/Source/core/svg/SVGFEColorMatrixElement.cpp b/Source/core/svg/SVGFEColorMatrixElement.cpp |
| index ad659d2fb97672c7ce24838030e85707b0390b71..58f6cb35725f13657618c1f485b268b0e85b1760 100644 |
| --- a/Source/core/svg/SVGFEColorMatrixElement.cpp |
| +++ b/Source/core/svg/SVGFEColorMatrixElement.cpp |
| @@ -77,8 +77,12 @@ bool SVGFEColorMatrixElement::setFilterEffectAttribute(FilterEffect* effect, con |
| FEColorMatrix* colorMatrix = static_cast<FEColorMatrix*>(effect); |
| if (attrName == SVGNames::typeAttr) |
| return colorMatrix->setType(m_type->currentValue()->enumValue()); |
| - if (attrName == SVGNames::valuesAttr) |
| - return colorMatrix->setValues(m_values->currentValue()->toFloatVector()); |
| + if (attrName == SVGNames::valuesAttr) { |
| + Vector<float> values = m_values->currentValue()->toFloatVector(); |
| + if (values.size() == 20) |
|
pdr.
2015/04/10 20:59:39
In a followup could you move this to be a named co
|
| + return colorMatrix->setValues(m_values->currentValue()->toFloatVector()); |
| + return false; |
| + } |
| ASSERT_NOT_REACHED(); |
| return false; |