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

Unified Diff: src/gpu/GrPath.h

Issue 1908433002: Batch multiple single NVPR draw paths to instanced draws Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 4 years, 8 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 | src/gpu/GrPathRendering.h » ('j') | src/gpu/gl/GrGLPathRendering.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/gpu/GrPath.h
diff --git a/src/gpu/GrPath.h b/src/gpu/GrPath.h
index 09d317ef2f2d3d0883c01933c67151091b68da20..42c37f4645a96e6b5581f4f00aaeb3504fcad465 100644
--- a/src/gpu/GrPath.h
+++ b/src/gpu/GrPath.h
@@ -36,6 +36,17 @@ public:
const SkRect& getBounds() const { return fBounds; }
GrPathRendering::FillType getFillType() const { return fFillType; }
+
+ /** Returns true if a path can be drawn in the same draw paths operation as the other
bsalomon 2016/04/22 13:20:38 We we usually write this style of comments as /**
Kimmo Kinnunen 2016/04/25 08:36:00 Done.
+ * path. Should return true only when the condition holds transitively with all other paths in
+ * the same group.
+ * E.g.
+ * canCombineDrawPathBatchWith(a) AND canCombineDrawPathBatchWith(b)
+ * canCombineDrawPathBatchWith(a) AND canCombineDrawPathBatchWith(c)
+ * implies
+ * canCombineDrawPathBatchWith(b) AND canCombineDrawPathBatchWith(c)
+ */
+ virtual bool canCombineDrawPathBatchWith(const GrPath& other) const = 0;
#ifdef SK_DEBUG
bool isEqualTo(const SkPath& path, const GrStrokeInfo& stroke) const;
#endif
« no previous file with comments | « no previous file | src/gpu/GrPathRendering.h » ('j') | src/gpu/gl/GrGLPathRendering.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698