Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(265)

Unified Diff: third_party/WebKit/Source/core/layout/svg/SVGResourcesCache.cpp

Issue 2400783002: Reformat comments in core/layout/svg (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/layout/svg/SVGResourcesCache.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/SVGResourcesCache.cpp b/third_party/WebKit/Source/core/layout/svg/SVGResourcesCache.cpp
index 4a6ac4370f54d54405b7d95d5b0384b2f52cac66..5422c57f26f73dbef20bcc70007e23e968304288 100644
--- a/third_party/WebKit/Source/core/layout/svg/SVGResourcesCache.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/SVGResourcesCache.cpp
@@ -65,7 +65,8 @@ void SVGResourcesCache::removeResourcesFromLayoutObject(LayoutObject* object) {
if (!resources)
return;
- // Walk resources and unregister the layout object as a client of each resource.
+ // Walk resources and unregister the layout object as a client of each
+ // resource.
HashSet<LayoutSVGResourceContainer*> resourceSet;
resources->buildSetOfResources(resourceSet);
@@ -110,16 +111,17 @@ void SVGResourcesCache::clientStyleChanged(LayoutObject* layoutObject,
if (!diff.hasDifference() || !layoutObject->parent())
return;
- // In this case the proper SVGFE*Element will decide whether the modified CSS properties require
+ // In this case the proper SVGFE*Element will decide whether the modified CSS
+ // properties require
// a relayout or paintInvalidation.
if (layoutObject->isSVGResourceFilterPrimitive() && !diff.needsLayout())
return;
- // Dynamic changes of CSS properties like 'clip-path' may require us to recompute the associated
- // resources for a LayoutObject.
- // TODO(fs): Avoid passing in a useless StyleDifference, but instead compare oldStyle/newStyle
- // to see which resources changed to be able to selectively rebuild individual resources,
- // instead of all of them.
+ // Dynamic changes of CSS properties like 'clip-path' may require us to
+ // recompute the associated resources for a LayoutObject.
+ // TODO(fs): Avoid passing in a useless StyleDifference, but instead compare
+ // oldStyle/newStyle to see which resources changed to be able to selectively
+ // rebuild individual resources, instead of all of them.
if (layoutObjectCanHaveResources(layoutObject)) {
SVGResourcesCache& cache = resourcesCache(layoutObject->document());
cache.removeResourcesFromLayoutObject(layoutObject);

Powered by Google App Engine
This is Rietveld 408576698