[3.0] Remove unused mEglImageKHRContainer from NativeImageSource and add error check 89/116889/1
authorHeeyong Song <heeyong.song@samsung.com>
Tue, 21 Feb 2017 06:02:27 +0000 (15:02 +0900)
committerHeeyong Song <heeyong.song@samsung.com>
Thu, 2 Mar 2017 00:42:28 +0000 (09:42 +0900)
Change-Id: I68de30cdf8630fcf5cafa8cc216dfbb0ecbc613f

adaptors/tizen/native-image-source-impl-tizen.cpp
adaptors/tizen/native-image-source-impl.h

index fe249e5..591d8b5 100755 (executable)
@@ -330,14 +330,6 @@ void NativeImageSource::SetSource( Any source )
     mWidth = tbm_surface_get_width( mTbmSurface );
     mHeight = tbm_surface_get_height( mTbmSurface );
   }
-
-  if( mEglImageKHRContainer.Size() > 2 )
-  {
-    mEglImageExtensions->DestroyImageKHR(mEglImageKHRContainer[0]);
-    mEglImageKHRContainer.Erase( mEglImageKHRContainer.Begin() );
-  }
-
-  mEglImageKHRContainer.PushBack( mEglImageKHR );
 }
 
 bool NativeImageSource::IsColorDepthSupported( Dali::NativeImageSource::ColorDepth colorDepth )
@@ -396,6 +388,10 @@ bool NativeImageSource::GlExtensionCreate()
   // casting from an unsigned int to a void *, which should then be cast back
   // to an unsigned int in the driver.
   EGLClientBuffer eglBuffer = reinterpret_cast< EGLClientBuffer > (mTbmSurface);
+  if( !eglBuffer )
+  {
+    return false;
+  }
 
   mEglImageKHR = mEglImageExtensions->CreateImageKHR( eglBuffer );
 
@@ -404,9 +400,12 @@ bool NativeImageSource::GlExtensionCreate()
 
 void NativeImageSource::GlExtensionDestroy()
 {
-  mEglImageExtensions->DestroyImageKHR(mEglImageKHR);
+  if( mEglImageKHR )
+  {
+    mEglImageExtensions->DestroyImageKHR(mEglImageKHR);
 
-  mEglImageKHR = NULL;
+    mEglImageKHR = NULL;
+  }
 }
 
 unsigned int NativeImageSource::TargetTexture()
index 9bed6e7..505d8b2 100755 (executable)
@@ -34,12 +34,8 @@ namespace Internal
 
 namespace Adaptor
 {
-class EglImageExtensions;
 
-namespace
-{
-  typedef Dali::Vector< void* > EglImageContainer;
-};
+class EglImageExtensions;
 
 /**
  * Dali internal NativeImageSource.
@@ -193,7 +189,6 @@ private:
   void* mEglImageKHR;                         ///< From EGL extension
   EglImageExtensions* mEglImageExtensions;    ///< The EGL Image Extensions
   bool mSetSource;
-  EglImageContainer mEglImageKHRContainer;
 };
 
 } // namespace Adaptor