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

Unified Diff: src/gpu/GrProcOptInfo.cpp

Issue 791743003: Remove GP from drawstate, revision of invariant output for GP (Closed) Base URL: https://skia.googlesource.com/skia.git@color-to-gp
Patch Set: feedback inc Created 6 years 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/GrProcOptInfo.cpp
diff --git a/src/gpu/GrProcOptInfo.cpp b/src/gpu/GrProcOptInfo.cpp
index a84ac2e17fb45dcfb2a9f556b2a628dd68bcf53f..d6f3492454d2aed183675633f4ad2d4375fb9b93 100644
--- a/src/gpu/GrProcOptInfo.cpp
+++ b/src/gpu/GrProcOptInfo.cpp
@@ -11,31 +11,49 @@
#include "GrFragmentStage.h"
#include "GrGeometryProcessor.h"
+void GrProcOptInfo::calcColorWithPrimProc(const GrPrimitiveProcessor* pp,
+ const GrFragmentStage* stages,
+ int stageCount) {
+ GrInitInvariantOutput out;
+ pp->getOutputColor(&out);
+ fInOut.reset(out);
+ this->internalCalc(stages, stageCount);
+}
+
+void GrProcOptInfo::calcCoverageWithPrimProc(const GrPrimitiveProcessor* pp,
+ const GrFragmentStage* stages,
+ int stageCount) {
+ GrInitInvariantOutput out;
+ pp->getOutputCoverage(&out);
+ fInOut.reset(out);
+ this->internalCalc(stages, stageCount);
+}
+
void GrProcOptInfo::calcWithInitialValues(const GrFragmentStage* stages,
int stageCount,
GrColor startColor,
GrColorComponentFlags flags,
- bool areCoverageStages,
- const GrGeometryProcessor* gp) {
- fInOut.reset(startColor, flags, areCoverageStages);
+ bool areCoverageStages) {
+ GrInitInvariantOutput out;
+ out.fIsSingleComponent = areCoverageStages;
+ out.fColor = startColor;
+ out.fValidFlags = flags;
+ this->internalCalc(stages, stageCount);
+}
+
+void GrProcOptInfo::internalCalc(const GrFragmentStage* stages, int stageCount) {
fFirstEffectStageIndex = 0;
fInputColorIsUsed = true;
- fInputColor = startColor;
+ fInputColor = fInOut.color();
fRemoveVertexAttrib = false;
fReadsDst = false;
fReadsFragPosition = false;
- if (areCoverageStages && gp) {
- gp->computeInvariantOutput(&fInOut);
- }
-
for (int i = 0; i < stageCount; ++i) {
const GrFragmentProcessor* processor = stages[i].getProcessor();
fInOut.resetWillUseInputColor();
processor->computeInvariantOutput(&fInOut);
-#ifdef SK_DEBUG
- fInOut.validate();
-#endif
+ SkDEBUGCODE(fInOut.validate());
if (!fInOut.willUseInputColor()) {
fFirstEffectStageIndex = i;
fInputColorIsUsed = false;
@@ -63,4 +81,3 @@ void GrProcOptInfo::calcWithInitialValues(const GrFragmentStage* stages,
}
}
}
-

Powered by Google App Engine
This is Rietveld 408576698