commit 4cf00a814f7d8e6428a10441930515f5a6a527e3
author Robert Phillips <robertphillips@google.com> Tue Sep 29 15:43:10 2020 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Sep 29 23:41:18 2020 +0000
tree 374c3be269138f5295b58f21c4652428602243ee
parent 9ecb3abfdfe804ca149639f436cb09db4d801c37

Make GrRRectBlurEffect use the thread-safe uniquely-keyed view cache

This yields a ~8% performance improvement on OOP-R/DDL-like rendering of the simpleblurroundrect GM.

Bug: 1108408
Change-Id: I1ec9477dffe870e5973f8a334a65b1013a4ca3dd
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/311720
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>

tests/GrThreadSafeViewCacheTest.cpp

diff --git a/tests/GrThreadSafeViewCacheTest.cpp b/tests/GrThreadSafeViewCacheTest.cpp
index 134d7e4..fe9b0f8 100644
--- a/tests/GrThreadSafeViewCacheTest.cpp
+++ b/tests/GrThreadSafeViewCacheTest.cpp
@@ -417,7 +417,7 @@
         GrSurfaceProxyView view;
 
         // We can "fail the lookup" to simulate a threaded race condition
-        if (auto view = threadSafeViewCache->find(key); !failLookup && view) {
+        if (view = threadSafeViewCache->find(key); !failLookup && view) {
             ++stats->fCacheHits;
             return view;
         }