reflow comments in core/svg/

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

 /*
  * Copyright (C) 2006 Apple Inc. All rights reserved.
  * Copyright (C) 2006 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2007 Rob Buis <buis@kde.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 81 matching lines @@
   if (RuntimeEnabledFeatures::smilEnabled()) {
     HeapVector<Member<SVGSVGElement>> timeContainers;
     copyToVector(m_timeContainers, timeContainers);
     for (const auto& container : timeContainers)
       container->timeContainer()->serviceAnimations();
   }
 
   SVGElementSet webAnimationsPendingSVGElements;
   webAnimationsPendingSVGElements.swap(m_webAnimationsPendingSVGElements);
 
-  // TODO(alancutter): Make SVG animation effect application a separate document lifecycle phase from servicing animations to be responsive to Javascript manipulation of exposed animation objects.
+  // TODO(alancutter): Make SVG animation effect application a separate document
+  // lifecycle phase from servicing animations to be responsive to Javascript
+  // manipulation of exposed animation objects.
   for (auto& svgElement : webAnimationsPendingSVGElements)
     svgElement->applyActiveWebAnimations();
 
   ASSERT(m_webAnimationsPendingSVGElements.isEmpty());
 }
 
 void SVGDocumentExtensions::startAnimations() {
-  // FIXME: Eventually every "Time Container" will need a way to latch on to some global timer
-  // starting animations for a document will do this "latching"
-  // FIXME: We hold a ref pointers to prevent a shadow tree from getting removed out from underneath us.
-  // In the future we should refactor the use-element to avoid this. See https://webkit.org/b/53704
+  // FIXME: Eventually every "Time Container" will need a way to latch on to
+  // some global timer starting animations for a document will do this
+  // "latching"
+  // FIXME: We hold a ref pointers to prevent a shadow tree from getting removed
+  // out from underneath us.  In the future we should refactor the use-element
+  // to avoid this. See https://webkit.org/b/53704
   HeapVector<Member<SVGSVGElement>> timeContainers;
   copyToVector(m_timeContainers, timeContainers);
   for (const auto& container : timeContainers) {
     SMILTimeContainer* timeContainer = container->timeContainer();
     if (!timeContainer->isStarted())
       timeContainer->start();
   }
 }
 
 void SVGDocumentExtensions::pauseAnimations() {
   for (SVGSVGElement* element : m_timeContainers)
     element->pauseAnimations();
 }
 
 void SVGDocumentExtensions::dispatchSVGLoadEventToOutermostSVGElements() {
   HeapVector<Member<SVGSVGElement>> timeContainers;
   copyToVector(m_timeContainers, timeContainers);
   for (const auto& container : timeContainers) {
     SVGSVGElement* outerSVG = container.get();
     if (!outerSVG->isOutermostSVGSVGElement())
       continue;
 
-    // don't dispatch the load event document is not wellformed (for XML/standalone svg)
+    // Don't dispatch the load event document is not wellformed (for
+    // XML/standalone svg).
     if (outerSVG->document().wellFormed() ||
         !outerSVG->document().isSVGDocument())
       outerSVG->sendSVGLoadEventIfPossible();
   }
 }
 
 void SVGDocumentExtensions::reportError(const String& message) {
   ConsoleMessage* consoleMessage = ConsoleMessage::create(
       RenderingMessageSource, ErrorMessageLevel, "Error: " + message);
   m_document->addConsoleMessage(consoleMessage);
@@ skipping 17 matching lines @@
 }
 
 bool SVGDocumentExtensions::hasPendingResource(const AtomicString& id) const {
   if (id.isEmpty())
     return false;
 
   return m_pendingResources.contains(id);
 }
 
 bool SVGDocumentExtensions::isElementPendingResources(Element* element) const {
-  // This algorithm takes time proportional to the number of pending resources and need not.
-  // If performance becomes an issue we can keep a counted set of elements and answer the question efficiently.
+  // This algorithm takes time proportional to the number of pending resources
+  // and need not.  If performance becomes an issue we can keep a counted set of
+  // elements and answer the question efficiently.
 
   ASSERT(element);
 
   for (const auto& entry : m_pendingResources) {
     SVGPendingElements* elements = entry.value.get();
     ASSERT(elements);
 
     if (elements->contains(element))
       return true;
   }
@@ skipping 29 matching lines @@
       ASSERT(elements);
       ASSERT(!elements->isEmpty());
 
       elements->remove(element);
       if (elements->isEmpty())
         toBeRemoved.append(entry.key);
     }
 
     clearHasPendingResourcesIfPossible(element);
 
-    // We use the removePendingResource function here because it deals with set lifetime correctly.
+    // We use the removePendingResource function here because it deals with set
+    // lifetime correctly.
     for (const AtomicString& id : toBeRemoved)
       removePendingResource(id);
   }
 
   // Remove the element from pending resources that were scheduled for removal.
   if (!m_pendingResourcesForRemoval.isEmpty()) {
     Vector<AtomicString> toBeRemoved;
     for (const auto& entry : m_pendingResourcesForRemoval) {
       SVGPendingElements* elements = entry.value.get();
       ASSERT(elements);
       ASSERT(!elements->isEmpty());
 
       elements->remove(element);
       if (elements->isEmpty())
         toBeRemoved.append(entry.key);
     }
 
-    // We use the removePendingResourceForRemoval function here because it deals with set lifetime correctly.
+    // We use the removePendingResourceForRemoval function here because it deals
+    // with set lifetime correctly.
     for (const AtomicString& id : toBeRemoved)
       removePendingResourceForRemoval(id);
   }
 }
 
 SVGDocumentExtensions::SVGPendingElements*
 SVGDocumentExtensions::removePendingResource(const AtomicString& id) {
   ASSERT(m_pendingResources.contains(id));
   return m_pendingResources.take(id);
 }
@@ skipping 96 matching lines @@
 DEFINE_TRACE(SVGDocumentExtensions) {
   visitor->trace(m_document);
   visitor->trace(m_timeContainers);
   visitor->trace(m_webAnimationsPendingSVGElements);
   visitor->trace(m_relativeLengthSVGRoots);
   visitor->trace(m_pendingResources);
   visitor->trace(m_pendingResourcesForRemoval);
 }
 
 }  // namespace blink