commit 411090f669744bd21a923042e691a907a7e8a2e0
author Bo Liu <boliu@google.com> Tue Jul 21 10:51:18 2020 -0400
committer Greg Daniel <egdaniel@google.com> Tue Jul 21 16:29:10 2020 +0000
tree 40b7c758b1e61f1e1bf89851594e7bee880ac2de
parent b43313646796dd2fe58afb881a81a31f9828e366

[RESTRICT AUTOMERGE]: Stop discarding stencil buffers on tilers

Bug: b/160958008
Test: SystemWebViewShell.apk
      - Tap 3 dot menu
      - Tap "Animation test"
      - Check "Rounded corner" checkbox
Change-Id: I67851cb4be1a95f9f1d813846f5d4ba4b1535d9c
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/304176
Reviewed-by: Chris Dalton <csmartdalton@google.com>

src/gpu/GrCaps.h

diff --git a/src/gpu/GrCaps.h b/src/gpu/GrCaps.h
index d98890d..9d7a0c1 100644
--- a/src/gpu/GrCaps.h
+++ b/src/gpu/GrCaps.h
@@ -74,9 +74,14 @@
     // Should we discard stencil values after a render pass? (Tilers get better performance if we
     // always load stencil buffers with a "clear" op, and then discard the content when finished.)
     bool discardStencilValuesAfterRenderPass() const {
+#if defined(SK_BUILD_FOR_ANDROID)
+        // b/160958008
+        return false;
+#else
         // This method is actually just a duplicate of preferFullscreenClears(), with a descriptive
         // name for the sake of readability.
         return this->preferFullscreenClears();
+#endif
     }
 
     bool preferVRAMUseOverFlushes() const { return fPreferVRAMUseOverFlushes; }