Introduce toHTMLSelectElement and remove isHTMLSelectElement

Source/core/testing/Internals.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  * Copyright (C) 2013 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 1969 matching lines @@
     if (!document) {
         ec = InvalidAccessError;
         return String();
     }
 
     return document->baseURL().string();
 }
 
 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);
 
     RenderObject* renderer = select->renderer();
     if (!renderer->isMenuList())
         return false;
 
     RenderMenuList* menuList = toRenderMenuList(renderer);
     return menuList->popupIsVisible();
 }
 
 }