Optimize render task completed notify signal

Let we follow up RenderTask also optimized like Animation.

The relationship between RenderTask <-> RenderTaskList is euqal with
Animation <-> AnimationPlayList. So we can use same logic.

Change-Id: I61072374252394e01fe542d8b02941c5b4e58476
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

diff --git a/dali/internal/event/render-tasks/render-task-list-impl.cpp b/dali/internal/event/render-tasks/render-task-list-impl.cpp
index 34912a08f86c7744419b02e84af5bfef3a9f710e..fed77c6cf293b870cf593ff15bad86ecfdf4cc51 100644 (file)
--- a/dali/internal/event/render-tasks/render-task-list-impl.cpp
+++ b/dali/internal/event/render-tasks/render-task-list-impl.cpp
@@ -216,22 +216,24 @@ void RenderTaskList::Initialize()
   mSceneObject->SetCompleteNotificationInterface(this);
 }
 
-void RenderTaskList::NotifyCompleted(CompleteNotificationInterface::ParameterList notifierList)
+void RenderTaskList::NotifyCompleted(CompleteNotificationInterface::ParameterList notifierIdList)
 {
   DALI_LOG_TRACE_METHOD(gLogRenderList);
 
   RenderTaskContainer finishedRenderTasks;
 
-  // TODO : Optimize here if required.
-  // Note : Actually, Total number of RenderTask should be small enough so full search might not overhead for now.
-
-  // Since render tasks can be unreferenced during the signal emissions, iterators into render tasks pointers may be invalidated.
-  // First copy the finished render tasks, then emit signals
-  for(RenderTaskContainer::iterator iter = mTasks.begin(), endIt = mTasks.end(); iter != endIt; ++iter)
+  for(const auto& notifierId : notifierIdList)
   {
-    if((*iter)->HasFinished())
+    auto* renderTask = GetEventObject(notifierId);
+    if(DALI_LIKELY(renderTask))
     {
-      finishedRenderTasks.push_back(*iter);
+      // Check if this render task hold inputed scenegraph render task.
+      DALI_ASSERT_DEBUG(renderTask->GetRenderTaskSceneObject()->GetNotifyId() == notifierId);
+
+      if(renderTask->HasFinished())
+      {
+        finishedRenderTasks.push_back(renderTask);
+      }
     }
   }