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

Unified Diff: third_party/WebKit/Source/core/layout/svg/LayoutSVGShape.h

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/LayoutSVGShape.h
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGShape.h b/third_party/WebKit/Source/core/layout/svg/LayoutSVGShape.h
index 928200fa063a34a93af9189cfbc091f24453a8fa..7b0708a1b5fb6ed9b198cd0680046f0cc04b6a0f 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGShape.h
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGShape.h
@@ -75,9 +75,10 @@ class LayoutSVGShape : public LayoutSVGModelObject {
bool hasPath() const { return m_path.get(); }
float dashScaleFactor() const;
- // This method is sometimes (rarely) called with a null path and crashes. The code managing
- // the path enforces the necessary invariants to ensure a valid path but somehow that fails.
- // The assert and check for hasPath() are intended to detect and prevent crashes.
+ // This method is sometimes (rarely) called with a null path and crashes. The
+ // code managing the path enforces the necessary invariants to ensure a valid
+ // path but somehow that fails. The assert and check for hasPath() are
+ // intended to detect and prevent crashes.
virtual bool isShapeEmpty() const {
DCHECK(m_path);
return !hasPath() || path().isEmpty();
@@ -153,9 +154,10 @@ class LayoutSVGShape : public LayoutSVGModelObject {
private:
AffineTransform m_localTransform;
- // TODO(fmalita): the Path is now cached in SVGPath; while this additional cache is just a
- // shallow copy, it certainly has a complexity/state management cost (plus allocation & storage
- // overhead) - so we should look into removing it.
+ // TODO(fmalita): the Path is now cached in SVGPath; while this additional
+ // cache is just a shallow copy, it certainly has a complexity/state
+ // management cost (plus allocation & storage overhead) - so we should look
+ // into removing it.
std::unique_ptr<Path> m_path;
mutable std::unique_ptr<LayoutSVGShapeRareData> m_rareData;

Powered by Google App Engine
This is Rietveld 408576698