Generate toSVGFooElement() for SVGFELight|AnimateMotion|FontFaceSrc|TextPositioningElement

Source/core/svg/SVGFELightElement.cpp

Index: Source/core/svg/SVGFELightElement.cpp
diff --git a/Source/core/svg/SVGFELightElement.cpp b/Source/core/svg/SVGFELightElement.cpp
index 4429044f7fb0c4a085af13363a692844063d81a4..054bd8ebb8ffce60ea3e960e09c82afa51be845d 100644
--- a/Source/core/svg/SVGFELightElement.cpp
+++ b/Source/core/svg/SVGFELightElement.cpp
@@ -70,7 +70,7 @@ SVGFELightElement* SVGFELightElement::findLightElement(const SVGElement* svgElem
         if (node->hasTagName(SVGNames::feDistantLightTag)
             || node->hasTagName(SVGNames::fePointLightTag)
             || node->hasTagName(SVGNames::feSpotLightTag)) {
-            return static_cast<SVGFELightElement*>(node);
+            return toSVGFELightElement(node);

[Stephen Chennney, 2014/01/10 12:32:53]

You are doing the hasTagName queries twice. I have some issues in this case with
using toBlah at all, as the calling code ensures safety and no other code calls
it. Regardless, this should be re-written to avoid all of the hasTagName queries
here.

[gyuyoung-inactive, 2014/01/12 04:37:31]

DEFINE_NODE_TYPE_CASTS macro only calls "hasTagName()" on debug build, not
release mode.

Frankly, it looks we don't need to call "hasTagName()" here. However,
toSVGFELightElement() needs to call it when it is used by other places which
don't check tag name in order to guarantee safer type cast. That's why
DEFINE_NODE_TYPE_CASTS checks tag name or child element. If I don't understand
what is your question correctly, please let me know.

#define DEFINE_TYPE_CASTS(thisType, argumentType, argumentName,
pointerPredicate, referencePredicate) \
inline thisType* to##thisType(argumentType* argumentName) \
{ \
    ASSERT_WITH_SECURITY_IMPLICATION(!argumentName || (pointerPredicate)); \
    return static_cast<thisType*>(argumentName); \
} \

BTW, it would be good to use isSVGFELightElement() instead of using
"hasTagName()" conditions.

         }
     }
     return 0;