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

Unified Diff: src/gpu/GrOvalRenderer.cpp

Issue 1467553002: New API for computing optimization invariants. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 1 month 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: src/gpu/GrOvalRenderer.cpp
diff --git a/src/gpu/GrOvalRenderer.cpp b/src/gpu/GrOvalRenderer.cpp
index c66e12575c83309916403e0111eb756e835112eb..b4690aec1109787d3fd5f8f332e25e581b5738de 100644
--- a/src/gpu/GrOvalRenderer.cpp
+++ b/src/gpu/GrOvalRenderer.cpp
@@ -667,13 +667,12 @@ public:
return string;
}
- void getInvariantOutputColor(GrInitInvariantOutput* out) const override {
+ void computeBatchToXPOverrides(GrInitInvariantOutput* color, GrInitInvariantOutput* coverage,
+ bool* usePLSDstRead) const override {
// When this is called on a batch, there is only one geometry bundle
- out->setKnownFourComponents(fGeoData[0].fColor);
- }
-
- void getInvariantOutputCoverage(GrInitInvariantOutput* out) const override {
- out->setUnknownSingleComponent();
+ color->setKnownFourComponents(fGeoData[0].fColor);
+ coverage->setUnknownSingleComponent();
+ *usePLSDstRead = false;
}
private:
@@ -887,12 +886,12 @@ public:
const char* name() const override { return "EllipseBatch"; }
- void getInvariantOutputColor(GrInitInvariantOutput* out) const override {
+ void computeBatchToXPOverrides(GrInitInvariantOutput* color, GrInitInvariantOutput* coverage,
+ bool* usePLSDstRead) const {
// When this is called on a batch, there is only one geometry bundle
- out->setKnownFourComponents(fGeoData[0].fColor);
- }
- void getInvariantOutputCoverage(GrInitInvariantOutput* out) const override {
- out->setUnknownSingleComponent();
+ color->setKnownFourComponents(fGeoData[0].fColor);
+ coverage->setUnknownSingleComponent();
+ *usePLSDstRead = false;
}
private:
@@ -1158,12 +1157,12 @@ public:
const char* name() const override { return "DIEllipseBatch"; }
- void getInvariantOutputColor(GrInitInvariantOutput* out) const override {
+ void computeBatchToXPOverrides(GrInitInvariantOutput* color, GrInitInvariantOutput* coverage,
+ bool* usePLSDstRead) const {
// When this is called on a batch, there is only one geometry bundle
- out->setKnownFourComponents(fGeoData[0].fColor);
- }
- void getInvariantOutputCoverage(GrInitInvariantOutput* out) const override {
- out->setUnknownSingleComponent();
+ color->setKnownFourComponents(fGeoData[0].fColor);
+ coverage->setUnknownSingleComponent();
+ *usePLSDstRead = false;
}
private:
@@ -1513,12 +1512,12 @@ public:
const char* name() const override { return "RRectCircleBatch"; }
- void getInvariantOutputColor(GrInitInvariantOutput* out) const override {
+ void computeBatchToXPOverrides(GrInitInvariantOutput* color, GrInitInvariantOutput* coverage,
+ bool* usePLSDstRead) const {
// When this is called on a batch, there is only one geometry bundle
- out->setKnownFourComponents(fGeoData[0].fColor);
- }
- void getInvariantOutputCoverage(GrInitInvariantOutput* out) const override {
- out->setUnknownSingleComponent();
+ color->setKnownFourComponents(fGeoData[0].fColor);
+ coverage->setUnknownSingleComponent();
+ *usePLSDstRead = false;
}
private:
@@ -1692,12 +1691,12 @@ public:
const char* name() const override { return "RRectEllipseRendererBatch"; }
- void getInvariantOutputColor(GrInitInvariantOutput* out) const override {
+ void computeBatchToXPOverrides(GrInitInvariantOutput* color, GrInitInvariantOutput* coverage,
+ bool* usePLSDstRead) const {
// When this is called on a batch, there is only one geometry bundle
- out->setKnownFourComponents(fGeoData[0].fColor);
- }
- void getInvariantOutputCoverage(GrInitInvariantOutput* out) const override {
- out->setUnknownSingleComponent();
+ color->setKnownFourComponents(fGeoData[0].fColor);
+ coverage->setUnknownSingleComponent();
+ *usePLSDstRead = false;
}
private:

Powered by Google App Engine
This is Rietveld 408576698