Clean up toHTMLFooElement() using DEFINE_NODE_TYPE_CASTS_WITH_FUNCTION

Source/core/html/HTMLProgressElement.h

Index: Source/core/html/HTMLProgressElement.h
diff --git a/Source/core/html/HTMLProgressElement.h b/Source/core/html/HTMLProgressElement.h
index e9f624e4342f8b638b661ca50a7472c23898058e..ae50218e2faac1715946a27eb2b679abc3107241 100644
--- a/Source/core/html/HTMLProgressElement.h
+++ b/Source/core/html/HTMLProgressElement.h
@@ -74,12 +74,13 @@ inline bool isHTMLProgressElement(Node* node)
     return node->hasTagName(HTMLNames::progressTag);
 }
 
-inline HTMLProgressElement* toHTMLProgressElement(Node* node)
+inline bool isHTMLProgressElement(Node& node)

[tkent, 2013/10/24 02:02:19]

This looks unnecessary.

[gyuyoung-inactive, 2013/10/24 02:08:19]

Oops, I thought that this one was original function.

 {
-    ASSERT_WITH_SECURITY_IMPLICATION(!node || isHTMLProgressElement(node));
-    return static_cast<HTMLProgressElement*>(node);
+    return node.hasTagName(HTMLNames::progressTag);
 }
 
+DEFINE_NODE_TYPE_CASTS(HTMLProgressElement, hasTagName(HTMLNames::progressTag));
+
 } // namespace
 
 #endif