(Vector) Call Finalize method instead of deleting VectorRenderer

Change-Id: Id7d0a5cab4229e6619122c6294914d512bae9b17

diff --git a/automated-tests/src/dali-toolkit/dali-toolkit-test-utils/toolkit-vector-animation-renderer.cpp b/automated-tests/src/dali-toolkit/dali-toolkit-test-utils/toolkit-vector-animation-renderer.cpp
index f8c0c36..b039c57 100755 (executable)
--- a/automated-tests/src/dali-toolkit/dali-toolkit-test-utils/toolkit-vector-animation-renderer.cpp
+++ b/automated-tests/src/dali-toolkit/dali-toolkit-test-utils/toolkit-vector-animation-renderer.cpp
@@ -108,10 +108,6 @@ public:
     height = 100;
   }
 
     height = 100;
   }
 

-  void GetLayerInfo( Property::Map& map ) const
-  {
-  }
-

   Dali::VectorAnimationRenderer::UploadCompletedSignalType& UploadCompletedSignal()
   {
     return mUploadCompletedSignal;
   Dali::VectorAnimationRenderer::UploadCompletedSignalType& UploadCompletedSignal()
   {
     return mUploadCompletedSignal;


@@ -186,6 +182,10 @@ VectorAnimationRenderer& VectorAnimationRenderer::operator=( const VectorAnimati
   return *this;
 }
 
   return *this;
 }
 

+void VectorAnimationRenderer::Finalize()
+{
+}
+

 void VectorAnimationRenderer::SetRenderer( Renderer renderer )
 {
   Internal::Adaptor::GetImplementation( *this ).SetRenderer( renderer );
 void VectorAnimationRenderer::SetRenderer( Renderer renderer )
 {
   Internal::Adaptor::GetImplementation( *this ).SetRenderer( renderer );


@@ -218,7 +218,6 @@ void VectorAnimationRenderer::GetDefaultSize( uint32_t& width, uint32_t& height
 
 void VectorAnimationRenderer::GetLayerInfo( Property::Map& map ) const
 {
 
 void VectorAnimationRenderer::GetLayerInfo( Property::Map& map ) const
 {

-  Internal::Adaptor::GetImplementation( *this ).GetLayerInfo( map );

 }
 
 VectorAnimationRenderer::UploadCompletedSignalType& VectorAnimationRenderer::UploadCompletedSignal()
 }
 
 VectorAnimationRenderer::UploadCompletedSignalType& VectorAnimationRenderer::UploadCompletedSignal()

diff --git a/dali-toolkit/internal/visuals/animated-vector-image/vector-animation-task.cpp b/dali-toolkit/internal/visuals/animated-vector-image/vector-animation-task.cpp
index ac89867..8258fb7 100644 (file)
--- a/dali-toolkit/internal/visuals/animated-vector-image/vector-animation-task.cpp
+++ b/dali-toolkit/internal/visuals/animated-vector-image/vector-animation-task.cpp
@@ -100,7 +100,7 @@ void VectorAnimationTask::Finalize()
     mAnimationFinishedTrigger.reset();
   }
 
     mAnimationFinishedTrigger.reset();
   }
 

-  mVectorRenderer.Reset();
+  mVectorRenderer.Finalize();

 }
 
 void VectorAnimationTask::SetRenderer( Renderer renderer )
 }
 
 void VectorAnimationTask::SetRenderer( Renderer renderer )

diff --git a/dali-toolkit/internal/visuals/animated-vector-image/vector-rasterize-thread.cpp b/dali-toolkit/internal/visuals/animated-vector-image/vector-rasterize-thread.cpp
index cb2d08a..08cad31 100644 (file)
--- a/dali-toolkit/internal/visuals/animated-vector-image/vector-rasterize-thread.cpp
+++ b/dali-toolkit/internal/visuals/animated-vector-image/vector-rasterize-thread.cpp
@@ -76,15 +76,15 @@ void VectorRasterizeThread::SetCompletedCallback( CallbackBase* callback )
 
 void VectorRasterizeThread::AddTask( VectorAnimationTaskPtr task )
 {
 
 void VectorRasterizeThread::AddTask( VectorAnimationTaskPtr task )
 {

+  // Lock while adding task to the queue
+  ConditionalWait::ScopedLock lock( mConditionalWait );
+

   if( !mIsThreadStarted )
   {
     Start();
     mIsThreadStarted = true;
   }
 
   if( !mIsThreadStarted )
   {
     Start();
     mIsThreadStarted = true;
   }
 

-  // Lock while adding task to the queue
-  ConditionalWait::ScopedLock lock( mConditionalWait );
-

   if( mRasterizeTasks.end() == std::find( mRasterizeTasks.begin(), mRasterizeTasks.end(), task ) )
   {
     mRasterizeTasks.push_back( task );
   if( mRasterizeTasks.end() == std::find( mRasterizeTasks.begin(), mRasterizeTasks.end(), task ) )
   {
     mRasterizeTasks.push_back( task );