Add handling for allocation failure in GrTextureOpList

I believe this is what has been causing the intermittent TSAN crash

Change-Id: Ie7170a3eb0495cebe8b1398a00f98ad099f78944
Reviewed-on: https://skia-review.googlesource.com/116629
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>

1 files changed, 7 insertions, 0 deletions

diff --git a/src/gpu/GrTextureOpList.cpp b/src/gpu/GrTextureOpList.cpp
index ad00d95a00..45deb2d59b 100644
--- a/src/gpu/GrTextureOpList.cpp
+++ b/src/gpu/GrTextureOpList.cpp
@@ -51,6 +51,7 @@ void GrTextureOpList::dump() const {
 #endif
 
 void GrTextureOpList::onPrepare(GrOpFlushState* flushState) {
+    SkASSERT(fTarget.get()->priv().peekTexture());
     SkASSERT(this->isClosed());
 
     // Loop over the ops that haven't yet generated their geometry
@@ -74,12 +75,18 @@ bool GrTextureOpList::onExecute(GrOpFlushState* flushState) {
         return false;
     }
 
+    SkASSERT(fTarget.get()->priv().peekTexture());
+
     std::unique_ptr<GrGpuTextureCommandBuffer> commandBuffer(
                          flushState->gpu()->createCommandBuffer(fTarget.get()->priv().peekTexture(),
                                                                 fTarget.get()->origin()));
     flushState->setCommandBuffer(commandBuffer.get());
 
     for (int i = 0; i < fRecordedOps.count(); ++i) {
+        if (!fRecordedOps[i]) {
+            continue;
+        }
+
         GrOpFlushState::OpArgs opArgs = {
             fRecordedOps[i].get(),
             nullptr,