Clean up toHTMLFooElement() using DEFINE_NODE_TYPE_CASTS_WITH_FUNCTION

Source/core/html/HTMLAreaElement.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2004, 2008, 2009, 2011 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 50 matching lines @@
     enum Shape { Default, Poly, Rect, Circle, Unknown };
     Path getRegion(const LayoutSize&) const;
     void invalidateCachedRegion();
 
     OwnPtr<Path> m_region;
     Vector<Length> m_coords;
     LayoutSize m_lastSize;
     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);
 }
 
 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
 
 #endif