Use isSVG*Element() helpers more in SVG code (Part 4)

Source/core/svg/SVGGraphicsElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006 Rob Buis <buis@kde.org>
  * Copyright (C) 2014 Google, Inc.
  *
  * 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 42 matching lines @@
         if (!targetCTM.isInvertible()) {
             exceptionState.throwDOMException(InvalidStateError, "The target transformation is not invertable.");
             return nullptr;
         }
         ctm = targetCTM.inverse() * ctm;
     }
 
     return SVGMatrixTearOff::create(ctm);
 }
 
-static bool isViewportElement(const Element* element)
+static bool isViewportElement(const Element& element)
 {
-    return (element->hasTagName(SVGNames::svgTag)
-        || element->hasTagName(SVGNames::symbolTag)
-        || element->hasTagName(SVGNames::foreignObjectTag)
-        || element->hasTagName(SVGNames::imageTag));
+    return (isSVGSVGElement(element)
+        || isSVGSymbolElement(element)
+        || isSVGForeignObjectElement(element)
+        || isSVGImageElement(element));
 }
 
 AffineTransform SVGGraphicsElement::computeCTM(SVGElement::CTMScope mode,
     SVGGraphicsElement::StyleUpdateStrategy styleUpdateStrategy, const SVGGraphicsElement* ancestor) const
 {
     if (styleUpdateStrategy == AllowStyleUpdate)
         document().updateLayoutIgnorePendingStylesheets();
 
     AffineTransform ctm;
     bool done = false;
 
     for (const Element* currentElement = this; currentElement && !done;
         currentElement = currentElement->parentOrShadowHostElement()) {
         if (!currentElement->isSVGElement())
             break;
 
         ctm = toSVGElement(currentElement)->localCoordinateSpaceTransform(mode).multiply(ctm);
 
         switch (mode) {
         case NearestViewportScope:
             // Stop at the nearest viewport ancestor.
-            done = currentElement != this && isViewportElement(currentElement);
+            done = currentElement != this && isViewportElement(*currentElement);
             break;
         case AncestorScope:
             // Stop at the designated ancestor.
             done = currentElement == ancestor;
             break;
         default:
             ASSERT(mode == ScreenScope);
             break;
         }
     }
@@ skipping 112 matching lines @@
         RenderSVGResource::markForLayoutAndParentResourceInvalidation(object);
         return;
     }
 
     ASSERT_NOT_REACHED();
 }
 
 SVGElement* SVGGraphicsElement::nearestViewportElement() const
 {
     for (Element* current = parentOrShadowHostElement(); current; current = current->parentOrShadowHostElement()) {
-        if (isViewportElement(current))
+        if (isViewportElement(*current))
             return toSVGElement(current);
     }
 
     return 0;
 }
 
 SVGElement* SVGGraphicsElement::farthestViewportElement() const
 {
     SVGElement* farthest = 0;
     for (Element* current = parentOrShadowHostElement(); current; current = current->parentOrShadowHostElement()) {
-        if (isViewportElement(current))
+        if (isViewportElement(*current))
             farthest = toSVGElement(current);
     }
     return farthest;
 }
 
 FloatRect SVGGraphicsElement::getBBox()
 {
     document().updateLayoutIgnorePendingStylesheets();
 
     // FIXME: Eventually we should support getBBox for detached elements.
@@ skipping 15 matching lines @@
 }
 
 void SVGGraphicsElement::toClipPath(Path& path)
 {
     updatePathFromGraphicsElement(this, path);
     // FIXME: How do we know the element has done a layout?
     path.transform(animatedLocalTransform());
 }
 
 }