diff options
author | 2014-12-08 13:26:43 -0800 | |
---|---|---|
committer | 2014-12-08 13:26:43 -0800 | |
commit | 8d95ffa497091d0c9c7cda099684c7bca6714a17 (patch) | |
tree | b47fb9fba90f1c9c945db4c9d0fcd73802c8773e /src/gpu/GrProcOptInfo.cpp | |
parent | 4ccf0b94054a4c16641c027edb4b0cfd0daddb65 (diff) |
Revert of Make all blending up to GrOptDrawState be handled by the xp/xp factory. (patchset #7 id:140001 of https://codereview.chromium.org/759713002/)
Reason for revert:
break many gm's
Original issue's description:
> Make all blending up to GrOptDrawState be handled by the xp/xp factory.
>
> In this cl the blending information is extracted for the xp and stored in the ODS
> which is then used as it currently is. In the follow up cl, an XP backend will be added
> and at that point all blending work will take place inside XP's.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/7c66342a399b529634bed0fabfaa562db2c0dbd4
TBR=bsalomon@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/766653008
Diffstat (limited to 'src/gpu/GrProcOptInfo.cpp')
-rw-r--r-- | src/gpu/GrProcOptInfo.cpp | 20 |
1 files changed, 5 insertions, 15 deletions
diff --git a/src/gpu/GrProcOptInfo.cpp b/src/gpu/GrProcOptInfo.cpp index a84ac2e17f..18a32020d4 100644 --- a/src/gpu/GrProcOptInfo.cpp +++ b/src/gpu/GrProcOptInfo.cpp @@ -7,9 +7,8 @@ #include "GrProcOptInfo.h" -#include "GrFragmentProcessor.h" -#include "GrFragmentStage.h" #include "GrGeometryProcessor.h" +#include "GrFragmentStage.h" void GrProcOptInfo::calcWithInitialValues(const GrFragmentStage* stages, int stageCount, @@ -23,7 +22,6 @@ void GrProcOptInfo::calcWithInitialValues(const GrFragmentStage* stages, fInputColor = startColor; fRemoveVertexAttrib = false; fReadsDst = false; - fReadsFragPosition = false; if (areCoverageStages && gp) { gp->computeInvariantOutput(&fInOut); @@ -33,22 +31,17 @@ void GrProcOptInfo::calcWithInitialValues(const GrFragmentStage* stages, const GrFragmentProcessor* processor = stages[i].getProcessor(); fInOut.resetWillUseInputColor(); processor->computeInvariantOutput(&fInOut); -#ifdef SK_DEBUG + #ifdef SK_DEBUG fInOut.validate(); -#endif + #endif if (!fInOut.willUseInputColor()) { fFirstEffectStageIndex = i; fInputColorIsUsed = false; - // Reset these since we don't care if previous stages read these values - fReadsDst = false; - fReadsFragPosition = false; + fReadsDst = false; // Reset this since we don't care if previous stages read dst } if (processor->willReadDstColor()) { fReadsDst = true; } - if (processor->willReadFragmentPosition()) { - fReadsFragPosition = true; - } if (kRGBA_GrColorComponentFlags == fInOut.validFlags()) { fFirstEffectStageIndex = i + 1; fInputColor = fInOut.color(); @@ -57,10 +50,7 @@ void GrProcOptInfo::calcWithInitialValues(const GrFragmentStage* stages, // Since we are clearing all previous color stages we are in a state where we have found // zero stages that don't multiply the inputColor. fInOut.resetNonMulStageFound(); - // Reset these since we don't care if previous stages read these values - fReadsDst = false; - fReadsFragPosition = false; + fReadsDst = false; // Reset this since we don't care if previous stages read dst } } } - |