Minor fix to VkWrapTest
authorGreg Daniel <egdaniel@google.com>
Mon, 27 Mar 2017 20:18:03 +0000 (16:18 -0400)
committerSkia Commit-Bot <skia-commit-bot@chromium.org>
Mon, 27 Mar 2017 20:48:00 +0000 (20:48 +0000)
We need to make sure we cleanup the VkImage created when testing the
wrapping of a render target.

BUG=skia:

Change-Id: I60ea7e880fecccca309a250cc3b19eb29d6aa89c
Reviewed-on: https://skia-review.googlesource.com/10206
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Greg Daniel <egdaniel@google.com>

tests/VkWrapTests.cpp

index 7de3160..9279ef6 100644 (file)
@@ -100,7 +100,9 @@ void wrap_rt_test(skiatest::Reporter* reporter, GrContext* context) {
     rt = gpu->wrapBackendRenderTarget(desc);
     REPORTER_ASSERT(reporter, rt);
 
-    gpu->deleteTestingOnlyBackendTexture(backendObj, true);
+    // When we wrapBackendRenderTarget it is always borrowed, so we must make sure to free the
+    // resource when we're done.
+    gpu->deleteTestingOnlyBackendTexture(backendObj, false);
 }
 
 void wrap_trt_test(skiatest::Reporter* reporter, GrContext* context) {