commit 88d7e625f9e083dba349373eeeeb73b0018a1f07
author Brian Salomon <bsalomon@google.com> Thu Nov 05 11:11:20 2020 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Nov 05 20:21:14 2020 +0000
tree 8eb26a3202f0c73061888586c7a5901d2cb3b0a3
parent 127797193962d40ec0f4aec52bc346a530d1f4a6

Make ManagedBackendTexture fail on invalid GrBackendTexture

Makes it more sensible.

A bunch of call sites we're already written to expect this. Update
some others.

Change-Id: I77c28045ebf01e6aa9d92d2ebc37287604ec10c9
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/332544
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>

tools/gpu/BackendTextureImageFactory.cpp

diff --git a/tools/gpu/BackendTextureImageFactory.cpp b/tools/gpu/BackendTextureImageFactory.cpp
index e2950af..700f64f5 100644
--- a/tools/gpu/BackendTextureImageFactory.cpp
+++ b/tools/gpu/BackendTextureImageFactory.cpp
@@ -31,6 +31,9 @@
         src = &temp;
     }
     auto mbet = ManagedBackendTexture::MakeWithData(dContext, src, 1, renderable, GrProtected::kNo);
+    if (!mbet) {
+        return nullptr;
+    }
     return SkImage::MakeFromTexture(dContext,
                                     mbet->texture(),
                                     origin,
@@ -61,7 +64,9 @@
                                                     mipmapped,
                                                     renderable,
                                                     GrProtected::kNo);
-
+    if (!mbet) {
+        return nullptr;
+    }
     return SkImage::MakeFromTexture(dContext,
                                     mbet->texture(),
                                     origin,