commit fb7a0c5f19a7bc4f4b2ac59598a501c34ff2e768
author Yahan Zhou <yahan@google.com> Thu Jun 02 16:15:43 2022 -0700
committer Yahan Zhou <yahan@google.com> Thu Jun 02 16:19:43 2022 -0700
tree ad61cc70872314f2c9e818cfd0fa82a5d4416257
parent c79a991147515301e51cc847539817169c85d112

Use share pointer for texture data

Skia tries to do the following:

1. create texture
2. bind texture to framebuffer
3. unbind framebuffer
4. delete texture
5. rebind framebuffer and expect its texture attachment still works.

We would have to rewrite texture data with smart pointers to support
such scenario.

Test: org.skia.skqp.SkQPRunner#UnitTest_GrContext_colorTypeSupportedAsSurface
Bug: 228188395
Change-Id: I0c05da49410080d506767a93983fc8b244f39e99

shared/OpenglCodecCommon/TextureSharedData.h

diff --git a/shared/OpenglCodecCommon/TextureSharedData.h b/shared/OpenglCodecCommon/TextureSharedData.h
index aee1996..fafb435 100644
--- a/shared/OpenglCodecCommon/TextureSharedData.h
+++ b/shared/OpenglCodecCommon/TextureSharedData.h
@@ -18,6 +18,7 @@
 
 #include <GLES/gl.h>
 #include <map>
+#include <memory>
 
 #include "android/base/synchronization/AndroidLock.h"
 
@@ -49,7 +50,7 @@
 };
 
 struct SharedTextureDataMap {
-  using MapType = std::map<GLuint, TextureRec*>;
+  using MapType = std::map<GLuint, std::shared_ptr<TextureRec>>;
 
   using iterator = MapType::iterator;
   using const_iterator = MapType::const_iterator;