From 93f1c333b9d1bda380f31f49bab6db30e6a9f35c Mon Sep 17 00:00:00 2001 From: Joogab Yun Date: Tue, 29 Sep 2020 15:41:10 +0900 Subject: [PATCH 1/1] There is no need to call mOverlayRenderer's AddRenderer() in play(). Change-Id: I37c2709e45d9e5a97f5801258e6ebee377e1337a --- dali-toolkit/internal/controls/video-view/video-view-impl.cpp | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) mode change 100644 => 100755 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 100644 new mode 100755 index 523fcf0..d7d812c --- a/dali-toolkit/internal/controls/video-view/video-view-impl.cpp +++ b/dali-toolkit/internal/controls/video-view/video-view-impl.cpp @@ -228,11 +228,6 @@ bool VideoView::IsLooping() void VideoView::Play() { - if( mOverlayRenderer ) - { - Self().AddRenderer( mOverlayRenderer ); - } - mVideoPlayer.Play(); mIsPlay = true; } @@ -295,11 +290,6 @@ Dali::Toolkit::VideoView::VideoViewSignalType& VideoView::FinishedSignal() void VideoView::EmitSignalFinish() { - if( mOverlayRenderer ) - { - Self().RemoveRenderer( mOverlayRenderer ); - } - if ( !mFinishedSignal.Empty() ) { Dali::Toolkit::VideoView handle( GetOwner() ); @@ -660,6 +650,7 @@ void VideoView::SetWindowSurfaceTarget() mOverlayRenderer = Renderer::New( geometry, shader ); mOverlayRenderer.SetProperty( Renderer::Property::BLEND_MODE, BlendMode::OFF ); } + Self().AddRenderer( mOverlayRenderer ); if( mIsPlay ) { -- 2.7.4