Clean up toHTMLFooElement() using DEFINE_NODE_TYPE_CASTS_WITH_FUNCTION

Source/core/html/HTMLProgressElement.h

 /*
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
  *
  * 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.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 56 matching lines @@
 
     ProgressValueElement* m_value;
 };
 
 inline bool isHTMLProgressElement(Node* node)
 {
     ASSERT(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