From cfc18867d982119d9dc2888bf09f1093012daadd Mon Sep 17 00:00:00 2001 From: jvanverth Date: Tue, 28 Apr 2015 08:48:20 -0700 Subject: Use GLSLCaps for creating processor keys and GLSL-specific programs Effectively all this does is future-proof any GLSL-specific code, as GLSLCaps is just a typedef of GLCaps. BUG=skia: Review URL: https://codereview.chromium.org/1109863004 --- src/gpu/GrAAConvexPathRenderer.cpp | 6 +++--- src/gpu/GrDefaultGeoProcFactory.cpp | 6 +++--- src/gpu/GrOvalRenderer.cpp | 18 +++++++++--------- src/gpu/GrPathProcessor.cpp | 4 ++-- src/gpu/GrPathProcessor.h | 4 ++-- src/gpu/GrPrimitiveProcessor.h | 5 +++-- src/gpu/GrXferProcessor.cpp | 2 +- src/gpu/effects/GrBezierEffect.cpp | 24 ++++++++++++------------ src/gpu/effects/GrBezierEffect.h | 12 ++++++------ src/gpu/effects/GrBicubicEffect.cpp | 4 ++-- src/gpu/effects/GrBicubicEffect.h | 2 +- src/gpu/effects/GrBitmapTextGeoProc.cpp | 6 +++--- src/gpu/effects/GrBitmapTextGeoProc.h | 4 ++-- src/gpu/effects/GrConfigConversionEffect.cpp | 4 ++-- src/gpu/effects/GrConfigConversionEffect.h | 2 +- src/gpu/effects/GrConstColorProcessor.cpp | 2 +- src/gpu/effects/GrConvexPolyEffect.cpp | 14 +++++++------- src/gpu/effects/GrConvexPolyEffect.h | 2 +- src/gpu/effects/GrConvolutionEffect.cpp | 6 +++--- src/gpu/effects/GrConvolutionEffect.h | 2 +- src/gpu/effects/GrCoverageSetOpXP.cpp | 6 +++--- src/gpu/effects/GrCustomXfermode.cpp | 10 +++++----- src/gpu/effects/GrCustomXfermodePriv.h | 2 +- src/gpu/effects/GrDashingEffect.cpp | 24 ++++++++++++------------ src/gpu/effects/GrDisableColorXP.cpp | 6 +++--- src/gpu/effects/GrDistanceFieldGeoProc.cpp | 18 +++++++++--------- src/gpu/effects/GrDistanceFieldGeoProc.h | 12 ++++++------ src/gpu/effects/GrDitherEffect.cpp | 4 ++-- src/gpu/effects/GrMatrixConvolutionEffect.cpp | 6 +++--- src/gpu/effects/GrMatrixConvolutionEffect.h | 2 +- src/gpu/effects/GrOvalEffect.cpp | 16 ++++++++-------- src/gpu/effects/GrPorterDuffXferProcessor.cpp | 6 +++--- src/gpu/effects/GrRRectEffect.cpp | 16 ++++++++-------- src/gpu/effects/GrSimpleTextureEffect.cpp | 2 +- src/gpu/effects/GrSimpleTextureEffect.h | 2 +- src/gpu/effects/GrTextureDomain.cpp | 6 +++--- src/gpu/effects/GrTextureDomain.h | 2 +- src/gpu/effects/GrYUVtoRGBEffect.cpp | 4 ++-- src/gpu/gl/GrGLCaps.h | 2 ++ src/gpu/gl/GrGLPathProcessor.cpp | 2 +- src/gpu/gl/GrGLPathProcessor.h | 2 +- src/gpu/gl/GrGLProcessor.h | 4 ++-- src/gpu/gl/GrGLProgramDesc.cpp | 3 +++ 43 files changed, 146 insertions(+), 140 deletions(-) (limited to 'src/gpu') diff --git a/src/gpu/GrAAConvexPathRenderer.cpp b/src/gpu/GrAAConvexPathRenderer.cpp index def69a04b9..d6eb3d2a2b 100644 --- a/src/gpu/GrAAConvexPathRenderer.cpp +++ b/src/gpu/GrAAConvexPathRenderer.cpp @@ -576,7 +576,7 @@ public: static inline void GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const BatchTracker& local = bt.cast(); uint32_t key = local.fInputColorType << 16; @@ -607,13 +607,13 @@ public: }; virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override { GLProcessor::GenKey(*this, bt, caps, b); } virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const override { + const GrGLSLCaps&) const override { return SkNEW_ARGS(GLProcessor, (*this, bt)); } diff --git a/src/gpu/GrDefaultGeoProcFactory.cpp b/src/gpu/GrDefaultGeoProcFactory.cpp index ea52462788..fac101d8fb 100644 --- a/src/gpu/GrDefaultGeoProcFactory.cpp +++ b/src/gpu/GrDefaultGeoProcFactory.cpp @@ -129,7 +129,7 @@ public: static inline void GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const DefaultGeoProc& def = gp.cast(); const BatchTracker& local = bt.cast(); @@ -168,13 +168,13 @@ public: }; virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override { GLProcessor::GenKey(*this, bt, caps, b); } virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const override { + const GrGLSLCaps&) const override { return SkNEW_ARGS(GLProcessor, (*this, bt)); } diff --git a/src/gpu/GrOvalRenderer.cpp b/src/gpu/GrOvalRenderer.cpp index fe15532b75..e9cda2fd64 100644 --- a/src/gpu/GrOvalRenderer.cpp +++ b/src/gpu/GrOvalRenderer.cpp @@ -127,7 +127,7 @@ public: static void GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const BatchTracker& local = bt.cast(); const CircleEdgeEffect& circleEffect = gp.cast(); @@ -158,13 +158,13 @@ public: }; virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override { GLProcessor::GenKey(*this, bt, caps, b); } virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const override { + const GrGLSLCaps&) const override { return SkNEW_ARGS(GLProcessor, (*this, bt)); } @@ -321,7 +321,7 @@ public: static void GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const BatchTracker& local = bt.cast(); const EllipseEdgeEffect& ellipseEffect = gp.cast(); @@ -353,13 +353,13 @@ public: }; virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override { GLProcessor::GenKey(*this, bt, caps, b); } virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const override { + const GrGLSLCaps&) const override { return SkNEW_ARGS(GLProcessor, (*this, bt)); } @@ -536,7 +536,7 @@ public: static void GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const BatchTracker& local = bt.cast(); const DIEllipseEdgeEffect& ellipseEffect = gp.cast(); @@ -568,13 +568,13 @@ public: }; virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override { GLProcessor::GenKey(*this, bt, caps, b); } virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const override { + const GrGLSLCaps&) const override { return SkNEW_ARGS(GLProcessor, (*this, bt)); } diff --git a/src/gpu/GrPathProcessor.cpp b/src/gpu/GrPathProcessor.cpp index aeb16d8797..fe01e7f2dd 100644 --- a/src/gpu/GrPathProcessor.cpp +++ b/src/gpu/GrPathProcessor.cpp @@ -63,13 +63,13 @@ bool GrPathProcessor::canMakeEqual(const GrBatchTracker& m, } void GrPathProcessor::getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLPathProcessor::GenKey(*this, bt, caps, b); } GrGLPrimitiveProcessor* GrPathProcessor::createGLInstance(const GrBatchTracker& bt, - const GrGLCaps& caps) const { + const GrGLSLCaps& caps) const { SkASSERT(caps.pathRenderingSupport()); return SkNEW_ARGS(GrGLPathProcessor, (*this, bt)); } diff --git a/src/gpu/GrPathProcessor.h b/src/gpu/GrPathProcessor.h index b6b6f6e5a8..5b233782c6 100644 --- a/src/gpu/GrPathProcessor.h +++ b/src/gpu/GrPathProcessor.h @@ -45,11 +45,11 @@ public: bool willUseGeoShader() const override { return false; } virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps& caps) const override; + const GrGLSLCaps& caps) const override; protected: GrPathProcessor(GrColor color, const SkMatrix& viewMatrix, const SkMatrix& localMatrix); diff --git a/src/gpu/GrPrimitiveProcessor.h b/src/gpu/GrPrimitiveProcessor.h index 45668e9be6..ea1237b9a2 100644 --- a/src/gpu/GrPrimitiveProcessor.h +++ b/src/gpu/GrPrimitiveProcessor.h @@ -68,6 +68,7 @@ private: class GrIndexBufferAllocPool; class GrGLCaps; +typedef GrGLCaps GrGLSLCaps; class GrGLPrimitiveProcessor; class GrVertexBufferAllocPool; @@ -163,7 +164,7 @@ public: * processor's GL backend implementation. */ virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const = 0; @@ -171,7 +172,7 @@ public: for the given GrProcessor; caller is responsible for deleting the object. */ virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps& caps) const = 0; + const GrGLSLCaps& caps) const = 0; bool isPathRendering() const { return fIsPathRendering; } diff --git a/src/gpu/GrXferProcessor.cpp b/src/gpu/GrXferProcessor.cpp index 476394f800..61cacac62e 100644 --- a/src/gpu/GrXferProcessor.cpp +++ b/src/gpu/GrXferProcessor.cpp @@ -22,7 +22,7 @@ GrXferProcessor::GrXferProcessor(const GrDeviceCoordTexture* dstCopy, bool willR } } -void GrXferProcessor::getGLProcessorKey(const GrGLCaps& caps, GrProcessorKeyBuilder* b) const { +void GrXferProcessor::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { uint32_t key = this->willReadDstColor() ? 0x1 : 0x0; if (this->getDstCopyTexture() && kTopLeft_GrSurfaceOrigin == this->getDstCopyTexture()->origin()) { diff --git a/src/gpu/effects/GrBezierEffect.cpp b/src/gpu/effects/GrBezierEffect.cpp index 90b91b0bd2..cb440d4bb0 100644 --- a/src/gpu/effects/GrBezierEffect.cpp +++ b/src/gpu/effects/GrBezierEffect.cpp @@ -28,7 +28,7 @@ public: static inline void GenKey(const GrGeometryProcessor&, const GrBatchTracker&, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder*); virtual void setData(const GrGLProgramDataManager& pdman, @@ -162,7 +162,7 @@ void GrGLConicEffect::onEmitCode(EmitArgs& args, GrGPArgs* gpArgs) { void GrGLConicEffect::GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const GrConicEffect& ce = gp.cast(); const ConicBatchTracker& local = bt.cast(); @@ -179,13 +179,13 @@ void GrGLConicEffect::GenKey(const GrGeometryProcessor& gp, GrConicEffect::~GrConicEffect() {} void GrConicEffect::getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLConicEffect::GenKey(*this, bt, caps, b); } GrGLPrimitiveProcessor* GrConicEffect::createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const { + const GrGLSLCaps&) const { return SkNEW_ARGS(GrGLConicEffect, (*this, bt)); } @@ -263,7 +263,7 @@ public: static inline void GenKey(const GrGeometryProcessor&, const GrBatchTracker&, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder*); virtual void setData(const GrGLProgramDataManager& pdman, @@ -383,7 +383,7 @@ void GrGLQuadEffect::onEmitCode(EmitArgs& args, GrGPArgs* gpArgs) { void GrGLQuadEffect::GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const GrQuadEffect& ce = gp.cast(); const QuadBatchTracker& local = bt.cast(); @@ -400,13 +400,13 @@ void GrGLQuadEffect::GenKey(const GrGeometryProcessor& gp, GrQuadEffect::~GrQuadEffect() {} void GrQuadEffect::getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLQuadEffect::GenKey(*this, bt, caps, b); } GrGLPrimitiveProcessor* GrQuadEffect::createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const { + const GrGLSLCaps&) const { return SkNEW_ARGS(GrGLQuadEffect, (*this, bt)); } @@ -484,7 +484,7 @@ public: static inline void GenKey(const GrGeometryProcessor&, const GrBatchTracker&, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder*); virtual void setData(const GrGLProgramDataManager& pdman, @@ -628,7 +628,7 @@ void GrGLCubicEffect::onEmitCode(EmitArgs& args, GrGPArgs* gpArgs) { void GrGLCubicEffect::GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const GrCubicEffect& ce = gp.cast(); const CubicBatchTracker& local = bt.cast(); @@ -644,13 +644,13 @@ void GrGLCubicEffect::GenKey(const GrGeometryProcessor& gp, GrCubicEffect::~GrCubicEffect() {} void GrCubicEffect::getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLCubicEffect::GenKey(*this, bt, caps, b); } GrGLPrimitiveProcessor* GrCubicEffect::createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const { + const GrGLSLCaps&) const { return SkNEW_ARGS(GrGLCubicEffect, (*this, bt)); } diff --git a/src/gpu/effects/GrBezierEffect.h b/src/gpu/effects/GrBezierEffect.h index 3390924243..7f79988470 100644 --- a/src/gpu/effects/GrBezierEffect.h +++ b/src/gpu/effects/GrBezierEffect.h @@ -99,11 +99,11 @@ public: inline GrPrimitiveEdgeType getEdgeType() const { return fEdgeType; } virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const override; + const GrGLSLCaps&) const override; void initBatchTracker(GrBatchTracker*, const GrPipelineInfo&) const override; bool onCanMakeEqual(const GrBatchTracker&, @@ -184,11 +184,11 @@ public: inline GrPrimitiveEdgeType getEdgeType() const { return fEdgeType; } virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const override; + const GrGLSLCaps&) const override; void initBatchTracker(GrBatchTracker*, const GrPipelineInfo&) const override; bool onCanMakeEqual(const GrBatchTracker&, @@ -265,11 +265,11 @@ public: inline GrPrimitiveEdgeType getEdgeType() const { return fEdgeType; } virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const override; + const GrGLSLCaps&) const override; void initBatchTracker(GrBatchTracker*, const GrPipelineInfo&) const override; bool onCanMakeEqual(const GrBatchTracker&, diff --git a/src/gpu/effects/GrBicubicEffect.cpp b/src/gpu/effects/GrBicubicEffect.cpp index 06d228e02a..7564467686 100644 --- a/src/gpu/effects/GrBicubicEffect.cpp +++ b/src/gpu/effects/GrBicubicEffect.cpp @@ -32,7 +32,7 @@ public: void setData(const GrGLProgramDataManager&, const GrProcessor&) override; - static inline void GenKey(const GrProcessor& effect, const GrGLCaps&, + static inline void GenKey(const GrProcessor& effect, const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const GrTextureDomain& domain = effect.cast().domain(); b->add32(GrTextureDomain::GLDomain::DomainKey(domain)); @@ -158,7 +158,7 @@ GrBicubicEffect::GrBicubicEffect(GrTexture* texture, GrBicubicEffect::~GrBicubicEffect() { } -void GrBicubicEffect::getGLProcessorKey(const GrGLCaps& caps, +void GrBicubicEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLBicubicEffect::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrBicubicEffect.h b/src/gpu/effects/GrBicubicEffect.h index ddbdd23437..b81c93e0a1 100644 --- a/src/gpu/effects/GrBicubicEffect.h +++ b/src/gpu/effects/GrBicubicEffect.h @@ -27,7 +27,7 @@ public: const char* name() const override { return "Bicubic"; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; diff --git a/src/gpu/effects/GrBitmapTextGeoProc.cpp b/src/gpu/effects/GrBitmapTextGeoProc.cpp index 25904f753f..7cce1df201 100644 --- a/src/gpu/effects/GrBitmapTextGeoProc.cpp +++ b/src/gpu/effects/GrBitmapTextGeoProc.cpp @@ -92,7 +92,7 @@ public: static inline void GenKey(const GrGeometryProcessor& proc, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const BitmapTextBatchTracker& local = bt.cast(); // We have to put the optional vertex attribute as part of the key. See the comment @@ -164,14 +164,14 @@ void GrBitmapTextGeoProc::onGetInvariantOutputCoverage(GrInitInvariantOutput* ou } void GrBitmapTextGeoProc::getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLBitmapTextGeoProc::GenKey(*this, bt, caps, b); } GrGLPrimitiveProcessor* GrBitmapTextGeoProc::createGLInstance(const GrBatchTracker& bt, - const GrGLCaps& caps) const { + const GrGLSLCaps& caps) const { return SkNEW_ARGS(GrGLBitmapTextGeoProc, (*this, bt)); } diff --git a/src/gpu/effects/GrBitmapTextGeoProc.h b/src/gpu/effects/GrBitmapTextGeoProc.h index 05f740fb57..136fd321b3 100644 --- a/src/gpu/effects/GrBitmapTextGeoProc.h +++ b/src/gpu/effects/GrBitmapTextGeoProc.h @@ -38,11 +38,11 @@ public: GrMaskFormat maskFormat() const { return fMaskFormat; } virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps& caps) const override; + const GrGLSLCaps& caps) const override; void initBatchTracker(GrBatchTracker*, const GrPipelineInfo&) const override; bool onCanMakeEqual(const GrBatchTracker&, diff --git a/src/gpu/effects/GrConfigConversionEffect.cpp b/src/gpu/effects/GrConfigConversionEffect.cpp index 27f8abf108..e536c8fadf 100644 --- a/src/gpu/effects/GrConfigConversionEffect.cpp +++ b/src/gpu/effects/GrConfigConversionEffect.cpp @@ -82,7 +82,7 @@ public: fsBuilder->codeAppend(modulate.c_str()); } - static inline void GenKey(const GrProcessor& processor, const GrGLCaps&, + static inline void GenKey(const GrProcessor& processor, const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const GrConfigConversionEffect& conv = processor.cast(); uint32_t key = (conv.swapsRedAndBlue() ? 0 : 1) | (conv.pmConversion() << 1); @@ -147,7 +147,7 @@ GrFragmentProcessor* GrConfigConversionEffect::TestCreate(SkRandom* random, /////////////////////////////////////////////////////////////////////////////// -void GrConfigConversionEffect::getGLProcessorKey(const GrGLCaps& caps, +void GrConfigConversionEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLConfigConversionEffect::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrConfigConversionEffect.h b/src/gpu/effects/GrConfigConversionEffect.h index 9be71f8277..3b471666f2 100644 --- a/src/gpu/effects/GrConfigConversionEffect.h +++ b/src/gpu/effects/GrConfigConversionEffect.h @@ -39,7 +39,7 @@ public: const char* name() const override { return "Config Conversion"; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; diff --git a/src/gpu/effects/GrConstColorProcessor.cpp b/src/gpu/effects/GrConstColorProcessor.cpp index 5f28694634..c48fc55abc 100644 --- a/src/gpu/effects/GrConstColorProcessor.cpp +++ b/src/gpu/effects/GrConstColorProcessor.cpp @@ -88,7 +88,7 @@ void GrConstColorProcessor::onComputeInvariantOutput(GrInvariantOutput* inout) c } } -void GrConstColorProcessor::getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder* b) const { +void GrConstColorProcessor::getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder* b) const { b->add32(fMode); } diff --git a/src/gpu/effects/GrConvexPolyEffect.cpp b/src/gpu/effects/GrConvexPolyEffect.cpp index a231df495d..538800e436 100644 --- a/src/gpu/effects/GrConvexPolyEffect.cpp +++ b/src/gpu/effects/GrConvexPolyEffect.cpp @@ -25,7 +25,7 @@ public: const char* name() const override { return "AARect"; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; @@ -92,7 +92,7 @@ public: const TransformedCoordsArray&, const TextureSamplerArray&) override; - static inline void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*); + static inline void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*); void setData(const GrGLProgramDataManager&, const GrProcessor&) override; @@ -160,13 +160,13 @@ void GLAARectEffect::setData(const GrGLProgramDataManager& pdman, const GrProces } } -void GLAARectEffect::GenKey(const GrProcessor& processor, const GrGLCaps&, +void GLAARectEffect::GenKey(const GrProcessor& processor, const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const AARectEffect& aare = processor.cast(); b->add32(aare.getEdgeType()); } -void AARectEffect::getGLProcessorKey(const GrGLCaps& caps, GrProcessorKeyBuilder* b) const { +void AARectEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLAARectEffect::GenKey(*this, caps, b); } @@ -187,7 +187,7 @@ public: const TransformedCoordsArray&, const TextureSamplerArray&) override; - static inline void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*); + static inline void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*); void setData(const GrGLProgramDataManager&, const GrProcessor&) override; @@ -252,7 +252,7 @@ void GrGLConvexPolyEffect::setData(const GrGLProgramDataManager& pdman, const Gr } } -void GrGLConvexPolyEffect::GenKey(const GrProcessor& processor, const GrGLCaps&, +void GrGLConvexPolyEffect::GenKey(const GrProcessor& processor, const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const GrConvexPolyEffect& cpe = processor.cast(); GR_STATIC_ASSERT(kGrProcessorEdgeTypeCnt <= 8); @@ -326,7 +326,7 @@ void GrConvexPolyEffect::onComputeInvariantOutput(GrInvariantOutput* inout) cons inout->mulByUnknownSingleComponent(); } -void GrConvexPolyEffect::getGLProcessorKey(const GrGLCaps& caps, +void GrConvexPolyEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLConvexPolyEffect::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrConvexPolyEffect.h b/src/gpu/effects/GrConvexPolyEffect.h index f47cc37afa..ba9456dc13 100644 --- a/src/gpu/effects/GrConvexPolyEffect.h +++ b/src/gpu/effects/GrConvexPolyEffect.h @@ -69,7 +69,7 @@ public: const SkScalar* getEdges() const { return fEdges; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; diff --git a/src/gpu/effects/GrConvolutionEffect.cpp b/src/gpu/effects/GrConvolutionEffect.cpp index 401cac3a3c..336778e037 100644 --- a/src/gpu/effects/GrConvolutionEffect.cpp +++ b/src/gpu/effects/GrConvolutionEffect.cpp @@ -27,7 +27,7 @@ public: void setData(const GrGLProgramDataManager& pdman, const GrProcessor&) override; - static inline void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*); + static inline void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*); private: int width() const { return Gr1DKernelEffect::WidthFromRadius(fRadius); } @@ -134,7 +134,7 @@ void GrGLConvolutionEffect::setData(const GrGLProgramDataManager& pdman, pdman.set1fv(fKernelUni, this->width(), conv.kernel()); } -void GrGLConvolutionEffect::GenKey(const GrProcessor& processor, const GrGLCaps&, +void GrGLConvolutionEffect::GenKey(const GrProcessor& processor, const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const GrConvolutionEffect& conv = processor.cast(); uint32_t key = conv.radius(); @@ -196,7 +196,7 @@ GrConvolutionEffect::GrConvolutionEffect(GrTexture* texture, GrConvolutionEffect::~GrConvolutionEffect() { } -void GrConvolutionEffect::getGLProcessorKey(const GrGLCaps& caps, +void GrConvolutionEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLConvolutionEffect::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrConvolutionEffect.h b/src/gpu/effects/GrConvolutionEffect.h index fa6329aa50..b885f19021 100644 --- a/src/gpu/effects/GrConvolutionEffect.h +++ b/src/gpu/effects/GrConvolutionEffect.h @@ -59,7 +59,7 @@ public: const char* name() const override { return "Convolution"; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; diff --git a/src/gpu/effects/GrCoverageSetOpXP.cpp b/src/gpu/effects/GrCoverageSetOpXP.cpp index 908ddd6f88..0c4818431a 100644 --- a/src/gpu/effects/GrCoverageSetOpXP.cpp +++ b/src/gpu/effects/GrCoverageSetOpXP.cpp @@ -40,7 +40,7 @@ public: private: CoverageSetOpXP(SkRegion::Op regionOp, bool fInvertCoverage); - void onGetGLProcessorKey(const GrGLCaps& caps, GrProcessorKeyBuilder* b) const override; + void onGetGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; void onGetBlendInfo(GrXferProcessor::BlendInfo* blendInfo) const override; @@ -64,7 +64,7 @@ public: ~GLCoverageSetOpXP() override {} - static void GenKey(const GrProcessor& processor, const GrGLCaps& caps, + static void GenKey(const GrProcessor& processor, const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) { const CoverageSetOpXP& xp = processor.cast(); uint32_t key = xp.invertCoverage() ? 0x0 : 0x1; @@ -99,7 +99,7 @@ CoverageSetOpXP::CoverageSetOpXP(SkRegion::Op regionOp, bool invertCoverage) CoverageSetOpXP::~CoverageSetOpXP() { } -void CoverageSetOpXP::onGetGLProcessorKey(const GrGLCaps& caps, GrProcessorKeyBuilder* b) const { +void CoverageSetOpXP::onGetGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLCoverageSetOpXP::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrCustomXfermode.cpp b/src/gpu/effects/GrCustomXfermode.cpp index 2d5352e934..195511d6d1 100644 --- a/src/gpu/effects/GrCustomXfermode.cpp +++ b/src/gpu/effects/GrCustomXfermode.cpp @@ -427,7 +427,7 @@ public: void setData(const GrGLProgramDataManager&, const GrProcessor&) override {} - static void GenKey(const GrFragmentProcessor& proc, const GrGLCaps&, GrProcessorKeyBuilder* b) { + static void GenKey(const GrFragmentProcessor& proc, const GrGLSLCaps&, GrProcessorKeyBuilder* b) { // The background may come from the dst or from a texture. uint32_t key = proc.numTextures(); SkASSERT(key <= 1); @@ -453,7 +453,7 @@ GrCustomXferFP::GrCustomXferFP(SkXfermode::Mode mode, GrTexture* background) this->addTextureAccess(&fBackgroundAccess); } -void GrCustomXferFP::getGLProcessorKey(const GrGLCaps& caps, GrProcessorKeyBuilder* b) const { +void GrCustomXferFP::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLCustomXferFP::GenKey(*this, caps, b); } @@ -514,7 +514,7 @@ public: private: CustomXP(SkXfermode::Mode mode, const GrDeviceCoordTexture* dstCopy, bool willReadDstColor); - void onGetGLProcessorKey(const GrGLCaps& caps, GrProcessorKeyBuilder* b) const override; + void onGetGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; bool onIsEqual(const GrXferProcessor& xpBase) const override; @@ -540,7 +540,7 @@ public: GLCustomXP(const GrXferProcessor&) {} ~GLCustomXP() override {} - static void GenKey(const GrXferProcessor& proc, const GrGLCaps&, GrProcessorKeyBuilder* b) { + static void GenKey(const GrXferProcessor& proc, const GrGLSLCaps&, GrProcessorKeyBuilder* b) { uint32_t key = proc.numTextures(); SkASSERT(key <= 1); key |= proc.cast().mode() << 1; @@ -573,7 +573,7 @@ CustomXP::CustomXP(SkXfermode::Mode mode, const GrDeviceCoordTexture* dstCopy, this->initClassID(); } -void CustomXP::onGetGLProcessorKey(const GrGLCaps& caps, GrProcessorKeyBuilder* b) const { +void CustomXP::onGetGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLCustomXP::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrCustomXfermodePriv.h b/src/gpu/effects/GrCustomXfermodePriv.h index 98883e04fa..216fb62d43 100644 --- a/src/gpu/effects/GrCustomXfermodePriv.h +++ b/src/gpu/effects/GrCustomXfermodePriv.h @@ -29,7 +29,7 @@ class GrCustomXferFP : public GrFragmentProcessor { public: GrCustomXferFP(SkXfermode::Mode mode, GrTexture* background); - void getGLProcessorKey(const GrGLCaps& caps, GrProcessorKeyBuilder* b) const override; + void getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; GrGLFragmentProcessor* createGLInstance() const override; diff --git a/src/gpu/effects/GrDashingEffect.cpp b/src/gpu/effects/GrDashingEffect.cpp index 0a9be3c311..08a3f670d9 100644 --- a/src/gpu/effects/GrDashingEffect.cpp +++ b/src/gpu/effects/GrDashingEffect.cpp @@ -797,11 +797,11 @@ public: DashAAMode aaMode() const { return fAAMode; } virtual void getGLProcessorKey(const GrBatchTracker&, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) const override; virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker&, - const GrGLCaps&) const override; + const GrGLSLCaps&) const override; void initBatchTracker(GrBatchTracker* bt, const GrPipelineInfo& init) const override; @@ -836,7 +836,7 @@ public: static inline void GenKey(const GrGeometryProcessor&, const GrBatchTracker&, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder*); virtual void setData(const GrGLProgramDataManager&, @@ -925,7 +925,7 @@ void GLDashingCircleEffect::setData(const GrGLProgramDataManager& pdman, void GLDashingCircleEffect::GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const DashingCircleBatchTracker& local = bt.cast(); const DashingCircleEffect& dce = gp.cast(); @@ -951,13 +951,13 @@ void DashingCircleEffect::onGetInvariantOutputCoverage(GrInitInvariantOutput* ou } void DashingCircleEffect::getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLDashingCircleEffect::GenKey(*this, bt, caps, b); } GrGLPrimitiveProcessor* DashingCircleEffect::createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const { + const GrGLSLCaps&) const { return SkNEW_ARGS(GLDashingCircleEffect, (*this, bt)); } @@ -1045,11 +1045,11 @@ public: DashAAMode aaMode() const { return fAAMode; } virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const override; + const GrGLSLCaps&) const override; void initBatchTracker(GrBatchTracker* bt, const GrPipelineInfo& init) const override; @@ -1084,7 +1084,7 @@ public: static inline void GenKey(const GrGeometryProcessor&, const GrBatchTracker&, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder*); virtual void setData(const GrGLProgramDataManager&, @@ -1185,7 +1185,7 @@ void GLDashingLineEffect::setData(const GrGLProgramDataManager& pdman, void GLDashingLineEffect::GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const DashingLineBatchTracker& local = bt.cast(); const DashingLineEffect& de = gp.cast(); @@ -1211,13 +1211,13 @@ void DashingLineEffect::onGetInvariantOutputCoverage(GrInitInvariantOutput* out) } void DashingLineEffect::getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLDashingLineEffect::GenKey(*this, bt, caps, b); } GrGLPrimitiveProcessor* DashingLineEffect::createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const { + const GrGLSLCaps&) const { return SkNEW_ARGS(GLDashingLineEffect, (*this, bt)); } diff --git a/src/gpu/effects/GrDisableColorXP.cpp b/src/gpu/effects/GrDisableColorXP.cpp index 08d7704cee..da84be1b1a 100644 --- a/src/gpu/effects/GrDisableColorXP.cpp +++ b/src/gpu/effects/GrDisableColorXP.cpp @@ -40,7 +40,7 @@ public: private: DisableColorXP(); - void onGetGLProcessorKey(const GrGLCaps& caps, GrProcessorKeyBuilder* b) const override; + void onGetGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; void onGetBlendInfo(GrXferProcessor::BlendInfo* blendInfo) const override; @@ -59,7 +59,7 @@ public: ~GLDisableColorXP() override {} - static void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*) {} + static void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*) {} private: void onEmitCode(const EmitArgs& args) override { @@ -81,7 +81,7 @@ DisableColorXP::DisableColorXP() { this->initClassID(); } -void DisableColorXP::onGetGLProcessorKey(const GrGLCaps& caps, GrProcessorKeyBuilder* b) const { +void DisableColorXP::onGetGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLDisableColorXP::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrDistanceFieldGeoProc.cpp b/src/gpu/effects/GrDistanceFieldGeoProc.cpp index ed087ddbd4..c15d45b588 100755 --- a/src/gpu/effects/GrDistanceFieldGeoProc.cpp +++ b/src/gpu/effects/GrDistanceFieldGeoProc.cpp @@ -165,7 +165,7 @@ public: static inline void GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const GrDistanceFieldA8TextGeoProc& dfTexEffect = gp.cast(); const DistanceFieldBatchTracker& local = bt.cast(); @@ -230,14 +230,14 @@ void GrDistanceFieldA8TextGeoProc::onGetInvariantOutputCoverage(GrInitInvariantO } void GrDistanceFieldA8TextGeoProc::getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLDistanceFieldA8TextGeoProc::GenKey(*this, bt, caps, b); } GrGLPrimitiveProcessor* GrDistanceFieldA8TextGeoProc::createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const { + const GrGLSLCaps&) const { return SkNEW_ARGS(GrGLDistanceFieldA8TextGeoProc, (*this, bt)); } @@ -420,7 +420,7 @@ public: static inline void GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const GrDistanceFieldPathGeoProc& dfTexEffect = gp.cast(); @@ -476,13 +476,13 @@ void GrDistanceFieldPathGeoProc::onGetInvariantOutputCoverage(GrInitInvariantOut } void GrDistanceFieldPathGeoProc::getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLDistanceFieldPathGeoProc::GenKey(*this, bt, caps, b); } GrGLPrimitiveProcessor* -GrDistanceFieldPathGeoProc::createGLInstance(const GrBatchTracker& bt, const GrGLCaps&) const { +GrDistanceFieldPathGeoProc::createGLInstance(const GrBatchTracker& bt, const GrGLSLCaps&) const { return SkNEW_ARGS(GrGLDistanceFieldPathGeoProc, (*this, bt)); } @@ -711,7 +711,7 @@ public: static inline void GenKey(const GrGeometryProcessor& gp, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const GrDistanceFieldLCDTextGeoProc& dfTexEffect = gp.cast(); @@ -763,14 +763,14 @@ void GrDistanceFieldLCDTextGeoProc::onGetInvariantOutputCoverage(GrInitInvariant } void GrDistanceFieldLCDTextGeoProc::getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLDistanceFieldLCDTextGeoProc::GenKey(*this, bt, caps, b); } GrGLPrimitiveProcessor* GrDistanceFieldLCDTextGeoProc::createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const { + const GrGLSLCaps&) const { return SkNEW_ARGS(GrGLDistanceFieldLCDTextGeoProc, (*this, bt)); } diff --git a/src/gpu/effects/GrDistanceFieldGeoProc.h b/src/gpu/effects/GrDistanceFieldGeoProc.h index 0ae0c0f711..362278f863 100644 --- a/src/gpu/effects/GrDistanceFieldGeoProc.h +++ b/src/gpu/effects/GrDistanceFieldGeoProc.h @@ -75,11 +75,11 @@ public: uint32_t getFlags() const { return fFlags; } virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const override; + const GrGLSLCaps&) const override; void initBatchTracker(GrBatchTracker* bt, const GrPipelineInfo& init) const override; @@ -139,11 +139,11 @@ public: uint32_t getFlags() const { return fFlags; } virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const override; + const GrGLSLCaps&) const override; void initBatchTracker(GrBatchTracker* bt, const GrPipelineInfo& init) const override; @@ -211,11 +211,11 @@ public: uint32_t getFlags() const { return fFlags; } virtual void getGLProcessorKey(const GrBatchTracker& bt, - const GrGLCaps& caps, + const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; virtual GrGLPrimitiveProcessor* createGLInstance(const GrBatchTracker& bt, - const GrGLCaps&) const override; + const GrGLSLCaps&) const override; void initBatchTracker(GrBatchTracker* bt, const GrPipelineInfo& init) const override; diff --git a/src/gpu/effects/GrDitherEffect.cpp b/src/gpu/effects/GrDitherEffect.cpp index 3a962b02f4..b692de8a06 100644 --- a/src/gpu/effects/GrDitherEffect.cpp +++ b/src/gpu/effects/GrDitherEffect.cpp @@ -26,7 +26,7 @@ public: const char* name() const override { return "Dither"; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; @@ -106,7 +106,7 @@ void GLDitherEffect::emitCode(GrGLFPBuilder* builder, ////////////////////////////////////////////////////////////////////////////// -void DitherEffect::getGLProcessorKey(const GrGLCaps& caps, +void DitherEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLDitherEffect::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrMatrixConvolutionEffect.cpp b/src/gpu/effects/GrMatrixConvolutionEffect.cpp index 0b0a8af194..e9c45363d0 100644 --- a/src/gpu/effects/GrMatrixConvolutionEffect.cpp +++ b/src/gpu/effects/GrMatrixConvolutionEffect.cpp @@ -20,7 +20,7 @@ public: const TransformedCoordsArray&, const TextureSamplerArray&) override; - static inline void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*); + static inline void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*); void setData(const GrGLProgramDataManager&, const GrProcessor&) override; @@ -112,7 +112,7 @@ void GrGLMatrixConvolutionEffect::emitCode(GrGLFPBuilder* builder, } void GrGLMatrixConvolutionEffect::GenKey(const GrProcessor& processor, - const GrGLCaps&, GrProcessorKeyBuilder* b) { + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const GrMatrixConvolutionEffect& m = processor.cast(); SkASSERT(m.kernelSize().width() <= 0x7FFF && m.kernelSize().height() <= 0xFFFF); uint32_t key = m.kernelSize().width() << 16 | m.kernelSize().height(); @@ -165,7 +165,7 @@ GrMatrixConvolutionEffect::GrMatrixConvolutionEffect(GrTexture* texture, GrMatrixConvolutionEffect::~GrMatrixConvolutionEffect() { } -void GrMatrixConvolutionEffect::getGLProcessorKey(const GrGLCaps& caps, +void GrMatrixConvolutionEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLMatrixConvolutionEffect::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrMatrixConvolutionEffect.h b/src/gpu/effects/GrMatrixConvolutionEffect.h index 33d4141da6..6a340fbbd5 100644 --- a/src/gpu/effects/GrMatrixConvolutionEffect.h +++ b/src/gpu/effects/GrMatrixConvolutionEffect.h @@ -62,7 +62,7 @@ public: const char* name() const override { return "MatrixConvolution"; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; diff --git a/src/gpu/effects/GrOvalEffect.cpp b/src/gpu/effects/GrOvalEffect.cpp index 0adbb67b9c..bce3c01b2c 100644 --- a/src/gpu/effects/GrOvalEffect.cpp +++ b/src/gpu/effects/GrOvalEffect.cpp @@ -24,7 +24,7 @@ public: const char* name() const override { return "Circle"; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; @@ -104,7 +104,7 @@ public: const TransformedCoordsArray&, const TextureSamplerArray&) override; - static inline void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*); + static inline void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*); void setData(const GrGLProgramDataManager&, const GrProcessor&) override; @@ -156,7 +156,7 @@ void GLCircleEffect::emitCode(GrGLFPBuilder* builder, (GrGLSLExpr4(inputColor) * GrGLSLExpr1("d")).c_str()); } -void GLCircleEffect::GenKey(const GrProcessor& processor, const GrGLCaps&, +void GLCircleEffect::GenKey(const GrProcessor& processor, const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const CircleEffect& ce = processor.cast(); b->add32(ce.getEdgeType()); @@ -179,7 +179,7 @@ void GLCircleEffect::setData(const GrGLProgramDataManager& pdman, const GrProces /////////////////////////////////////////////////////////////////////////////////////////////////// -void CircleEffect::getGLProcessorKey(const GrGLCaps& caps, +void CircleEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLCircleEffect::GenKey(*this, caps, b); } @@ -199,7 +199,7 @@ public: const char* name() const override { return "Ellipse"; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; @@ -282,7 +282,7 @@ public: const TransformedCoordsArray&, const TextureSamplerArray&) override; - static inline void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*); + static inline void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*); void setData(const GrGLProgramDataManager&, const GrProcessor&) override; @@ -347,7 +347,7 @@ void GLEllipseEffect::emitCode(GrGLFPBuilder* builder, (GrGLSLExpr4(inputColor) * GrGLSLExpr1("alpha")).c_str()); } -void GLEllipseEffect::GenKey(const GrProcessor& effect, const GrGLCaps&, +void GLEllipseEffect::GenKey(const GrProcessor& effect, const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const EllipseEffect& ee = effect.cast(); b->add32(ee.getEdgeType()); @@ -366,7 +366,7 @@ void GLEllipseEffect::setData(const GrGLProgramDataManager& pdman, const GrProce /////////////////////////////////////////////////////////////////////////////////////////////////// -void EllipseEffect::getGLProcessorKey(const GrGLCaps& caps, +void EllipseEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLEllipseEffect::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrPorterDuffXferProcessor.cpp b/src/gpu/effects/GrPorterDuffXferProcessor.cpp index 77b7fcaac0..eebf2e4ac7 100644 --- a/src/gpu/effects/GrPorterDuffXferProcessor.cpp +++ b/src/gpu/effects/GrPorterDuffXferProcessor.cpp @@ -97,7 +97,7 @@ private: PorterDuffXferProcessor(GrBlendCoeff srcBlend, GrBlendCoeff dstBlend, GrColor constant, const GrDeviceCoordTexture* dstCopy, bool willReadDstColor); - void onGetGLProcessorKey(const GrGLCaps& caps, GrProcessorKeyBuilder* b) const override; + void onGetGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override; void onGetBlendInfo(GrXferProcessor::BlendInfo* blendInfo) const override { if (!this->willReadDstColor()) { @@ -190,7 +190,7 @@ public: virtual ~GLPorterDuffXferProcessor() {} - static void GenKey(const GrProcessor& processor, const GrGLCaps& caps, + static void GenKey(const GrProcessor& processor, const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) { const PorterDuffXferProcessor& xp = processor.cast(); b->add32(xp.primaryOutputType()); @@ -291,7 +291,7 @@ PorterDuffXferProcessor::PorterDuffXferProcessor(GrBlendCoeff srcBlend, PorterDuffXferProcessor::~PorterDuffXferProcessor() { } -void PorterDuffXferProcessor::onGetGLProcessorKey(const GrGLCaps& caps, +void PorterDuffXferProcessor::onGetGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLPorterDuffXferProcessor::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrRRectEffect.cpp b/src/gpu/effects/GrRRectEffect.cpp index 24d5077d48..7b6595a88d 100644 --- a/src/gpu/effects/GrRRectEffect.cpp +++ b/src/gpu/effects/GrRRectEffect.cpp @@ -50,7 +50,7 @@ public: const char* name() const override { return "CircularRRect"; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; @@ -139,7 +139,7 @@ public: const TransformedCoordsArray&, const TextureSamplerArray&) override; - static inline void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*); + static inline void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*); void setData(const GrGLProgramDataManager&, const GrProcessor&) override; @@ -287,7 +287,7 @@ void GLCircularRRectEffect::emitCode(GrGLFPBuilder* builder, (GrGLSLExpr4(inputColor) * GrGLSLExpr1("alpha")).c_str()); } -void GLCircularRRectEffect::GenKey(const GrProcessor& processor, const GrGLCaps&, +void GLCircularRRectEffect::GenKey(const GrProcessor& processor, const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const CircularRRectEffect& crre = processor.cast(); GR_STATIC_ASSERT(kGrProcessorEdgeTypeCnt <= 8); @@ -375,7 +375,7 @@ void GLCircularRRectEffect::setData(const GrGLProgramDataManager& pdman, //////////////////////////////////////////////////////////////////////////////////////////////////// -void CircularRRectEffect::getGLProcessorKey(const GrGLCaps& caps, +void CircularRRectEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLCircularRRectEffect::GenKey(*this, caps, b); } @@ -394,7 +394,7 @@ public: const char* name() const override { return "EllipticalRRect"; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; @@ -496,7 +496,7 @@ public: const TransformedCoordsArray&, const TextureSamplerArray&) override; - static inline void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*); + static inline void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*); void setData(const GrGLProgramDataManager&, const GrProcessor&) override; @@ -587,7 +587,7 @@ void GLEllipticalRRectEffect::emitCode(GrGLFPBuilder* builder, (GrGLSLExpr4(inputColor) * GrGLSLExpr1("alpha")).c_str()); } -void GLEllipticalRRectEffect::GenKey(const GrProcessor& effect, const GrGLCaps&, +void GLEllipticalRRectEffect::GenKey(const GrProcessor& effect, const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const EllipticalRRectEffect& erre = effect.cast(); GR_STATIC_ASSERT(kLast_GrProcessorEdgeType < (1 << 3)); @@ -633,7 +633,7 @@ void GLEllipticalRRectEffect::setData(const GrGLProgramDataManager& pdman, //////////////////////////////////////////////////////////////////////////////////////////////////// -void EllipticalRRectEffect::getGLProcessorKey(const GrGLCaps& caps, +void EllipticalRRectEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GLEllipticalRRectEffect::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrSimpleTextureEffect.cpp b/src/gpu/effects/GrSimpleTextureEffect.cpp index 76eece4af1..abbd89cafa 100644 --- a/src/gpu/effects/GrSimpleTextureEffect.cpp +++ b/src/gpu/effects/GrSimpleTextureEffect.cpp @@ -43,7 +43,7 @@ void GrSimpleTextureEffect::onComputeInvariantOutput(GrInvariantOutput* inout) c this->updateInvariantOutputForModulation(inout); } -void GrSimpleTextureEffect::getGLProcessorKey(const GrGLCaps& caps, +void GrSimpleTextureEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLSimpleTextureEffect::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrSimpleTextureEffect.h b/src/gpu/effects/GrSimpleTextureEffect.h index 21d0398a73..79e660f0a3 100644 --- a/src/gpu/effects/GrSimpleTextureEffect.h +++ b/src/gpu/effects/GrSimpleTextureEffect.h @@ -49,7 +49,7 @@ public: const char* name() const override { return "SimpleTexture"; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; diff --git a/src/gpu/effects/GrTextureDomain.cpp b/src/gpu/effects/GrTextureDomain.cpp index f14ebaa855..45b5546c1b 100644 --- a/src/gpu/effects/GrTextureDomain.cpp +++ b/src/gpu/effects/GrTextureDomain.cpp @@ -178,7 +178,7 @@ public: void setData(const GrGLProgramDataManager&, const GrProcessor&) override; - static inline void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*); + static inline void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*); private: GrTextureDomain::GLDomain fGLDomain; @@ -209,7 +209,7 @@ void GrGLTextureDomainEffect::setData(const GrGLProgramDataManager& pdman, fGLDomain.setData(pdman, domain, processor.texture(0)->origin()); } -void GrGLTextureDomainEffect::GenKey(const GrProcessor& processor, const GrGLCaps&, +void GrGLTextureDomainEffect::GenKey(const GrProcessor& processor, const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const GrTextureDomain& domain = processor.cast().textureDomain(); b->add32(GrTextureDomain::GLDomain::DomainKey(domain)); @@ -256,7 +256,7 @@ GrTextureDomainEffect::~GrTextureDomainEffect() { } -void GrTextureDomainEffect::getGLProcessorKey(const GrGLCaps& caps, +void GrTextureDomainEffect::getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const { GrGLTextureDomainEffect::GenKey(*this, caps, b); } diff --git a/src/gpu/effects/GrTextureDomain.h b/src/gpu/effects/GrTextureDomain.h index 6a462f96a8..161058037d 100644 --- a/src/gpu/effects/GrTextureDomain.h +++ b/src/gpu/effects/GrTextureDomain.h @@ -168,7 +168,7 @@ public: const char* name() const override { return "TextureDomain"; } - void getGLProcessorKey(const GrGLCaps&, GrProcessorKeyBuilder*) const override; + void getGLProcessorKey(const GrGLSLCaps&, GrProcessorKeyBuilder*) const override; GrGLFragmentProcessor* createGLInstance() const override; diff --git a/src/gpu/effects/GrYUVtoRGBEffect.cpp b/src/gpu/effects/GrYUVtoRGBEffect.cpp index e92500a29d..0296cd76cd 100644 --- a/src/gpu/effects/GrYUVtoRGBEffect.cpp +++ b/src/gpu/effects/GrYUVtoRGBEffect.cpp @@ -56,7 +56,7 @@ public: static const GrGLfloat kRec601ConversionMatrix[16]; // this class always generates the same code. - static void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*) {} + static void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*) {} GLProcessor(const GrProcessor&) {} @@ -100,7 +100,7 @@ public: typedef GrGLFragmentProcessor INHERITED; }; - virtual void getGLProcessorKey(const GrGLCaps& caps, + virtual void getGLProcessorKey(const GrGLSLCaps& caps, GrProcessorKeyBuilder* b) const override { GLProcessor::GenKey(*this, caps, b); } diff --git a/src/gpu/gl/GrGLCaps.h b/src/gpu/gl/GrGLCaps.h index 71bbafd8d9..954b158809 100644 --- a/src/gpu/gl/GrGLCaps.h +++ b/src/gpu/gl/GrGLCaps.h @@ -395,4 +395,6 @@ private: typedef GrDrawTargetCaps INHERITED; }; +typedef GrGLCaps GrGLSLCaps; + #endif diff --git a/src/gpu/gl/GrGLPathProcessor.cpp b/src/gpu/gl/GrGLPathProcessor.cpp index 75bd0d631c..96175913bc 100644 --- a/src/gpu/gl/GrGLPathProcessor.cpp +++ b/src/gpu/gl/GrGLPathProcessor.cpp @@ -41,7 +41,7 @@ void GrGLPathProcessor::emitCode(EmitArgs& args) { void GrGLPathProcessor::GenKey(const GrPathProcessor&, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b) { const PathBatchTracker& local = bt.cast(); b->add32(local.fInputColorType | local.fInputCoverageType << 16); diff --git a/src/gpu/gl/GrGLPathProcessor.h b/src/gpu/gl/GrGLPathProcessor.h index 66a39fe103..d6fc1706d7 100644 --- a/src/gpu/gl/GrGLPathProcessor.h +++ b/src/gpu/gl/GrGLPathProcessor.h @@ -20,7 +20,7 @@ public: static void GenKey(const GrPathProcessor&, const GrBatchTracker& bt, - const GrGLCaps&, + const GrGLSLCaps&, GrProcessorKeyBuilder* b); void emitCode(EmitArgs&) override; diff --git a/src/gpu/gl/GrGLProcessor.h b/src/gpu/gl/GrGLProcessor.h index a4fad753ec..017585d306 100644 --- a/src/gpu/gl/GrGLProcessor.h +++ b/src/gpu/gl/GrGLProcessor.h @@ -17,7 +17,7 @@ include/gpu/GrProcessor.h. Objects of type GrGLProcessor are responsible for emitting the GLSL code that implements a GrProcessor and for uploading uniforms at draw time. If they don't always emit the same GLSL code, they must have a function: - static inline void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*) + static inline void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*) that is used to implement a program cache. When two GrProcessors produce the same key this means that their GrGLProcessors would emit the same GLSL code. @@ -111,7 +111,7 @@ public: // TODO update this to pass in GrFragmentProcessor virtual void setData(const GrGLProgramDataManager&, const GrProcessor&) {} - static void GenKey(const GrProcessor&, const GrGLCaps&, GrProcessorKeyBuilder*) {} + static void GenKey(const GrProcessor&, const GrGLSLCaps&, GrProcessorKeyBuilder*) {} private: typedef GrGLProcessor INHERITED; diff --git a/src/gpu/gl/GrGLProgramDesc.cpp b/src/gpu/gl/GrGLProgramDesc.cpp index 87e7f2d834..af688129ba 100644 --- a/src/gpu/gl/GrGLProgramDesc.cpp +++ b/src/gpu/gl/GrGLProgramDesc.cpp @@ -106,6 +106,7 @@ bool GrGLProgramDescBuilder::Build(GrProgramDesc* desc, GrProcessorKeyBuilder b(&glDesc->key()); primProc.getGLProcessorKey(batchTracker, gpu->glCaps(), &b); + //**** use glslCaps here? if (!get_meta_key(primProc, gpu->glCaps(), 0, &b)) { glDesc->key().reset(); return false; @@ -115,6 +116,7 @@ bool GrGLProgramDescBuilder::Build(GrProgramDesc* desc, const GrPendingFragmentStage& fps = pipeline.getFragmentStage(s); const GrFragmentProcessor& fp = *fps.processor(); fp.getGLProcessorKey(gpu->glCaps(), &b); + //**** use glslCaps here? if (!get_meta_key(fp, gpu->glCaps(), primProc.getTransformKey(fp.coordTransforms()), &b)) { glDesc->key().reset(); return false; @@ -123,6 +125,7 @@ bool GrGLProgramDescBuilder::Build(GrProgramDesc* desc, const GrXferProcessor& xp = *pipeline.getXferProcessor(); xp.getGLProcessorKey(gpu->glCaps(), &b); + //**** use glslCaps here? if (!get_meta_key(xp, gpu->glCaps(), 0, &b)) { glDesc->key().reset(); return false; -- cgit v1.2.3