Index: Source/core/svg/SVGFELightElement.cpp |
diff --git a/Source/core/svg/SVGFELightElement.cpp b/Source/core/svg/SVGFELightElement.cpp |
index eea5a77ccfdc6ac05802465eb726725c0a28f6a3..c112b9a1536eafb249d4c75dbdd8a204a4c0d4b9 100644 |
--- a/Source/core/svg/SVGFELightElement.cpp |
+++ b/Source/core/svg/SVGFELightElement.cpp |
@@ -85,33 +85,8 @@ FloatPoint3D SVGFELightElement::pointsAt() const |
return FloatPoint3D(pointsAtX()->currentValue()->value(), pointsAtY()->currentValue()->value(), pointsAtZ()->currentValue()->value()); |
} |
-bool SVGFELightElement::isSupportedAttribute(const QualifiedName& attrName) |
-{ |
- DEFINE_STATIC_LOCAL(HashSet<QualifiedName>, supportedAttributes, ()); |
- if (supportedAttributes.isEmpty()) { |
- supportedAttributes.add(SVGNames::azimuthAttr); |
- supportedAttributes.add(SVGNames::elevationAttr); |
- supportedAttributes.add(SVGNames::xAttr); |
- supportedAttributes.add(SVGNames::yAttr); |
- supportedAttributes.add(SVGNames::zAttr); |
- supportedAttributes.add(SVGNames::pointsAtXAttr); |
- supportedAttributes.add(SVGNames::pointsAtYAttr); |
- supportedAttributes.add(SVGNames::pointsAtZAttr); |
- supportedAttributes.add(SVGNames::specularExponentAttr); |
- supportedAttributes.add(SVGNames::limitingConeAngleAttr); |
- } |
- return supportedAttributes.contains<SVGAttributeHashTranslator>(attrName); |
-} |
- |
void SVGFELightElement::svgAttributeChanged(const QualifiedName& attrName) |
{ |
- if (!isSupportedAttribute(attrName)) { |
- SVGElement::svgAttributeChanged(attrName); |
- return; |
- } |
- |
- SVGElement::InvalidationGuard invalidationGuard(this); |
- |
if (attrName == SVGNames::azimuthAttr |
|| attrName == SVGNames::elevationAttr |
|| attrName == SVGNames::xAttr |
@@ -122,6 +97,7 @@ void SVGFELightElement::svgAttributeChanged(const QualifiedName& attrName) |
|| attrName == SVGNames::pointsAtZAttr |
|| attrName == SVGNames::specularExponentAttr |
|| attrName == SVGNames::limitingConeAngleAttr) { |
+ SVGElement::InvalidationGuard invalidationGuard(this); |
fs
2015/04/10 14:25:24
Move this down to just before isSVGFEDiffuseLighti
Erik Dahlström (inactive)
2015/04/10 15:33:44
Done.
|
ContainerNode* parent = parentNode(); |
if (!parent) |
return; |
@@ -140,7 +116,7 @@ void SVGFELightElement::svgAttributeChanged(const QualifiedName& attrName) |
} |
} |
fs
2015/04/10 14:25:24
Should probably move the ASSERT_NOT_REACHED() here
Erik Dahlström (inactive)
2015/04/10 15:33:44
Done.
|
- ASSERT_NOT_REACHED(); |
+ SVGElement::svgAttributeChanged(attrName); |
} |
void SVGFELightElement::childrenChanged(const ChildrenChange& change) |