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

Unified Diff: third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceClipper.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/LayoutSVGResourceClipper.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceClipper.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceClipper.cpp
index f7edaff756d569eb4329ad39789a66a6ae81f8f9..94338f814f1c2371b20507c39dfb9529055ef6c8 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceClipper.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceClipper.cpp
@@ -63,7 +63,8 @@ bool LayoutSVGResourceClipper::calculateClipContentPathIfNeeded() {
if (!m_clipContentPath.isEmpty())
return true;
- // If the current clip-path gets clipped itself, we have to fallback to masking.
+ // If the current clip-path gets clipped itself, we have to fallback to
+ // masking.
if (styleRef().clipPath())
return false;
@@ -77,7 +78,8 @@ bool LayoutSVGResourceClipper::calculateClipContentPathIfNeeded() {
LayoutObject* childLayoutObject = childElement->layoutObject();
if (!childLayoutObject)
continue;
- // Only shapes or paths are supported for direct clipping. We need to fallback to masking for texts.
+ // Only shapes or paths are supported for direct clipping. We need to
+ // fallback to masking for texts.
if (childLayoutObject->isSVGText()) {
m_clipContentPath.clear();
return false;
@@ -107,8 +109,9 @@ bool LayoutSVGResourceClipper::calculateClipContentPathIfNeeded() {
continue;
}
- // Multiple shapes require PathOps. In some degenerate cases PathOps can exhibit quadratic
- // behavior, so we cap the number of ops to a reasonable count.
+ // Multiple shapes require PathOps. In some degenerate cases PathOps can
+ // exhibit quadratic behavior, so we cap the number of ops to a reasonable
+ // count.
const unsigned kMaxOps = 42;
if (++opCount > kMaxOps) {
m_clipContentPath.clear();
@@ -167,9 +170,10 @@ sk_sp<const SkPicture> LayoutSVGResourceClipper::createContentPicture() {
if (m_clipContentPicture)
return m_clipContentPicture;
- // Using strokeBoundingBox (instead of paintInvalidationRectInLocalSVGCoordinates) to avoid the intersection
- // with local clips/mask, which may yield incorrect results when mixing objectBoundingBox and
- // userSpaceOnUse units (http://crbug.com/294900).
+ // Using strokeBoundingBox (instead of
+ // paintInvalidationRectInLocalSVGCoordinates) to avoid the intersection with
+ // local clips/mask, which may yield incorrect results when mixing
+ // objectBoundingBox and userSpaceOnUse units (http://crbug.com/294900).
FloatRect bounds = strokeBoundingBox();
SkPictureBuilder pictureBuilder(bounds, nullptr, nullptr);
@@ -207,7 +211,8 @@ sk_sp<const SkPicture> LayoutSVGResourceClipper::createContentPicture() {
if (isUseElement)
layoutObject = childElement->layoutObject();
- // Switch to a paint behavior where all children of this <clipPath> will be laid out using special constraints:
+ // Switch to a paint behavior where all children of this <clipPath> will be
+ // laid out using special constraints:
// - fill-opacity/stroke-opacity/opacity set to 1
// - masker/filter not applied when laying out the children
// - fill is set to the initial fill paint server (solid, black)
@@ -223,7 +228,8 @@ sk_sp<const SkPicture> LayoutSVGResourceClipper::createContentPicture() {
}
void LayoutSVGResourceClipper::calculateLocalClipBounds() {
- // This is a rough heuristic to appraise the clip size and doesn't consider clip on clip.
+ // This is a rough heuristic to appraise the clip size and doesn't consider
+ // clip on clip.
for (SVGElement* childElement = Traversal<SVGElement>::firstChild(*element());
childElement;
childElement = Traversal<SVGElement>::nextSibling(*childElement)) {

Powered by Google App Engine
This is Rietveld 408576698