Cleanup SVGElement::layoutObjectIsNeeded.

third_party/WebKit/Source/core/svg/SVGElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Nikolas Zimmermann
  * <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2008 Rob Buis <buis@kde.org>
  * Copyright (C) 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Alp Toker <alp@atoker.com>
  * Copyright (C) 2009 Cameron McCormack <cam@mcc.id.au>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 1008 matching lines @@
   const ComputedStyle* style = nullptr;
   if (Element* parent = parentOrShadowHostElement()) {
     if (LayoutObject* layoutObject = parent->layoutObject())
       style = layoutObject->style();
   }
 
   return document().ensureStyleResolver().styleForElement(
       correspondingElement(), style, DisallowStyleSharing);
 }
 
+bool SVGElement::layoutObjectIsNeeded(const ComputedStyle& style) {
+  // SVG elements only render when inside <svg>, or if the element is an <svg>
+  // itself.
+  if (!isSVGSVGElement(*this) &&
+      (!parentOrShadowHostElement() ||
+       !parentOrShadowHostElement()->isSVGElement())) {

[rune, 2017/02/09 18:09:28]

Shouldn't this have been the flat tree parent? Although, I don't know if there's
a way to use slot inside svg.

+    return false;
+  }
+
+  if (!isValid())
+    return false;
+
+  return Element::layoutObjectIsNeeded(style);
+}
+
 MutableStylePropertySet* SVGElement::animatedSMILStyleProperties() const {
   if (hasSVGRareData())
     return svgRareData()->animatedSMILStyleProperties();
   return nullptr;
 }
 
 MutableStylePropertySet* SVGElement::ensureAnimatedSMILStyleProperties() {
   return ensureSVGRareData()->ensureAnimatedSMILStyleProperties();
 }
 
@@ skipping 250 matching lines @@
   visitor->trace(m_className);
   Element::trace(visitor);
 }
 
 const AtomicString& SVGElement::eventParameterName() {
   DEFINE_STATIC_LOCAL(const AtomicString, evtString, ("evt"));
   return evtString;
 }
 
 }  // namespace blink