Use new is*Element() helper functions more in bindings/ code

Source/bindings/v8/V8GCController.cpp

 /*
  * Copyright (C) 2009 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 52 matching lines @@
         V8AbstractEventListener* v8listener = static_cast<V8AbstractEventListener*>(listener);
         if (!v8listener->hasExistingListenerObject())
             continue;
 
         isolate->SetReference(wrapper, v8::Persistent<v8::Value>::Cast(v8listener->existingListenerObjectPersistentHandle()));
     }
 }
 
 Node* V8GCController::opaqueRootForGC(Node* node, v8::Isolate*)
 {
+    ASSERT(node);
     // FIXME: Remove the special handling for image elements.
     // The same special handling is in V8GCController::gcTree().
     // Maybe should image elements be active DOM nodes?
     // See https://code.google.com/p/chromium/issues/detail?id=164882
-    if (node->inDocument() || (node->hasTagName(HTMLNames::imgTag) && toHTMLImageElement(node)->hasPendingActivity()))
+    if (node->inDocument() || (isHTMLImageElement(*node) && toHTMLImageElement(*node).hasPendingActivity()))
         return &node->document();
 
     if (node->isAttributeNode()) {
         Node* ownerElement = toAttr(node)->ownerElement();
         if (!ownerElement)
             return node;
         node = ownerElement;
     }
 
     while (Node* parent = node->parentOrShadowHostOrTemplateHostNode())
@@ skipping 39 matching lines @@
         // do not have wrappers in the main world.
         if (node->containsWrapper()) {
             const WrapperTypeInfo* type = toWrapperTypeInfo(*wrapper);
             ActiveDOMObject* activeDOMObject = type->toActiveDOMObject(*wrapper);
             if (activeDOMObject && activeDOMObject->hasPendingActivity())
                 return;
             // FIXME: Remove the special handling for image elements.
             // The same special handling is in V8GCController::opaqueRootForGC().
             // Maybe should image elements be active DOM nodes?
             // See https://code.google.com/p/chromium/issues/detail?id=164882
-            if (node->hasTagName(HTMLNames::imgTag) && toHTMLImageElement(node)->hasPendingActivity())
+            if (isHTMLImageElement(*node) && toHTMLImageElement(*node).hasPendingActivity())
                 return;
             // FIXME: Remove the special handling for SVG context elements.
             if (node->isSVGElement() && toSVGElement(node)->isContextElement())
                 return;
 
             m_nodesInNewSpace.append(node);
             node->markV8CollectableDuringMinorGC();
         }
     }
 
@@ skipping 32 matching lines @@
                 if (!traverseTree(shadowRoot, partiallyDependentNodes))
                     return false;
             } else if (node->isShadowRoot()) {
                 if (ShadowRoot* shadowRoot = toShadowRoot(node)->olderShadowRoot()) {
                     if (!traverseTree(shadowRoot, partiallyDependentNodes))
                         return false;
                 }
             }
             // <template> has a |content| property holding a DOM fragment which we must traverse,
             // just like we do for the shadow trees above.
-            if (node->hasTagName(HTMLNames::templateTag)) {
-                if (!traverseTree(toHTMLTemplateElement(node)->content(), partiallyDependentNodes))
+            if (isHTMLTemplateElement(*node)) {
+                if (!traverseTree(toHTMLTemplateElement(*node).content(), partiallyDependentNodes))
                     return false;
             }
         }
         return true;
     }
 
     void gcTree(v8::Isolate* isolate, Node* startNode)
     {
         Vector<Node*, initialNodeVectorSize> partiallyDependentNodes;
 
@@ skipping 188 matching lines @@
         TRACE_EVENT_SET_NONCONST_SAMPLING_STATE(V8PerIsolateData::from(isolate)->previousSamplingState());
 }
 
 void V8GCController::collectGarbage(v8::Isolate* isolate)
 {
     V8ExecutionScope scope(isolate);
     V8ScriptRunner::compileAndRunInternalScript(v8String(isolate, "if (gc) gc();"), isolate);
 }
 
 }  // namespace WebCore