From: Seoyeon Kim <seoyeon2.kim@samsung.com>
Date: Mon, 10 Jun 2019 05:30:03 +0000 (+0900)
Subject: Revert "[Tizen]Fix video underlay issue"
X-Git-Tag: accepted/tizen/unified/20190611.050103~4
X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fuifw%2Fdali-adaptor.git;a=commitdiff_plain;h=a2db4045f5ca2918bf4af60a5156c5d977c2faf7

Revert "[Tizen]Fix video underlay issue"

This reverts commit e4cba0ad213ac711925e7d3bf8720cc61a08cecd.

Change-Id: Ie16c66025003a1d678ef3e49533a630fb38757d0
---

diff --git a/dali/internal/graphics/gles/egl-implementation.cpp b/dali/internal/graphics/gles/egl-implementation.cpp
index 2ac0a63..2172e37 100755
--- a/dali/internal/graphics/gles/egl-implementation.cpp
+++ b/dali/internal/graphics/gles/egl-implementation.cpp
@@ -326,6 +326,8 @@ bool EglImplementation::ChooseConfig( bool isWindowType, ColorDepth depth )
     return true;
   }
 
+  bool isTransparent = ( depth == COLOR_DEPTH_32 );
+
   mColorDepth = depth;
   mIsWindow = isWindowType;
 
@@ -370,10 +372,18 @@ bool EglImplementation::ChooseConfig( bool isWindowType, ColorDepth depth )
   configAttribs.PushBack( EGL_BLUE_SIZE );
   configAttribs.PushBack( 8 );
 
-  // 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 );
+  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_
+  }
 
   configAttribs.PushBack( EGL_DEPTH_SIZE );
   configAttribs.PushBack( mDepthBufferRequired ? 24 : 0 );