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

Unified Diff: third_party/WebKit/Source/core/layout/svg/LayoutSVGForeignObject.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/LayoutSVGForeignObject.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGForeignObject.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGForeignObject.cpp
index 7b7b3079d4da864dd626a25711ea0bc20a3eeeb9..f3417a6c3cd280c1df0482f9054b06d3cbdfd990 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGForeignObject.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGForeignObject.cpp
@@ -36,7 +36,8 @@ LayoutSVGForeignObject::~LayoutSVGForeignObject() {}
bool LayoutSVGForeignObject::isChildAllowed(LayoutObject* child,
const ComputedStyle& style) const {
- // Disallow arbitary SVG content. Only allow proper <svg xmlns="svgNS"> subdocuments.
+ // Disallow arbitary SVG content. Only allow proper <svg xmlns="svgNS">
+ // subdocuments.
return !child->isSVG() || child->isSVGRoot();
}
@@ -54,7 +55,8 @@ const AffineTransform& LayoutSVGForeignObject::localToSVGParentTransform()
void LayoutSVGForeignObject::updateLogicalWidth() {
// FIXME: Investigate in size rounding issues
- // FIXME: Remove unnecessary rounding when layout is off ints: webkit.org/b/63656
+ // FIXME: Remove unnecessary rounding when layout is off ints:
+ // webkit.org/b/63656
setWidth(LayoutUnit(static_cast<int>(roundf(m_viewport.width()))));
}
@@ -63,7 +65,8 @@ void LayoutSVGForeignObject::computeLogicalHeight(
LayoutUnit logicalTop,
LogicalExtentComputedValues& computedValues) const {
// FIXME: Investigate in size rounding issues
- // FIXME: Remove unnecessary rounding when layout is off ints: webkit.org/b/63656
+ // FIXME: Remove unnecessary rounding when layout is off ints:
+ // webkit.org/b/63656
// FIXME: Is this correct for vertical writing mode?
computedValues.m_extent =
LayoutUnit(static_cast<int>(roundf(m_viewport.height())));
@@ -100,11 +103,14 @@ void LayoutSVGForeignObject::layout() {
if (!updateCachedBoundariesInParents)
updateCachedBoundariesInParents = oldViewport != m_viewport;
- // Set box origin to the foreignObject x/y translation, so positioned objects in XHTML content get correct
- // positions. A regular LayoutBoxModelObject would pull this information from ComputedStyle - in SVG those
- // properties are ignored for non <svg> elements, so we mimic what happens when specifying them through CSS.
+ // Set box origin to the foreignObject x/y translation, so positioned objects
+ // in XHTML content get correct positions. A regular LayoutBoxModelObject
+ // would pull this information from ComputedStyle - in SVG those properties
+ // are ignored for non <svg> elements, so we mimic what happens when
+ // specifying them through CSS.
- // FIXME: Investigate in location rounding issues - only affects LayoutSVGForeignObject & LayoutSVGText
+ // FIXME: Investigate in location rounding issues - only affects
+ // LayoutSVGForeignObject & LayoutSVGText
setLocation(roundedIntPoint(viewportLocation));
bool layoutChanged = everHadLayout() && selfNeedsLayout();

Powered by Google App Engine
This is Rietveld 408576698