diff options
author | robertphillips <robertphillips@google.com> | 2014-10-02 12:12:48 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-10-02 12:12:48 -0700 |
commit | b4b7a4c9ea2e399db550f93e3754c351e5b2079a (patch) | |
tree | f67932963699b8dac498d3e67f70cc2c505c70a3 /src/gpu/GrOvalRenderer.cpp | |
parent | 85d3427d4932c6cfaf02f8a7454f0c6f213b0c2e (diff) |
Revert of Add isSingleComponent bool to getConstantColorComponent (patchset #6 id:100001 of https://codereview.chromium.org/608253002/)
Reason for revert:
Changing some GMs
Original issue's description:
> Add isSingleComponent bool to getConstantColorComponent
>
> Initial step to allowing effects to use/output 1 or 4 color/coverage components. This cl doesn't change any current logic and all effects still assume they are working with 4 components.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/3b8af078281a5a20f951b9fd84f38d92b8f6217b
TBR=joshualitt@chromium.org,bsalomon@google.com,reed@google.com,egdaniel@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/617853003
Diffstat (limited to 'src/gpu/GrOvalRenderer.cpp')
-rw-r--r-- | src/gpu/GrOvalRenderer.cpp | 30 |
1 files changed, 15 insertions, 15 deletions
diff --git a/src/gpu/GrOvalRenderer.cpp b/src/gpu/GrOvalRenderer.cpp index 7a84f80805..4ae0ebc7ef 100644 --- a/src/gpu/GrOvalRenderer.cpp +++ b/src/gpu/GrOvalRenderer.cpp @@ -75,6 +75,11 @@ public: } } + virtual void getConstantColorComponents(GrColor* color, + uint32_t* validFlags) const SK_OVERRIDE { + *validFlags = 0; + } + const GrShaderVar& inCircleEdge() const { return fInCircleEdge; } virtual const GrBackendGeometryProcessorFactory& getFactory() const SK_OVERRIDE { @@ -145,11 +150,6 @@ private: return cee.fStroke == fStroke; } - virtual void onComputeInvariantOutput(InvariantOutput* inout) const SK_OVERRIDE { - inout->fValidFlags = 0; - inout->fIsSingleComponent = false; - } - const GrShaderVar& fInCircleEdge; bool fStroke; @@ -192,6 +192,11 @@ public: } } + virtual void getConstantColorComponents(GrColor* color, + uint32_t* validFlags) const SK_OVERRIDE { + *validFlags = 0; + } + virtual const GrBackendGeometryProcessorFactory& getFactory() const SK_OVERRIDE { return GrTBackendGeometryProcessorFactory<EllipseEdgeEffect>::getInstance(); } @@ -286,11 +291,6 @@ private: return eee.fStroke == fStroke; } - virtual void onComputeInvariantOutput(InvariantOutput* inout) const SK_OVERRIDE { - inout->fValidFlags = 0; - inout->fIsSingleComponent = false; - } - const GrShaderVar& fInEllipseOffset; const GrShaderVar& fInEllipseRadii; bool fStroke; @@ -341,6 +341,11 @@ public: } } + virtual void getConstantColorComponents(GrColor* color, + uint32_t* validFlags) const SK_OVERRIDE { + *validFlags = 0; + } + virtual const GrBackendGeometryProcessorFactory& getFactory() const SK_OVERRIDE { return GrTBackendGeometryProcessorFactory<DIEllipseEdgeEffect>::getInstance(); } @@ -455,11 +460,6 @@ private: return eee.fMode == fMode; } - virtual void onComputeInvariantOutput(InvariantOutput* inout) const SK_OVERRIDE { - inout->fValidFlags = 0; - inout->fIsSingleComponent = false; - } - const GrShaderVar& fInEllipseOffsets0; const GrShaderVar& fInEllipseOffsets1; Mode fMode; |