Revert "[Tizen] (Vector) Fix event trigger issue"
authorWonsik Jung <sidein@samsung.com>
Mon, 10 Aug 2020 05:45:38 +0000 (14:45 +0900)
committerWonsik Jung <sidein@samsung.com>
Mon, 10 Aug 2020 05:45:38 +0000 (14:45 +0900)
This reverts commit e47ae36b9102a61eb1ff71b45eb3231294e2ec46.

dali-extension/vector-animation-renderer/tizen-vector-animation-manager.cpp

index 13deede..97668c8 100755 (executable)
@@ -56,16 +56,12 @@ void TizenVectorAnimationManager::AddEventHandler( TizenVectorAnimationEventHand
       Adaptor::Get().RegisterProcessor( *this );
     }
 
-    mEventHandlers.push_back( &handler );
-
+    if( !mEventTrigger )
     {
-      Dali::Mutex::ScopedLock lock( mMutex );
-
-      if( !mEventTrigger )
-      {
-        mEventTrigger = std::unique_ptr< EventThreadCallback >( new EventThreadCallback( MakeCallback( this, &TizenVectorAnimationManager::OnEventTriggered ) ) );
-      }
+      mEventTrigger = std::unique_ptr< EventThreadCallback >( new EventThreadCallback( MakeCallback( this, &TizenVectorAnimationManager::OnEventTriggered ) ) );
     }
+
+    mEventHandlers.push_back( &handler );
   }
 }
 
@@ -75,18 +71,16 @@ void TizenVectorAnimationManager::RemoveEventHandler( TizenVectorAnimationEventH
   if( iter != mEventHandlers.end() )
   {
     mEventHandlers.erase( iter );
-  }
 
-  bool releaseEventTrigger = false;
-
-  if( mEventHandlers.empty() )
-  {
-    if( Adaptor::IsAvailable() )
+    if( mEventHandlers.empty() )
     {
-      Adaptor::Get().UnregisterProcessor( *this );
-    }
+      if( Adaptor::IsAvailable() )
+      {
+        Adaptor::Get().UnregisterProcessor( *this );
+      }
 
-    releaseEventTrigger = true;
+      mEventTrigger.release();
+    }
   }
 
   {
@@ -97,11 +91,6 @@ void TizenVectorAnimationManager::RemoveEventHandler( TizenVectorAnimationEventH
     {
       mTriggeredHandlers.erase( triggeredHandler );
     }
-
-    if( releaseEventTrigger )
-    {
-      mEventTrigger.reset();
-    }
   }
 }
 
@@ -113,10 +102,7 @@ void TizenVectorAnimationManager::TriggerEvent( TizenVectorAnimationEventHandler
   {
     mTriggeredHandlers.push_back( &handler );
 
-    if( mEventTrigger )
-    {
-      mEventTrigger->Trigger();
-    }
+    mEventTrigger->Trigger();
   }
 }