commit 1694a8604be7399a0b445c9b7814b9d40cc9de9a
author Jim Van Verth <jvanverth@google.com> Mon Dec 17 19:48:42 2018 +0000
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Mon Dec 17 20:20:45 2018 +0000
tree 39f94494d0e08dcc5df3323a666ff529c2ea0286
parent fe64e0099732d74ec9107765e8e6de8bcead2552

Revert "Reland Reland "Remove use of integers for atlas indexing""

This reverts commit 0711094b9150940e8c27b0d7b2c2ad85991a74db.

Reason for revert: Seeing artifacts on Pixel 3 and iOS.

Original change's description:
> Reland Reland "Remove use of integers for atlas indexing"
> 
> Bug: skia:
> Change-Id: Ie37b3fd6c8682b2d778b1dd0bc38976e6761524e
> Reviewed-on: https://skia-review.googlesource.com/c/172142
> Reviewed-by: Brian Salomon <bsalomon@google.com>
> Commit-Queue: Jim Van Verth <jvanverth@google.com>

TBR=jvanverth@google.com,bsalomon@google.com

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: skia:
Change-Id: Iff9df856ff74eeb66bf5e37496a8361474e99491
Reviewed-on: https://skia-review.googlesource.com/c/178265
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>

src/gpu/effects/GrBitmapTextGeoProc.cpp

diff --git a/src/gpu/effects/GrBitmapTextGeoProc.cpp b/src/gpu/effects/GrBitmapTextGeoProc.cpp
index cd98f1d..370b0e5 100644
--- a/src/gpu/effects/GrBitmapTextGeoProc.cpp
+++ b/src/gpu/effects/GrBitmapTextGeoProc.cpp
@@ -40,7 +40,8 @@
                                                           &atlasSizeInvName);
 
         GrGLSLVarying uv(kFloat2_GrSLType);
-        GrGLSLVarying texIdx(kFloat_GrSLType);
+        GrSLType texIdxType = args.fShaderCaps->integerSupport() ? kInt_GrSLType : kFloat_GrSLType;
+        GrGLSLVarying texIdx(texIdxType);
         append_index_uv_varyings(args, btgp.inTextureCoords().name(), atlasSizeInvName, &uv,
                                  &texIdx, nullptr);