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

Unified Diff: third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.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
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.cpp
index 25854ad5070267c7f004ad98041babba08100a24..ab22ca704f8ed34c827c1d5caea4ea34c6851c66 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.cpp
@@ -34,23 +34,32 @@ LayoutSVGBlock::LayoutSVGBlock(SVGElement* element)
: LayoutBlockFlow(element) {}
bool LayoutSVGBlock::allowsOverflowClip() const {
- // LayoutSVGBlock, used by Layout(SVGText|ForeignObject), is not allowed to have overflow clip.
- // LayoutBlock assumes a layer to be present when the overflow clip functionality is requested. Both
- // Layout(SVGText|ForeignObject) return 'NoPaintLayer' on 'layerTypeRequired'. Fine for LayoutSVGText.
+ // LayoutSVGBlock, used by Layout(SVGText|ForeignObject), is not allowed to
+ // have overflow clip.
+ // LayoutBlock assumes a layer to be present when the overflow clip
+ // functionality is requested. Both Layout(SVGText|ForeignObject) return
+ // 'NoPaintLayer' on 'layerTypeRequired'.
+ // Fine for LayoutSVGText.
//
- // If we want to support overflow rules for <foreignObject> we can choose between two solutions:
+ // If we want to support overflow rules for <foreignObject> we can choose
+ // between two solutions:
// a) make LayoutSVGForeignObject require layers and SVG layer aware
- // b) refactor overflow logic out of Layer (as suggested by dhyatt), which is a large task
+ // b) refactor overflow logic out of Layer (as suggested by dhyatt), which is
+ // a large task
//
- // Until this is resolved, disable overflow support. Opera/FF don't support it as well at the moment (Feb 2010).
+ // Until this is resolved, disable overflow support. Opera/FF don't support it
+ // as well at the moment (Feb 2010).
//
- // Note: This does NOT affect overflow handling on outer/inner <svg> elements - this is handled
- // manually by LayoutSVGRoot - which owns the documents enclosing root layer and thus works fine.
+ // Note: This does NOT affect overflow handling on outer/inner <svg> elements
+ // - this is handled
+ // manually by LayoutSVGRoot - which owns the documents enclosing root layer
+ // and thus works fine.
return false;
}
void LayoutSVGBlock::absoluteRects(Vector<IntRect>&, const LayoutPoint&) const {
- // This code path should never be taken for SVG, as we're assuming useTransforms=true everywhere, absoluteQuads should be used.
+ // This code path should never be taken for SVG, as we're assuming
+ // useTransforms=true everywhere, absoluteQuads should be used.
ASSERT_NOT_REACHED();
}
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698