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

Unified Diff: third_party/WebKit/Source/core/paint/SVGClipPainter.cpp

Issue 2392443009: reflow comments in core/paint (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/paint/SVGClipPainter.cpp
diff --git a/third_party/WebKit/Source/core/paint/SVGClipPainter.cpp b/third_party/WebKit/Source/core/paint/SVGClipPainter.cpp
index bbfc613548ed0317cd7da5e714dd49480ea4dce5..a7991e4061040b8d9a756475c56ce402ad0470e8 100644
--- a/third_party/WebKit/Source/core/paint/SVGClipPainter.cpp
+++ b/third_party/WebKit/Source/core/paint/SVGClipPainter.cpp
@@ -55,9 +55,10 @@ bool SVGClipPainter::prepareEffect(const LayoutObject& target,
AffineTransform animatedLocalTransform =
toSVGClipPathElement(m_clip.element())->calculateAnimatedLocalTransform();
- // When drawing a clip for non-SVG elements, the CTM does not include the zoom factor.
- // In this case, we need to apply the zoom scale explicitly - but only for clips with
- // userSpaceOnUse units (the zoom is accounted for objectBoundingBox-resolved lengths).
+ // When drawing a clip for non-SVG elements, the CTM does not include the zoom
+ // factor. In this case, we need to apply the zoom scale explicitly - but
+ // only for clips with userSpaceOnUse units (the zoom is accounted for
+ // objectBoundingBox-resolved lengths).
if (!target.isSVG() &&
m_clip.clipPathUnits() == SVGUnitTypes::kSvgUnitTypeUserspaceonuse) {
DCHECK(m_clip.style());
@@ -105,7 +106,8 @@ void SVGClipPainter::finishEffect(const LayoutObject& target,
ClipperState& clipperState) {
switch (clipperState) {
case ClipperState::AppliedPath:
- // Path-only clipping, no layers to restore but we need to emit an end to the clip path display item.
+ // Path-only clipping, no layers to restore but we need to emit an end to
+ // the clip path display item.
context.getPaintController().endItem<EndClipPathDisplayItem>(target);
break;
case ClipperState::AppliedMask:

Powered by Google App Engine
This is Rietveld 408576698