| Index: third_party/WebKit/Source/core/svg/SVGDocumentExtensions.cpp
|
| diff --git a/third_party/WebKit/Source/core/svg/SVGDocumentExtensions.cpp b/third_party/WebKit/Source/core/svg/SVGDocumentExtensions.cpp
|
| index e11c49272de93390a84d8c424726645a6077190d..1e32b5af0aa4d5eee53792f1f7a28bcca7fcee17 100644
|
| --- a/third_party/WebKit/Source/core/svg/SVGDocumentExtensions.cpp
|
| +++ b/third_party/WebKit/Source/core/svg/SVGDocumentExtensions.cpp
|
| @@ -99,7 +99,9 @@ void SVGDocumentExtensions::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();
|
|
|
| @@ -107,10 +109,12 @@ void SVGDocumentExtensions::serviceAnimations() {
|
| }
|
|
|
| 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) {
|
| @@ -133,7 +137,8 @@ void SVGDocumentExtensions::dispatchSVGLoadEventToOutermostSVGElements() {
|
| 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();
|
| @@ -171,8 +176,9 @@ bool SVGDocumentExtensions::hasPendingResource(const AtomicString& id) const {
|
| }
|
|
|
| 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);
|
|
|
| @@ -222,7 +228,8 @@ void SVGDocumentExtensions::removeElementFromPendingResources(
|
|
|
| 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);
|
| }
|
| @@ -240,7 +247,8 @@ void SVGDocumentExtensions::removeElementFromPendingResources(
|
| 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);
|
| }
|
|
|