Clean up toHTMLFooElement() using DEFINE_NODE_TYPE_CASTS_WITH_FUNCTION

Source/core/html/HTMLAreaElement.h

Index: Source/core/html/HTMLAreaElement.h
diff --git a/Source/core/html/HTMLAreaElement.h b/Source/core/html/HTMLAreaElement.h
index b5f22211e0974577f9a6860790f53f669e4f9fa2..30e9fe613cc490c4e50afbf1ebb21af51590b926 100644
--- a/Source/core/html/HTMLAreaElement.h
+++ b/Source/core/html/HTMLAreaElement.h
@@ -68,6 +68,11 @@ private:
     Shape m_shape;
 };
 
+inline bool isHTMLAreaElement(const Node& node)

[tkent, 2013/10/24 01:22:32]

I don't think we need to add isHTMLAreaElement(const Node&). 
DEFINE_NODE_TYPE_CASTS(HTMLAreaElement, hasTagName(HTMLnames::areaTag)) should
be enough and simpler.

[gyuyoung-inactive, 2013/10/24 01:53:12]

I agree with your suggestion. It looks it is more simpler than this approach.

+{
+    return node.hasTagName(HTMLNames::areaTag);
+}
+
 inline bool isHTMLAreaElement(const Node* node)
 {
     return node->hasTagName(HTMLNames::areaTag);
@@ -78,11 +83,7 @@ inline bool isHTMLAreaElement(const Element* element)
     return element->hasTagName(HTMLNames::areaTag);
 }
 
-inline HTMLAreaElement* toHTMLAreaElement(Node* node)
-{
-    ASSERT_WITH_SECURITY_IMPLICATION(!node || isHTMLAreaElement(node));
-    return static_cast<HTMLAreaElement*>(node);
-}
+DEFINE_NODE_TYPE_CASTS_WITH_FUNCTION(HTMLAreaElement);
 
 } //namespace