Revert "[Tizen] Fix video underlay issue"
authorSunghyun kim <scholb.kim@samsung.com>
Wed, 24 Jul 2019 04:38:32 +0000 (13:38 +0900)
committerSunghyun kim <scholb.kim@samsung.com>
Wed, 24 Jul 2019 04:38:42 +0000 (13:38 +0900)
This reverts commit 327627defee6d6f7d0a631b4bf472a158bfe5756.

Change-Id: I1710b443657253d720943dd9dd1669935e6e0af1

dali/internal/graphics/gles/egl-implementation.cpp

index b8a42f7..68d7925 100755 (executable)
@@ -341,6 +341,8 @@ bool EglImplementation::ChooseConfig( bool isWindowType, ColorDepth depth )
     return true;
   }
 
+  bool isTransparent = ( depth == COLOR_DEPTH_32 );
+
   mColorDepth = depth;
   mIsWindow = isWindowType;
 
@@ -381,10 +383,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 );