Fixed crash issues
[platform/core/uifw/dali-adaptor.git] / adaptors / x11 / ecore-x-render-surface.cpp
index 79ba5da..df81aca 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2014 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2017 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 // INTERNAL INCLUDES
 #include <ecore-x-types.h>
 #include <trigger-event.h>
+#include <gl/egl-implementation.h>
 
 namespace Dali
 {
 
-namespace Internal
-{
-
-namespace Adaptor
-{
-
 #if defined(DEBUG_ENABLED)
 Debug::Filter* gRenderSurfaceLogFilter = Debug::Filter::New(Debug::Verbose, false, "LOG_ECORE_X_RENDER_SURFACE");
 #endif
@@ -49,40 +44,19 @@ Debug::Filter* gRenderSurfaceLogFilter = Debug::Filter::New(Debug::Verbose, fals
 namespace ECore
 {
 
-namespace
-{
-
-const float MINIMUM_DIMENSION_CHANGE = 1.0f; ///< Minimum change for window to be considered to have moved
-static bool gXInitThreadsCalled = false;     ///< global to say whether XInitThreads has been called in this process
-const unsigned int MICROSECONDS_PER_SECOND = 1000000;
-const unsigned int MILLISECONDS_PER_SECOND = 1000;
-
-} // unnamed namespace
-
-RenderSurface::RenderSurface( SurfaceType type,
-                              Dali::PositionSize positionSize,
-                              Any surface,
-                              Any display,
-                              const std::string& name,
-                              bool isTransparent)
-: mMainDisplay(NULL),
-  mType(type),
-  mPosition(positionSize),
+EcoreXRenderSurface::EcoreXRenderSurface(Dali::PositionSize positionSize,
+                                         Any surface,
+                                         const std::string& name,
+                                         bool isTransparent)
+: mPosition(positionSize),
   mTitle(name),
+  mRenderNotification(NULL),
   mColorDepth(isTransparent ? COLOR_DEPTH_32 : COLOR_DEPTH_24),
-  mRenderMode((type == PIXMAP) ? RENDER_SYNC : RENDER_DEFAULT),
-  mRenderNotification( NULL ),
-  mSyncMode(SYNC_MODE_NONE),
-  mSyncReceived( false ),
-  mOwnSurface(false),
-  mOwnDisplay(false),
-  mIsStopped( false )
+  mOwnSurface(false)
 {
-  // see if there is a display in Any display
-  SetDisplay( display );
 }
 
-void RenderSurface::Init( Any surface )
+void EcoreXRenderSurface::Init( Any surface )
 {
   // see if there is a surface in Any surface
   unsigned int surfaceId  = GetSurfaceId( surface );
@@ -90,13 +64,6 @@ void RenderSurface::Init( Any surface )
   // if the surface is empty, create a new one.
   if ( surfaceId == 0 )
   {
-    // make sure XInitThreads is called
-    if ( !gXInitThreadsCalled )
-    {
-      XInitThreads();
-      gXInitThreadsCalled = true;
-    }
-
     // we own the surface about to created
     mOwnSurface = true;
     CreateXRenderable();
@@ -106,191 +73,42 @@ void RenderSurface::Init( Any surface )
     // XLib should already be initialized so no point in calling XInitThreads
     UseExistingRenderable( surfaceId );
   }
-
-#ifdef DEBUG_ENABLED
-  // prints out 'INFO: DALI: new RenderSurface, created display xx, used existing surface xx
-  // we can not use LOG_INFO because the surface can be created before Dali Core is created.
-  printf(  "INFO: DALI: new RenderSurface, %s display %p, %s %s surface %X \n",
-             mOwnDisplay?"created":"used existing",mMainDisplay,
-             mOwnSurface?"created":"used existing",
-             Dali::RenderSurface::PIXMAP==mType?" pixmap" :"window",
-             GetDrawable() );
-#endif
-}
-
-RenderSurface::~RenderSurface()
-{
-  // release the display connection if we use our own
-  if( mOwnDisplay )
-  {
-    if( mMainDisplay )
-    {
-      // NOTE, on 64bit desktop with some NVidia driver versions this crashes
-#ifdef _ARCH_ARM_
-      XCloseDisplay( mMainDisplay );
-#endif
-    }
-  }
 }
 
-Ecore_X_Window RenderSurface::GetXWindow()
+EcoreXRenderSurface::~EcoreXRenderSurface()
 {
-  return 0;
 }
 
-XDisplay* RenderSurface::GetMainDisplay()
-{
-  return mMainDisplay;
-}
-
-void RenderSurface::SetRenderNotification( TriggerEvent* renderNotification )
+void EcoreXRenderSurface::SetRenderNotification(TriggerEventInterface* renderNotification)
 {
   mRenderNotification = renderNotification;
 }
 
-Ecore_X_Drawable RenderSurface::GetDrawable()
+Ecore_X_Window EcoreXRenderSurface::GetXWindow()
 {
   return 0;
 }
 
-Any RenderSurface::GetDisplay()
+Ecore_X_Drawable EcoreXRenderSurface::GetDrawable()
 {
-  // this getter is used by main thread so we need to return the main thread version of the display
-  return Any( ecore_x_display_get() );
+  return 0;
 }
 
-PositionSize RenderSurface::GetPositionSize() const
+PositionSize EcoreXRenderSurface::GetPositionSize() const
 {
   return mPosition;
 }
 
-void RenderSurface::SetRenderMode(RenderMode mode)
-{
-  mRenderMode = mode;
-}
-
-Dali::RenderSurface::RenderMode RenderSurface::GetRenderMode() const
-{
-  return mRenderMode;
-}
-
-void RenderSurface::MoveResize( Dali::PositionSize positionSize )
+void EcoreXRenderSurface::MoveResize( Dali::PositionSize positionSize )
 {
   // nothing to do in base class
 }
 
-void RenderSurface::GetDpi( unsigned int& dpiHorizontal, unsigned int& dpiVertical ) const
+void EcoreXRenderSurface::SetViewMode( ViewMode viewMode )
 {
-  // calculate DPI
-  float xres, yres;
-
-  // 1 inch = 25.4 millimeters
-  xres = ecore_x_dpi_get();
-  yres = ecore_x_dpi_get();
-
-  dpiHorizontal = int(xres + 0.5f);  // rounding
-  dpiVertical   = int(yres + 0.5f);
-}
-
-void RenderSurface::Map()
-{
-}
-
-void RenderSurface::TransferDisplayOwner( Internal::Adaptor::RenderSurface& newSurface )
-{
-  // if we don't own the display return
-  if( mOwnDisplay == false )
-  {
-    return;
-  }
-
-  RenderSurface* other = dynamic_cast< RenderSurface* >( &newSurface );
-  if( other )
-  {
-    // if both surfaces share the same display, and this surface owns it,
-    // then transfer the ownership to the new surface
-    if( other->mMainDisplay == mMainDisplay )
-    {
-      mOwnDisplay = false;
-      other->mOwnDisplay = true;
-    }
-  }
 }
 
-void RenderSurface::ConsumeEvents()
-{
-  // if the render surface has own display, check events so that we can flush the queue and avoid
-  // any potential memory leaks in X
-  if( mOwnDisplay )
-  {
-    // looping if events remain
-    int events( 0 );
-    do
-    {
-      // Check if there are any events in the queue
-      events = XEventsQueued( mMainDisplay, QueuedAfterFlush );
-
-      if ( events > 0 )
-      {
-        // Just flush event to prevent memory leak from event queue as the events get built up in
-        // memory but are only deleted when we retrieve them
-        XEvent ev;
-        XNextEvent( mMainDisplay, &ev );
-      }
-    }
-    while( events > 0 );
-  }
-}
-
-void RenderSurface::StopRender()
-{
-  // Stop blocking waiting for sync
-  SetSyncMode( RenderSurface::SYNC_MODE_NONE );
-  // Simulate a RenderSync in case render-thread is currently blocked
-  RenderSync();
-
-  mIsStopped = true;
-}
-
-void RenderSurface::SetViewMode( ViewMode )
-{
-}
-
-void RenderSurface::SetDisplay( Any display )
-{
-  // the render surface can be passed either EFL e-core types, or x11 types
-  // we use boost any to determine at run time which type
-
-  if ( display.Empty() == false )
-  {
-    // check we have a valid type
-    DALI_ASSERT_ALWAYS( ( ( display.GetType() == typeid (Ecore_X_Display *)) ||
-                          ( display.GetType() == typeid (XDisplay *) ) )
-                        &&
-                        "Display type is invalid" );
-
-    mOwnDisplay = false;
-
-    // display may point to EcoreXDisplay so may need to cast
-    if( display.GetType() == typeid (Ecore_X_Display*) )
-    {
-      mMainDisplay = static_cast< XDisplay* >( AnyCast< Ecore_X_Display* >( display ) );
-    }
-    else
-    {
-      mMainDisplay = AnyCast< XDisplay* >( display );
-    }
-  }
-  else
-  {
-    mOwnDisplay = true;
-    // mMainDisplay = (XDisplay*)ecore_x_display_get();
-    // Because of DDK issue, we need to use separated x display instead of ecore default display
-    mMainDisplay = XOpenDisplay(0);
-  }
-}
-
-unsigned int RenderSurface::GetSurfaceId( Any surface ) const
+unsigned int EcoreXRenderSurface::GetSurfaceId( Any surface ) const
 {
   unsigned int surfaceId = 0;
 
@@ -313,42 +131,11 @@ unsigned int RenderSurface::GetSurfaceId( Any surface ) const
   return surfaceId;
 }
 
-void RenderSurface::RenderSync()
+RenderSurface::Type EcoreXRenderSurface::GetSurfaceType()
 {
-  // nothing to do
-}
-
-void RenderSurface::DoRenderSync( unsigned int timeDelta, SyncMode syncMode )
-{
-  // Should block waiting for RenderSync?
-  if( mRenderMode == Dali::RenderSurface::RENDER_SYNC )
-  {
-    boost::unique_lock< boost::mutex > lock( mSyncMutex );
-
-    // wait for sync
-    if( syncMode != SYNC_MODE_NONE &&
-        mSyncMode != SYNC_MODE_NONE &&
-        !mSyncReceived )
-    {
-      mSyncNotify.wait( lock );
-    }
-    mSyncReceived = false;
-  }
-  // Software sync based on a timed delay?
-  else if( mRenderMode > Dali::RenderSurface::RENDER_SYNC )
-  {
-    unsigned int syncPeriod( MICROSECONDS_PER_SECOND / static_cast< unsigned int >( mRenderMode ) - MILLISECONDS_PER_SECOND );
-    if( timeDelta < syncPeriod )
-    {
-      usleep( syncPeriod - timeDelta );
-    }
-  }
+  return RenderSurface::ECORE_RENDER_SURFACE;
 }
 
 } // namespace ECore
 
-} // namespace Adaptor
-
-} // namespace Internal
-
 } // namespace Dali