diff options
author | robertphillips <robertphillips@google.com> | 2016-02-03 12:25:40 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-02-03 12:25:40 -0800 |
commit | 9cdb9920fcad286ecf7875ea19902022b644fbdc (patch) | |
tree | d3bd4068495f5b18530889c60c73f6ce27489323 /src/effects/SkTableColorFilter.cpp | |
parent | d32d19a2299d03fa9e9da9aa8b5383c7f64b9495 (diff) |
Clean up GrGLSLFragmentProcessor-derived classes
In some other patches it was observed that many of the GrGLSLFragmentProcessor-derived classes needlessly pass a parameter to their constructors.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1666773002
Review URL: https://codereview.chromium.org/1666773002
Diffstat (limited to 'src/effects/SkTableColorFilter.cpp')
-rw-r--r-- | src/effects/SkTableColorFilter.cpp | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/src/effects/SkTableColorFilter.cpp b/src/effects/SkTableColorFilter.cpp index d9c07e254c..c06e4363bf 100644 --- a/src/effects/SkTableColorFilter.cpp +++ b/src/effects/SkTableColorFilter.cpp @@ -377,9 +377,7 @@ private: class GLColorTableEffect : public GrGLSLFragmentProcessor { public: - GLColorTableEffect(const GrProcessor&); - - virtual void emitCode(EmitArgs&) override; + void emitCode(EmitArgs&) override; static void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder* b) {} @@ -391,9 +389,6 @@ private: typedef GrGLSLFragmentProcessor INHERITED; }; -GLColorTableEffect::GLColorTableEffect(const GrProcessor&) { -} - void GLColorTableEffect::onSetData(const GrGLSLProgramDataManager& pdm, const GrProcessor& proc) { // The textures are organized in a strip where the rows are ordered a, r, g, b. float rgbaYValues[4]; @@ -507,7 +502,7 @@ void ColorTableEffect::onGetGLSLProcessorKey(const GrGLSLCaps& caps, } GrGLSLFragmentProcessor* ColorTableEffect::onCreateGLSLInstance() const { - return new GLColorTableEffect(*this); + return new GLColorTableEffect; } bool ColorTableEffect::onIsEqual(const GrFragmentProcessor& other) const { |