Track whether TextureGL owns the handle and should delete it

Change-Id: I2cf21fb0924c47aeb1aeae298661dc4a128a0c6c
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/147100
Reviewed-by: Loko Kung <[email protected]>
Reviewed-by: Stephen White <[email protected]>
Commit-Queue: Austin Eng <[email protected]>
Kokoro: Kokoro <[email protected]>

Author: austinEng

src/dawn/native/opengl/TextureGL.cpp

@@ -186,6 +186,7 @@ Texture::Texture(Device* device, const TextureDescriptor* descriptor)
     const OpenGLFunctions& gl = device->GetGL();
 
     gl.GenTextures(1, &mHandle);
+    mOwnsHandle = true;
     uint32_t levels = GetNumMipLevels();
 
     const GLFormat& glFormat = GetGLFormat();
@@ -216,7 +217,7 @@ Texture::~Texture() {}
 
 void Texture::DestroyImpl() {
     TextureBase::DestroyImpl();
-    if (mHandle != 0) {
+    if (mOwnsHandle) {
         const OpenGLFunctions& gl = ToBackend(GetDevice())->GetGL();
         gl.DeleteTextures(1, &mHandle);
         mHandle = 0;

src/dawn/native/opengl/TextureGL.h

@@ -45,6 +45,7 @@ class Texture final : public TextureBase {
     MaybeError ClearTexture(const SubresourceRange& range, TextureBase::ClearValue clearValue);
 
     GLuint mHandle;
+    bool mOwnsHandle = false;
     GLenum mTarget;
     uint32_t mGenID = 0;
 };