diff options
author | Mike Klein <mtklein@google.com> | 2018-06-19 01:40:57 +0000 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-06-19 01:41:10 +0000 |
commit | 5045e501d2aec23e5f1e4b46346033ac3202c6b0 (patch) | |
tree | b0179c300d6bc1822b0d945be812fff267bb414a /src/gpu/ccpr/GrCCPathProcessor.h | |
parent | 63b3bfb711d7e3d4f9ad75681d77a69a3c454ab0 (diff) |
Revert "Change how vertex/instance attributes are handled in geometry processors."
This reverts commit 19c1233c447f625c2522e7ecd0a0adecc629bb2f.
Reason for revert: want to make sure Google3 can roll
Original change's description:
> Change how vertex/instance attributes are handled in geometry processors.
>
> * No longer register vertex/instance attributes on base class, just counts
>
> * Separate instance and vertex attributes and remove InputRate and offset
>
> * Make attributes constexpr where possible
>
> Change-Id: I1f1d5e772fa177a96d2aeb805aab7b69f35bfae6
> Reviewed-on: https://skia-review.googlesource.com/132405
> Commit-Queue: Brian Salomon <bsalomon@google.com>
> Reviewed-by: Chris Dalton <csmartdalton@google.com>
TBR=egdaniel@google.com,bsalomon@google.com,csmartdalton@google.com
Change-Id: I4800632515e14fbf54af52826928ac915657b59f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/135661
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
Diffstat (limited to 'src/gpu/ccpr/GrCCPathProcessor.h')
-rw-r--r-- | src/gpu/ccpr/GrCCPathProcessor.h | 23 |
1 files changed, 9 insertions, 14 deletions
diff --git a/src/gpu/ccpr/GrCCPathProcessor.h b/src/gpu/ccpr/GrCCPathProcessor.h index 54c1b7e2ed..785dd2c919 100644 --- a/src/gpu/ccpr/GrCCPathProcessor.h +++ b/src/gpu/ccpr/GrCCPathProcessor.h @@ -77,11 +77,16 @@ public: const GrTexture* atlas() const { return fAtlasAccess.peekTexture(); } const SkMatrix& localMatrix() const { return fLocalMatrix; } const Attribute& getInstanceAttrib(InstanceAttribs attribID) const { - int idx = static_cast<int>(attribID); - SkASSERT(idx >= 0 && idx < static_cast<int>(SK_ARRAY_COUNT(kInstanceAttribs))); - return kInstanceAttribs[idx]; + const Attribute& attrib = this->getAttrib((int)attribID); + SkASSERT(Attribute::InputRate::kPerInstance == attrib.inputRate()); + return attrib; + } + const Attribute& getEdgeNormsAttrib() const { + SkASSERT(1 + kNumInstanceAttribs == this->numAttribs()); + const Attribute& attrib = this->getAttrib(kNumInstanceAttribs); + SkASSERT(Attribute::InputRate::kPerVertex == attrib.inputRate()); + return attrib; } - const Attribute& getEdgeNormsAttrib() const { return kEdgeNormsAttrib; } void getGLSLProcessorKey(const GrShaderCaps&, GrProcessorKeyBuilder*) const override {} GrGLSLPrimitiveProcessor* createGLSLInstance(const GrShaderCaps&) const override; @@ -90,18 +95,8 @@ public: int baseInstance, int endInstance, const SkRect& bounds) const; private: - const Attribute& onVertexAttribute(int i) const override { return kEdgeNormsAttrib; } - const Attribute& onInstanceAttribute(int i) const override { return kInstanceAttribs[i]; } - const TextureSampler fAtlasAccess; SkMatrix fLocalMatrix; - static constexpr Attribute kInstanceAttribs[kNumInstanceAttribs] = { - {"devbounds", kFloat4_GrVertexAttribType}, - {"devbounds45", kFloat4_GrVertexAttribType}, - {"dev_to_atlas_offset", kInt2_GrVertexAttribType}, - {"color", kUByte4_norm_GrVertexAttribType} - }; - static constexpr Attribute kEdgeNormsAttrib = {"edge_norms", kFloat4_GrVertexAttribType}; typedef GrGeometryProcessor INHERITED; }; |