reflow comments in core/svg/

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

 /*
- * Copyright (C) 2004, 2005, 2006, 2007, 2008 Nikolas Zimmermann <zimmermann@kde.org>
+ * Copyright (C) 2004, 2005, 2006, 2007, 2008 Nikolas Zimmermann
+ * <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007 Rob Buis <buis@kde.org>
  * Copyright (C) Research In Motion Limited 2009-2010. All rights reserved.
  * Copyright (C) 2011 Torch Mobile (Beijing) Co. Ltd. All rights reserved.
  * Copyright (C) 2012 University of Szeged
  * Copyright (C) 2012 Renata Hodovan <reni@webkit.org>
  *
  * 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 91 matching lines @@
 #if ENABLE(ASSERT)
 static inline bool isWellFormedDocument(Document* document) {
   if (document->isXMLDocument())
     return static_cast<XMLDocumentParser*>(document->parser())->wellFormed();
   return true;
 }
 #endif
 
 Node::InsertionNotificationRequest SVGUseElement::insertedInto(
     ContainerNode* rootParent) {
-  // This functions exists to assure assumptions made in the code regarding SVGElementInstance creation/destruction are satisfied.
+  // This functions exists to assure assumptions made in the code regarding
+  // SVGElementInstance creation/destruction are satisfied.
   SVGGraphicsElement::insertedInto(rootParent);
   if (!rootParent->isConnected())
     return InsertionDone;
   ASSERT(!m_targetElementInstance || !isWellFormedDocument(&document()));
   ASSERT(!hasPendingResources() || !isWellFormedDocument(&document()));
   invalidateShadowTree();
   return InsertionDone;
 }
 
 void SVGUseElement::removedFrom(ContainerNode* rootParent) {
@@ skipping 126 matching lines @@
     SVGElement::InvalidationGuard invalidationGuard(this);
     updateTargetReference();
     invalidateShadowTree();
     return;
   }
 
   SVGGraphicsElement::svgAttributeChanged(attrName);
 }
 
 static bool isDisallowedElement(const Element& element) {
-  // Spec: "Any 'svg', 'symbol', 'g', graphics element or other 'use' is potentially a template object that can be re-used
-  // (i.e., "instanced") in the SVG document via a 'use' element."
-  // "Graphics Element" is defined as 'circle', 'ellipse', 'image', 'line', 'path', 'polygon', 'polyline', 'rect', 'text'
-  // Excluded are anything that is used by reference or that only make sense to appear once in a document.
+  // Spec: "Any 'svg', 'symbol', 'g', graphics element or other 'use' is
+  // potentially a template object that can be re-used (i.e., "instanced") in
+  // the SVG document via a 'use' element." "Graphics Element" is defined as
+  // 'circle', 'ellipse', 'image', 'line', 'path', 'polygon', 'polyline',
+  // 'rect', 'text' Excluded are anything that is used by reference or that only
+  // make sense to appear once in a document.
   if (!element.isSVGElement())
     return true;
 
   DEFINE_STATIC_LOCAL(
       HashSet<QualifiedName>, allowedElementTags,
       ({
           SVGNames::aTag,       SVGNames::circleTag,   SVGNames::descTag,
           SVGNames::ellipseTag, SVGNames::gTag,        SVGNames::imageTag,
           SVGNames::lineTag,    SVGNames::metadataTag, SVGNames::pathTag,
           SVGNames::polygonTag, SVGNames::polylineTag, SVGNames::rectTag,
@@ skipping 90 matching lines @@
 // Though if there are disallowed elements in the subtree, we have to remove
 // them.  For instance: <use> on <g> containing <foreignObject> (indirect
 // case).
 static inline void removeDisallowedElementsFromSubtree(SVGElement& subtree) {
   ASSERT(!subtree.isConnected());
   Element* element = ElementTraversal::firstWithin(subtree);
   while (element) {
     if (isDisallowedElement(*element)) {
       Element* next =
           ElementTraversal::nextSkippingChildren(*element, &subtree);
-      // The subtree is not in document so this won't generate events that could mutate the tree.
+      // The subtree is not in document so this won't generate events that could
+      // mutate the tree.
       element->parentNode()->removeChild(element);
       element = next;
     } else {
       element = ElementTraversal::next(*element, &subtree);
     }
   }
 }
 
 static void moveChildrenToReplacementElement(ContainerNode& sourceRoot,
                                              ContainerNode& destinationRoot) {
@@ skipping 28 matching lines @@
                                     targetRoot);
   associateCorrespondingElements(targetRoot, toSVGElement(*instanceRoot));
   removeDisallowedElementsFromSubtree(toSVGElement(*instanceRoot));
   return instanceRoot;
 }
 
 void SVGUseElement::buildShadowAndInstanceTree(SVGElement& target) {
   ASSERT(!m_targetElementInstance);
   ASSERT(!m_needsShadowTreeRecreation);
 
-  // <use> creates a "user agent" shadow root. Do not build the shadow/instance tree for <use>
-  // elements living in a user agent shadow tree because they will get expanded in a second
-  // pass -- see expandUseElementsInShadowTree().
+  // <use> creates a "user agent" shadow root. Do not build the shadow/instance
+  // tree for <use> elements living in a user agent shadow tree because they
+  // will get expanded in a second pass -- see expandUseElementsInShadowTree().
   if (inUseShadowTree())
     return;
 
   // Do not allow self-referencing.
   if (&target == this || isDisallowedElement(target))
     return;
 
   // Set up root SVG element in shadow tree.
-  // Clone the target subtree into the shadow tree, not handling <use> and <symbol> yet.
+  // Clone the target subtree into the shadow tree, not handling <use> and
+  // <symbol> yet.
   Element* instanceRoot = createInstanceTree(target);
   m_targetElementInstance = toSVGElement(instanceRoot);
   ShadowRoot* shadowTreeRootElement = userAgentShadowRoot();
   shadowTreeRootElement->appendChild(instanceRoot);
 
   addReferencesToFirstDegreeNestedUseElements(target);
 
   if (instanceTreeIsLoading())
     return;
 
@@ skipping 58 matching lines @@
   if (!element.isSVGGraphicsElement())
     return nullptr;
 
   if (!element.layoutObject())
     return nullptr;
 
   const ComputedStyle* style = element.layoutObject()->style();
   if (!style || style->visibility() != EVisibility::Visible)
     return nullptr;
 
-  // Spec: "If a <use> element is a child of a clipPath element, it must directly
-  // reference <path>, <text> or basic shapes elements. Indirect references are an
-  // error and the clipPath element must be ignored."
+  // Spec: "If a <use> element is a child of a clipPath element, it must
+  // directly reference <path>, <text> or basic shapes elements. Indirect
+  // references are an error and the clipPath element must be ignored."
   // http://dev.w3.org/fxtf/css-masking-1/#the-clip-path
   if (!isDirectReference(element)) {
     // Spec: Indirect references are an error (14.3.5)
     return nullptr;
   }
 
   return &toSVGGraphicsElement(element);
 }
 
 void SVGUseElement::addReferencesToFirstDegreeNestedUseElements(
@@ skipping 52 matching lines @@
   replacementElement.removeAttribute(SVGNames::widthAttr);
   replacementElement.removeAttribute(SVGNames::heightAttr);
   replacementElement.removeAttribute(SVGNames::hrefAttr);
   replacementElement.removeAttribute(XLinkNames::hrefAttr);
 }
 
 bool SVGUseElement::expandUseElementsInShadowTree() {
   // Why expand the <use> elements in the shadow tree here, and not just
   // do this directly in buildShadowTree, if we encounter a <use> element?
   //
-  // Short answer: Because we may miss to expand some elements. For example, if a <symbol>
-  // contains <use> tags, we'd miss them. So once we're done with setting up the
-  // actual shadow tree (after the special case modification for svg/symbol) we have
-  // to walk it completely and expand all <use> elements.
+  // Short answer: Because we may miss to expand some elements. For example, if
+  // a <symbol> contains <use> tags, we'd miss them. So once we're done with
+  // setting up the actual shadow tree (after the special case modification for
+  // svg/symbol) we have to walk it completely and expand all <use> elements.
   ShadowRoot* shadowRoot = userAgentShadowRoot();
   for (SVGUseElement* use = Traversal<SVGUseElement>::firstWithin(*shadowRoot);
        use;) {
     ASSERT(!use->resourceIsStillLoading());
 
     SVGUseElement& originalUse = toSVGUseElement(*use->correspondingElement());
     SVGElement* target = nullptr;
     if (hasCycleUseReferencing(originalUse, *use, target))
       return false;
 
@@ skipping 132 matching lines @@
 
   if (m_resource)
     m_resource->removeClient(this);
 
   m_resource = resource;
   if (m_resource)
     m_resource->addClient(this);
 }
 
 }  // namespace blink