Remove recursive mutex

To avoid reentrantlock, add a new method without using lock code.

shell/platform/tizen/external_texture_gl.cc

@@ -120,20 +120,23 @@ bool ExternalTextureGL::PopulateTextureWithIdentifier(
   return true;
 }
 
-void ExternalTextureGL::DestructionTbmSurface() {
+void ExternalTextureGL::DestructionTbmSurfaceWithLock() {
   mutex_.lock();
+  DestructionTbmSurface();
+  mutex_.unlock();
+}
+
+void ExternalTextureGL::DestructionTbmSurface() {
   if (!texture_tbm_surface_) {
     LoggerE("tbm_surface_h is NULL");
-    mutex_.unlock();
     return;
   }
   tbm_surface_internal_unref(texture_tbm_surface_);
   texture_tbm_surface_ = NULL;
-  mutex_.unlock();
 }
 
 void ExternalTextureGL::destructionCallback(void* user_data) {
   ExternalTextureGL* externalTextureGL =
       reinterpret_cast<ExternalTextureGL*>(user_data);
-  externalTextureGL->DestructionTbmSurface();
+  externalTextureGL->DestructionTbmSurfaceWithLock();
 }

shell/platform/tizen/external_texture_gl.h

@@ -43,10 +43,11 @@ class ExternalTextureGL {
                                      FlutterOpenGLTexture* opengl_texture);
   bool OnFrameAvailable(tbm_surface_h tbm_surface);
   void DestructionTbmSurface();
+  void DestructionTbmSurfaceWithLock();
 
  private:
   std::unique_ptr<ExternalTextureGLState> state_;
-  std::recursive_mutex mutex_;
+  std::mutex mutex_;
   tbm_surface_h texture_tbm_surface_;
   static void destructionCallback(void* user_data);
   const long texture_id_;