commit d3420de4a9fffe6b188a140875fe22d07ddbca15
author Jim Van Verth <jvanverth@google.com> Thu Jun 28 15:22:43 2018 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Jun 28 19:50:47 2018 +0000
tree 1734e705da0784de607bb482e2c001ec64b0d58e
parent a1e5630183c1b0d5a7d6df53f30e026aedb2c539

Return to low-precision ovals

Bug: skia:
Change-Id: I3d4399f54ab73ecabf2cf646ea072c34d516fdde
Reviewed-on: https://skia-review.googlesource.com/138268
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>

src/gpu/ops/GrOvalOpFactory.cpp

diff --git a/src/gpu/ops/GrOvalOpFactory.cpp b/src/gpu/ops/GrOvalOpFactory.cpp
index ebc602d..cb7dade 100644
--- a/src/gpu/ops/GrOvalOpFactory.cpp
+++ b/src/gpu/ops/GrOvalOpFactory.cpp
@@ -595,11 +595,7 @@
             fragBuilder->codeAppend("half grad_dot = dot(grad, grad);");
 
             // avoid calling inversesqrt on zero.
-            if (args.fShaderCaps->halfIs32Bits()) {
-                fragBuilder->codeAppend("grad_dot = max(grad_dot, 1.0e-6);");
-            } else {
-                fragBuilder->codeAppend("grad_dot = max(grad_dot, 5.0e-5);");
-            }
+            fragBuilder->codeAppend("grad_dot = max(grad_dot, 1.0e-4);");
             fragBuilder->codeAppend("half invlen = inversesqrt(grad_dot);");
             fragBuilder->codeAppend("half edgeAlpha = clamp(0.5-test*invlen, 0.0, 1.0);");
 
@@ -751,11 +747,7 @@
 
             fragBuilder->codeAppend("half grad_dot = dot(grad, grad);");
             // avoid calling inversesqrt on zero.
-            if (args.fShaderCaps->halfIs32Bits()) {
-                fragBuilder->codeAppend("grad_dot = max(grad_dot, 1.0e-6);");
-            } else {
-                fragBuilder->codeAppend("grad_dot = max(grad_dot, 5.0e-5);");
-            }
+            fragBuilder->codeAppend("grad_dot = max(grad_dot, 1.0e-4);");
             fragBuilder->codeAppend("half invlen = inversesqrt(grad_dot);");
             if (DIEllipseStyle::kHairline == diegp.fStyle) {
                 // can probably do this with one step