Revert "[Tizen] There is no need to call mOverlayRenderer's AddRenderer() in play()."
authorDaekwang Ryu <dkdk.ryu@samsung.com>
Tue, 6 Oct 2020 04:03:21 +0000 (13:03 +0900)
committerDaekwang Ryu <dkdk.ryu@samsung.com>
Tue, 6 Oct 2020 04:03:21 +0000 (13:03 +0900)
This reverts commit 36e0fc16439e369be279d1fc8625cf35fa8cf374.

dali-toolkit/internal/controls/video-view/video-view-impl.cpp [changed mode: 0755->0644]

old mode 100755 (executable)
new mode 100644 (file)
index d7d812c..523fcf0
@@ -228,6 +228,11 @@ bool VideoView::IsLooping()
 
 void VideoView::Play()
 {
+  if( mOverlayRenderer )
+  {
+    Self().AddRenderer( mOverlayRenderer );
+  }
+
   mVideoPlayer.Play();
   mIsPlay = true;
 }
@@ -290,6 +295,11 @@ Dali::Toolkit::VideoView::VideoViewSignalType& VideoView::FinishedSignal()
 
 void VideoView::EmitSignalFinish()
 {
+  if( mOverlayRenderer )
+  {
+    Self().RemoveRenderer( mOverlayRenderer );
+  }
+
   if ( !mFinishedSignal.Empty() )
   {
     Dali::Toolkit::VideoView handle( GetOwner() );
@@ -650,7 +660,6 @@ void VideoView::SetWindowSurfaceTarget()
     mOverlayRenderer = Renderer::New( geometry, shader );
     mOverlayRenderer.SetProperty( Renderer::Property::BLEND_MODE, BlendMode::OFF );
   }
-  Self().AddRenderer( mOverlayRenderer );
 
   if( mIsPlay )
   {