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

Unified Diff: third_party/WebKit/Source/core/svg/SVGLengthContext.cpp

Issue 2390773004: reflow comments in core/svg/ (Closed)
Patch Set: comments (heh!) 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/svg/SVGLengthContext.cpp
diff --git a/third_party/WebKit/Source/core/svg/SVGLengthContext.cpp b/third_party/WebKit/Source/core/svg/SVGLengthContext.cpp
index 8d0ca0fc78b444917f688e6d96c6647880ede658..2670221a4c2a590ac206e0a08d65c43cd50bd97d 100644
--- a/third_party/WebKit/Source/core/svg/SVGLengthContext.cpp
+++ b/third_party/WebKit/Source/core/svg/SVGLengthContext.cpp
@@ -191,7 +191,9 @@ FloatPoint SVGLengthContext::resolvePoint(const SVGElement* context,
return FloatPoint(x.value(lengthContext), y.value(lengthContext));
}
- // FIXME: valueAsPercentage() won't be correct for eg. cm units. They need to be resolved in user space and then be considered in objectBoundingBox space.
+ // FIXME: valueAsPercentage() won't be correct for eg. cm units. They need to
+ // be resolved in user space and then be considered in objectBoundingBox
+ // space.
return FloatPoint(x.valueAsPercentage(), y.valueAsPercentage());
}
@@ -204,7 +206,9 @@ float SVGLengthContext::resolveLength(const SVGElement* context,
return x.value(lengthContext);
}
- // FIXME: valueAsPercentage() won't be correct for eg. cm units. They need to be resolved in user space and then be considered in objectBoundingBox space.
+ // FIXME: valueAsPercentage() won't be correct for eg. cm units. They need to
+ // be resolved in user space and then be considered in objectBoundingBox
+ // space.
return x.valueAsPercentage();
}
@@ -393,8 +397,9 @@ float SVGLengthContext::convertValueFromUserUnitsToEXS(float value) const {
if (!style)
return 0;
- // Use of ceil allows a pixel match to the W3Cs expected output of coords-units-03-b.svg
- // if this causes problems in real world cases maybe it would be best to remove this
+ // Use of ceil allows a pixel match to the W3Cs expected output of
+ // coords-units-03-b.svg, if this causes problems in real world cases maybe it
+ // would be best to remove this.
float xHeight =
ceilf(style->getFontMetrics().xHeight() / style->effectiveZoom());
if (!xHeight)
@@ -408,8 +413,9 @@ float SVGLengthContext::convertValueFromEXSToUserUnits(float value) const {
if (!style)
return 0;
- // Use of ceil allows a pixel match to the W3Cs expected output of coords-units-03-b.svg
- // if this causes problems in real world cases maybe it would be best to remove this
+ // Use of ceil allows a pixel match to the W3Cs expected output of
+ // coords-units-03-b.svg, if this causes problems in real world cases maybe it
+ // would be best to remove this.
return value *
ceilf(style->getFontMetrics().xHeight() / style->effectiveZoom());
}
« no previous file with comments | « third_party/WebKit/Source/core/svg/SVGLength.cpp ('k') | third_party/WebKit/Source/core/svg/SVGLengthList.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698