Introduce toHTMLSelectElement and remove isHTMLSelectElement

Source/core/testing/Internals.cpp

Index: Source/core/testing/Internals.cpp
diff --git a/Source/core/testing/Internals.cpp b/Source/core/testing/Internals.cpp
index a9cf5c51b5f18e846456a954b5e9d5a419329b68..3068d0a5f4a62c6d76b8f9f100382a326803eb01 100644
--- a/Source/core/testing/Internals.cpp
+++ b/Source/core/testing/Internals.cpp
@@ -1987,7 +1987,7 @@ String Internals::baseURL(Document* document, ExceptionCode& ec)
 
 bool Internals::isSelectPopupVisible(Node* node)
 {
-    if (!isHTMLSelectElement(node))

[tkent, 2013/07/17 01:47:56]

This isHTMLSelectElement usage is good because toHTMLSelectElement follows.  You
don't need to revert this.

[kangil_, 2013/07/17 02:16:22]

I understood that we agree to remove isFooElement because not all elements are
presented by tag names. :)
So, I would like to remove usage of isFooElement here. :)

+    if (!node->hasTagName(HTMLNames::selectTag))
         return false;
 
     HTMLSelectElement* select = toHTMLSelectElement(node);