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

Unified Diff: third_party/WebKit/Source/core/layout/svg/LayoutSVGResourcePattern.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/LayoutSVGResourcePattern.h
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourcePattern.h b/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourcePattern.h
index 1c86b240854ff10f63707f74962ac7ec7e0d5a5f..788b7b4754c2bd6587a2df89961689a75a47348e 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourcePattern.h
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourcePattern.h
@@ -71,12 +71,13 @@ class LayoutSVGResourcePattern final : public LayoutSVGResourcePaintServer {
return m_attributesWrapper->attributes();
}
- // FIXME: we can almost do away with this per-object map, but not quite: the tile size can be
- // relative to the client bounding box, and it gets captured in the cached Pattern shader.
- // Hence, we need one Pattern shader per client. The display list OTOH is the same => we
- // should be able to cache a single display list per LayoutSVGResourcePattern + one
- // Pattern(shader) for each client -- this would avoid re-recording when multiple clients
- // share the same pattern.
+ // FIXME: we can almost do away with this per-object map, but not quite: the
+ // tile size can be relative to the client bounding box, and it gets captured
+ // in the cached Pattern shader.
+ // Hence, we need one Pattern shader per client. The display list OTOH is the
+ // same => we should be able to cache a single display list per
+ // LayoutSVGResourcePattern + one Pattern(shader) for each client -- this
+ // would avoid re-recording when multiple clients share the same pattern.
HashMap<const LayoutObject*, std::unique_ptr<PatternData>> m_patternMap;
};

Powered by Google App Engine
This is Rietveld 408576698