Don't check 'visibility' in LayoutSVGResourceMasker

third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceMasker.cpp

 /*
  * Copyright (C) Research In Motion Limited 2009-2010. All rights reserved.
  *
  * 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 67 matching lines @@
   FloatRect bounds = strokeBoundingBox();
 
   SkPictureBuilder pictureBuilder(bounds, nullptr, &context);
 
   ColorFilter maskContentFilter =
       style()->svgStyle().colorInterpolation() == CI_LINEARRGB
           ? ColorFilterSRGBToLinearRGB
           : ColorFilterNone;
   pictureBuilder.context().setColorFilter(maskContentFilter);
 
-  for (SVGElement* childElement = Traversal<SVGElement>::firstChild(*element());
-       childElement;
-       childElement = Traversal<SVGElement>::nextSibling(*childElement)) {
-    LayoutObject* layoutObject = childElement->layoutObject();
-    if (!layoutObject)
+  for (const SVGElement& childElement :
+       Traversal<SVGElement>::childrenOf(*element())) {
+    const LayoutObject* layoutObject = childElement.layoutObject();
+    if (!layoutObject || layoutObject->styleRef().display() == EDisplay::None)
       continue;
-    const ComputedStyle* style = layoutObject->style();
-    if (!style || style->display() == EDisplay::None ||
-        style->visibility() != EVisibility::Visible)
-      continue;
-
     SVGPaintContext::paintSubtree(pictureBuilder.context(), layoutObject);
   }
 
   m_maskContentPicture = pictureBuilder.endRecording();
   return m_maskContentPicture;
 }
 
 void LayoutSVGResourceMasker::calculateMaskContentVisualRect() {
-  for (SVGElement* childElement = Traversal<SVGElement>::firstChild(*element());
-       childElement;
-       childElement = Traversal<SVGElement>::nextSibling(*childElement)) {
-    LayoutObject* layoutObject = childElement->layoutObject();
-    if (!layoutObject)
-      continue;
-    const ComputedStyle* style = layoutObject->style();
-    if (!style || style->display() == EDisplay::None ||
-        style->visibility() != EVisibility::Visible)
+  for (const SVGElement& childElement :
+       Traversal<SVGElement>::childrenOf(*element())) {
+    const LayoutObject* layoutObject = childElement.layoutObject();
+    if (!layoutObject || layoutObject->styleRef().display() == EDisplay::None)
       continue;
     m_maskContentBoundaries.unite(
         layoutObject->localToSVGParentTransform().mapRect(
             layoutObject->visualRectInLocalSVGCoordinates()));
   }
 }
 
 FloatRect LayoutSVGResourceMasker::resourceBoundingBox(
     const LayoutObject* object) {
   SVGMaskElement* maskElement = toSVGMaskElement(element());
@@ skipping 19 matching lines @@
     transform.scaleNonUniform(objectBoundingBox.width(),
                               objectBoundingBox.height());
     maskRect = transform.mapRect(maskRect);
   }
 
   maskRect.intersect(maskBoundaries);
   return maskRect;
 }
 
 }  // namespace blink