From f5bde6fc6d2b89947dc1ffdf86ed7dc2633d4ee0 Mon Sep 17 00:00:00 2001 From: Seoyeon Kim Date: Mon, 10 Jun 2019 14:33:07 +0900 Subject: [PATCH] [Tizen]Fix video underlay issue This reverts commit a2db4045f5ca2918bf4af60a5156c5d977c2faf7. Change-Id: Iad13a845d51baef20b57bdf7996bdb096c2f1f62 --- dali/internal/graphics/gles/egl-implementation.cpp | 18 ++++-------------- 1 file changed, 4 insertions(+), 14 deletions(-) diff --git a/dali/internal/graphics/gles/egl-implementation.cpp b/dali/internal/graphics/gles/egl-implementation.cpp index 2172e37..2ac0a63 100755 --- a/dali/internal/graphics/gles/egl-implementation.cpp +++ b/dali/internal/graphics/gles/egl-implementation.cpp @@ -326,8 +326,6 @@ bool EglImplementation::ChooseConfig( bool isWindowType, ColorDepth depth ) return true; } - bool isTransparent = ( depth == COLOR_DEPTH_32 ); - mColorDepth = depth; mIsWindow = isWindowType; @@ -372,18 +370,10 @@ bool EglImplementation::ChooseConfig( bool isWindowType, ColorDepth depth ) configAttribs.PushBack( EGL_BLUE_SIZE ); configAttribs.PushBack( 8 ); - if ( isTransparent ) - { - configAttribs.PushBack( EGL_ALPHA_SIZE ); -#ifdef _ARCH_ARM_ - // For underlay video playback, we also need to set the alpha value of the 24/32bit window. - configAttribs.PushBack( 8 ); -#else - // There is a bug in the desktop emulator - // setting EGL_ALPHA_SIZE to 8 results in eglChooseConfig failing - configAttribs.PushBack( 8 ); -#endif // _ARCH_ARM_ - } + // In the previous code, there was a branch for ARM. + // If there is an issue in only ARM, we need to check here again. + configAttribs.PushBack( EGL_ALPHA_SIZE ); + configAttribs.PushBack( 8 ); configAttribs.PushBack( EGL_DEPTH_SIZE ); configAttribs.PushBack( mDepthBufferRequired ? 24 : 0 ); -- 2.7.4