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

Unified Diff: third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceGradient.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/LayoutSVGResourceGradient.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceGradient.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceGradient.cpp
index 75423014916590b5ba141e6fb4c67dd2bf04e414..bf50575f9f451007210a86b6401df2ff16ec21d8 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceGradient.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceGradient.cpp
@@ -52,10 +52,12 @@ SVGPaintServer LayoutSVGResourceGradient::preparePaintServer(
const LayoutObject& object) {
clearInvalidationMask();
- // Be sure to synchronize all SVG properties on the gradientElement _before_ processing any further.
- // Otherwhise the call to collectGradientAttributes() in createTileImage(), may cause the SVG DOM property
- // synchronization to kick in, which causes removeAllClientsFromCache() to be called, which in turn deletes our
- // GradientData object! Leaving out the line below will cause svg/dynamic-updates/SVG*GradientElement-svgdom* to crash.
+ // Be sure to synchronize all SVG properties on the gradientElement _before_
+ // processing any further. Otherwhise the call to collectGradientAttributes()
+ // in createTileImage(), may cause the SVG DOM property synchronization to
+ // kick in, which causes removeAllClientsFromCache() to be called, which in
+ // turn deletes our GradientData object! Leaving out the line below will cause
+ // svg/dynamic-updates/SVG*GradientElement-svgdom* to crash.
SVGGradientElement* gradientElement = toSVGGradientElement(element());
if (!gradientElement)
return SVGPaintServer::invalid();
@@ -68,8 +70,9 @@ SVGPaintServer LayoutSVGResourceGradient::preparePaintServer(
m_shouldCollectGradientAttributes = false;
}
- // Spec: When the geometry of the applicable element has no width or height and objectBoundingBox is specified,
- // then the given effect (e.g. a gradient or a filter) will be ignored.
+ // Spec: When the geometry of the applicable element has no width or height
+ // and objectBoundingBox is specified, then the given effect (e.g. a gradient
+ // or a filter) will be ignored.
FloatRect objectBoundingBox = object.objectBoundingBox();
if (gradientUnits() == SVGUnitTypes::kSvgUnitTypeObjectboundingbox &&
objectBoundingBox.isEmpty())
@@ -84,7 +87,8 @@ SVGPaintServer LayoutSVGResourceGradient::preparePaintServer(
if (!gradientData->gradient) {
gradientData->gradient = buildGradient();
- // We want the text bounding box applied to the gradient space transform now, so the gradient shader can use it.
+ // We want the text bounding box applied to the gradient space transform
+ // now, so the gradient shader can use it.
if (gradientUnits() == SVGUnitTypes::kSvgUnitTypeObjectboundingbox &&
!objectBoundingBox.isEmpty()) {
gradientData->userspaceTransform.translate(objectBoundingBox.x(),

Powered by Google App Engine
This is Rietveld 408576698