commit 19ec80fb37ceea46b7c010ef729d1824b18482f0
author Brian Salomon <bsalomon@google.com> Fri Nov 16 13:27:30 2018 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Nov 16 19:38:05 2018 +0000
tree 970b0408fa7e57d72117e7bdd7fd070821713501
parent 09181ef04259c7ebafb4ea72f6a0e0ee45245522

Discard ops that don't have a finite bounds.

Bug: chromium:905782
Change-Id: I2f83d0371dad5d812163cd15af4690983a6e074f
Reviewed-on: https://skia-review.googlesource.com/c/171533
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>

src/gpu/GrRenderTargetOpList.cpp

diff --git a/src/gpu/GrRenderTargetOpList.cpp b/src/gpu/GrRenderTargetOpList.cpp
index 30b5f5c..010fe6c 100644
--- a/src/gpu/GrRenderTargetOpList.cpp
+++ b/src/gpu/GrRenderTargetOpList.cpp
@@ -619,6 +619,10 @@
 
     // A closed GrOpList should never receive new/more ops
     SkASSERT(!this->isClosed());
+    if (!op->bounds().isFinite()) {
+        fOpMemoryPool->release(std::move(op));
+        return;
+    }
 
     // Check if there is an op we can combine with by linearly searching back until we either
     // 1) check every op