diff options
author | 2018-06-06 17:16:05 -0400 | |
---|---|---|
committer | 2018-06-06 21:56:24 +0000 | |
commit | 969bdef64139c417f5c0b8f2fbf2638bb9d7dc95 (patch) | |
tree | fb59076e70b44760382842b574dd4e132cbc25e6 /src/gpu/GrDefaultGeoProcFactory.cpp | |
parent | 7db95a645147b7fef47647dc5d3df32b05b141d9 (diff) |
Remove GrGeometryProcessor knowledge of explicit local coords
This seems to be part of an older model where the base class was more
aware of how subclasses handled local coords.
Only the default geometry processor uses it and it already incorporates
the existence of explicit local coords into its shader key.
Other GPs that used explicit local coords don't use this.
Change-Id: Ia396f2a5e626e57470905ae770f1576386c0cefb
Reviewed-on: https://skia-review.googlesource.com/132665
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
Auto-Submit: Brian Salomon <bsalomon@google.com>
Diffstat (limited to 'src/gpu/GrDefaultGeoProcFactory.cpp')
-rw-r--r-- | src/gpu/GrDefaultGeoProcFactory.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/gpu/GrDefaultGeoProcFactory.cpp b/src/gpu/GrDefaultGeoProcFactory.cpp index 1448aec660..dced58a67d 100644 --- a/src/gpu/GrDefaultGeoProcFactory.cpp +++ b/src/gpu/GrDefaultGeoProcFactory.cpp @@ -139,7 +139,7 @@ public: gp.viewMatrix(), &fViewMatrixUniform); - if (gp.hasExplicitLocalCoords()) { + if (gp.inLocalCoords()) { // emit transforms with explicit local coords this->emitTransforms(vertBuilder, varyingHandler, @@ -260,7 +260,6 @@ private: } if (fFlags & kLocalCoordAttribute_GPFlag) { fInLocalCoords = &this->addVertexAttrib("inLocalCoord", kFloat2_GrVertexAttribType); - this->setHasExplicitLocalCoords(); } if (fFlags & kCoverageAttribute_GPFlag) { fInCoverage = &this->addVertexAttrib("inCoverage", kHalf_GrVertexAttribType); |