aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/gpu/GrPath.h
diff options
context:
space:
mode:
authorGravatar stephana <stephana@google.com>2016-04-25 07:01:22 -0700
committerGravatar Commit bot <commit-bot@chromium.org>2016-04-25 07:01:23 -0700
commit1dc172165b8186950fda4a269bd1b12c11947b14 (patch)
treef1482ee0e2d88d139c990b212d479885df5d7c8a /src/gpu/GrPath.h
parentedf7fcd8b9e339800edb61ae422f8b6c1b77c3e8 (diff)
Revert of Batch multiple single NVPR draw paths to instanced draws (patchset #5 id:80001 of https://codereview.chromium.org/1908433002/ )
Reason for revert: This causes what looks like significantly wrong results. Here is an example: https://gold.skia.org/diff?test=stroke-fill&left=cb47dc2cabbad7d146aedaac199ebace&top=2c6e65d597c574e3bb170a19ca9634c6 Original issue's description: > Batch multiple single NVPR draw paths to instanced draws > > Batch multiple single NVPR draw paths to instanced draws. > A draw path batch can be combined to other batch if the > batches do not overlap and have same draw characteristics. > > Join the batches in linked list and flatten the list to a > path list during draw time. > > Replace GrPathRendering::drawPath with GrPathRendering::drawPaths. > > Perf changes ARM, Shield TV device, Tegra X1 GPU: > desk_chalkboard.skp_1 21.5ms -> 17.8ms 0.83x > desk_mapsvg.skp_1 7.49ms -> 6.18ms 0.82x > Others results are more volatile. > GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1908433002 > > Committed: https://skia.googlesource.com/skia/+/3c33c389e9f8d14d86756ea1ddeba2097f31ad22 TBR=joshualitt@chromium.org,bsalomon@google.com,kkinnunen@nvidia.com # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1919843002
Diffstat (limited to 'src/gpu/GrPath.h')
-rw-r--r--src/gpu/GrPath.h12
1 files changed, 0 insertions, 12 deletions
diff --git a/src/gpu/GrPath.h b/src/gpu/GrPath.h
index 975dad0afd..4cb12f4325 100644
--- a/src/gpu/GrPath.h
+++ b/src/gpu/GrPath.h
@@ -36,18 +36,6 @@ 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
- * 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