commit e9ea414ac07361998dc9e204e1d3fc6f885f5ed6
author Adlai Holler <adlai@google.com> Tue Apr 27 14:31:56 2021 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Apr 27 21:01:55 2021 +0000
tree 3b4e36ed3ab960b4f62f3ce614c1f8dd3a76421d
parent 169c8903be80da59206600f9145c5888857ffb4f

Improve API for skipping render tasks

This lets the user query if a task is skippable, makes
the naming clearer, and only calls the virtual once.

Bug: skia:10877
Change-Id: Ia8a7e0e3014d447bd8b4e914edf4f705a2ea107b
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/401856
Commit-Queue: Adlai Holler <adlai@google.com>
Auto-Submit: Adlai Holler <adlai@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>

tests/WrappedSurfaceCopyOnWriteTest.cpp

diff --git a/tests/WrappedSurfaceCopyOnWriteTest.cpp b/tests/WrappedSurfaceCopyOnWriteTest.cpp
index 52fbad6..1a91568 100644
--- a/tests/WrappedSurfaceCopyOnWriteTest.cpp
+++ b/tests/WrappedSurfaceCopyOnWriteTest.cpp
@@ -110,7 +110,7 @@
         return;
     }
 
-    task->canSkip();
+    task->makeSkippable();
 
     SkAutoPixmapStorage pixels;
     pixels.alloc(SkImageInfo::Make({10, 10}, kRGBA_8888_SkColorType, kPremul_SkAlphaType));
@@ -151,7 +151,7 @@
     temp->clear(SkPMColor4f{0, 0, 0, 0});
 
     GrSurfaceProxyView readView = dst->readSurfaceView();
-    task->canSkip();
+    task->makeSkippable();
 
     SkAutoPixmapStorage pixels;
     pixels.alloc(SkImageInfo::Make({10, 10}, kRGBA_8888_SkColorType, kPremul_SkAlphaType));