From 87ac7a052ebf2b1da03b5b5d49e0ebde57851751 Mon Sep 17 00:00:00 2001 From: Daekwang Ryu Date: Tue, 6 Oct 2020 13:03:21 +0900 Subject: [PATCH] Revert "[Tizen] There is no need to call mOverlayRenderer's AddRenderer() in play()." This reverts commit 36e0fc16439e369be279d1fc8625cf35fa8cf374. --- dali-toolkit/internal/controls/video-view/video-view-impl.cpp | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) mode change 100755 => 100644 dali-toolkit/internal/controls/video-view/video-view-impl.cpp diff --git a/dali-toolkit/internal/controls/video-view/video-view-impl.cpp b/dali-toolkit/internal/controls/video-view/video-view-impl.cpp old mode 100755 new mode 100644 index d7d812c..523fcf0 --- a/dali-toolkit/internal/controls/video-view/video-view-impl.cpp +++ b/dali-toolkit/internal/controls/video-view/video-view-impl.cpp @@ -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 ) { -- 2.7.4