From d145f75cd16839fdc31e9356ae8a035470a5950f Mon Sep 17 00:00:00 2001
From: Taehyub Kim <taehyub.kim@samsung.com>
Date: Tue, 3 May 2022 17:14:33 +0900
Subject: [PATCH] [Tizen] Do not call gl functions during shutdown

This reverts commit 4163085fcc0f47080ec1e18c924a86c92191c659.
---
 .../graphics/gles-impl/egl-graphics-controller.h   | 63 ++++++++++++++++------
 1 file changed, 48 insertions(+), 15 deletions(-)

diff --git a/dali/internal/graphics/gles-impl/egl-graphics-controller.h b/dali/internal/graphics/gles-impl/egl-graphics-controller.h
index 6af7383..ec6c894 100644
--- a/dali/internal/graphics/gles-impl/egl-graphics-controller.h
+++ b/dali/internal/graphics/gles-impl/egl-graphics-controller.h
@@ -136,6 +136,16 @@ public:
     // Final flush
     Flush();
 
+    ClearTextureUpdateQueue();
+
+    // Remove all create queue and command queue.
+    // Note that all memory are already deallocated at Final flush.
+    mCreateTextureQueue              = {};
+    mCreateBufferQueue               = {};
+    mCreateFramebufferQueue          = {};
+    mTextureMipmapGenerationRequests = {};
+    mCommandQueue                    = {};
+
     if(mContext)
     {
       mContext->GlContextDestroyed();
@@ -464,6 +474,26 @@ public:
   }
 
   /**
+   * @brief Clears the texture update queue
+   */
+  void ClearTextureUpdateQueue()
+  {
+    // Remove remained CPU-allocated texture memory
+    while(!mTextureUpdateRequests.empty())
+    {
+      auto& request = mTextureUpdateRequests.front();
+      auto& source  = request.second;
+
+      if(source.sourceType == Graphics::TextureUpdateSourceInfo::Type::MEMORY)
+      {
+        // free staging memory
+        free(source.memorySource.memory);
+      }
+      mTextureUpdateRequests.pop();
+    }
+  }
+
+  /**
    * @brief Flushes all pending updates
    *
    * Function flushes all pending resource constructions,
@@ -471,26 +501,29 @@ public:
    */
   void Flush()
   {
-    if(!mCreateTextureQueue.empty() ||
-       !mCreateBufferQueue.empty() ||
-       !mCreateFramebufferQueue.empty() ||
-       !mTextureUpdateRequests.empty() ||
-       !mTextureMipmapGenerationRequests.empty())
+    if(DALI_LIKELY(!mIsShuttingDown))
     {
-      mGraphics->ActivateResourceContext();
-    }
+      if(!mCreateTextureQueue.empty() ||
+         !mCreateBufferQueue.empty() ||
+         !mCreateFramebufferQueue.empty() ||
+         !mTextureUpdateRequests.empty() ||
+         !mTextureMipmapGenerationRequests.empty())
+      {
+        mGraphics->ActivateResourceContext();
+      }
 
-    // Process creations
-    ProcessCreateQueues();
+      // Process creations
+      ProcessCreateQueues();
 
-    // Process updates
-    ProcessTextureUpdateQueue();
+      // Process updates
+      ProcessTextureUpdateQueue();
 
-    // Process texture mipmap generation requests
-    ProcessTextureMipmapGenerationQueue();
+      // Process texture mipmap generation requests
+      ProcessTextureMipmapGenerationQueue();
 
-    // Process main command queue
-    ProcessCommandQueues();
+      // Process main command queue
+      ProcessCommandQueues();
+    }
 
     // Reset texture cache in the contexts while destroying textures
     ResetTextureCache();
-- 
2.7.4