[4.0] Avoid to call SurfaceResize functions

Sometimes, screen rotation events occurs with same value.
This patch is to avoid unnecessary works.

Change-Id: Ic575b174c01585efc4dfabccd320aeaf665c62b1

diff --git a/adaptors/ecore/wayland/window-impl-ecore-wl.cpp b/adaptors/ecore/wayland/window-impl-ecore-wl.cpp
index 3eb89ee..162438f 100755 (executable)
--- a/adaptors/ecore/wayland/window-impl-ecore-wl.cpp
+++ b/adaptors/ecore/wayland/window-impl-ecore-wl.cpp
@@ -78,6 +78,7 @@ struct Window::EventHandler
     mEventQueue( NULL ),
     mTizenPolicy( NULL ),
     mTizenDisplayPolicy( NULL ),
+    mScreenRotationAngle( 0 ),
     mNotificationLevel( -1 ),
     mNotificationChangeState( 0 ),
     mNotificationLevelChangeDone( true ),
@@ -225,24 +226,31 @@ struct Window::EventHandler
       ECore::WindowRenderSurface* wlSurface( dynamic_cast< ECore::WindowRenderSurface * >( handler->mWindow->mSurface ) );
       if( wlSurface )
       {
-        wlSurface->OutputTransformed();
+        int transform = ecore_wl_output_transform_get( ecore_wl_window_output_find( handler->mEcoreWindow ) );
+        int screenRotationAngle = transform * 90;
 
-        bool forceUpdate = false;
-        int orientation = wlSurface->GetOrientation();
-
-        // If PreRotation is supported, don't work client rotation by core.
-        // Client rotation means both window resize and camera rotation.
-        if( wlSurface->IsPreRotationSupported() )
+        if( handler->mScreenRotationAngle != screenRotationAngle)
         {
-          forceUpdate = true;
-          orientation = 0;
-        }
+          handler->mScreenRotationAngle = screenRotationAngle;
+
+          wlSurface->OutputTransformed();
+
+          bool forceUpdate = false;
+          int orientation = wlSurface->GetOrientation();
 
-        PositionSize positionSize = wlSurface->GetPositionSize();
-        handler->mWindow->mAdaptor->SurfaceResizePrepare( Adaptor::SurfaceSize( positionSize.width, positionSize.height ), orientation, forceUpdate );
-        handler->mWindow->mAdaptor->SurfaceResizeComplete( Adaptor::SurfaceSize( positionSize.width, positionSize.height ) );
+          // If PreRotation is supported, don't work client rotation by core.
+          // Client rotation means both window resize and camera rotation.
+          if( wlSurface->IsPreRotationSupported() )
+          {
+            forceUpdate = true;
+            orientation = 0;
+          }
 
-        DALI_LOG_INFO( gWindowLogFilter, Debug::General, "Window (%p) EcoreEventOutputTransform [%d, %d]\n", handler->mEcoreWindow, positionSize.width, positionSize.height );
+          PositionSize positionSize = wlSurface->GetPositionSize();
+          handler->mWindow->mAdaptor->SurfaceResizePrepare( Adaptor::SurfaceSize( positionSize.width, positionSize.height ), orientation, forceUpdate );
+          handler->mWindow->mAdaptor->SurfaceResizeComplete( Adaptor::SurfaceSize( positionSize.width, positionSize.height ) );
+          DALI_LOG_INFO( gWindowLogFilter, Debug::General, "Window (%p) EcoreEventOutputTransform [%d, %d]\n", handler->mEcoreWindow, positionSize.width, positionSize.height );
+        }
       }
     }
 
@@ -260,24 +268,31 @@ struct Window::EventHandler
       ECore::WindowRenderSurface* wlSurface( dynamic_cast< ECore::WindowRenderSurface * >( handler->mWindow->mSurface ) );
       if( wlSurface )
       {
-        wlSurface->OutputTransformed();
-
-        bool forceUpdate = false;
-        int orientation = wlSurface->GetOrientation();
+        int transform = ecore_wl_output_transform_get( ecore_wl_window_output_find( handler->mEcoreWindow ) );
+        int screenRotationAngle = transform * 90;
 
-        // If PreRotation is supported, don't work client rotation by core.
-        // Client rotation means both window resize and camera rotation.
-        if( wlSurface->IsPreRotationSupported() )
+        if( handler->mScreenRotationAngle != screenRotationAngle)
         {
-          forceUpdate = true;
-          orientation = 0;
-        }
+          handler->mScreenRotationAngle = screenRotationAngle;
+          wlSurface->OutputTransformed();
+
+          bool forceUpdate = false;
+          int orientation = wlSurface->GetOrientation();
 
-        PositionSize positionSize = wlSurface->GetPositionSize();
-        handler->mWindow->mAdaptor->SurfaceResizePrepare( Adaptor::SurfaceSize( positionSize.width, positionSize.height ), orientation, forceUpdate );
-        handler->mWindow->mAdaptor->SurfaceResizeComplete( Adaptor::SurfaceSize( positionSize.width, positionSize.height ) );
+          // If PreRotation is supported, don't work client rotation by core.
+          // Client rotation means both window resize and camera rotation.
+          if( wlSurface->IsPreRotationSupported() )
+          {
+            forceUpdate = true;
+            orientation = 0;
+          }
 
-        DALI_LOG_INFO( gWindowLogFilter, Debug::General, "Window (%p) EcoreEventIgnoreOutputTransform [%d, %d]\n", handler->mEcoreWindow, positionSize.width, positionSize.height );
+          PositionSize positionSize = wlSurface->GetPositionSize();
+          handler->mWindow->mAdaptor->SurfaceResizePrepare( Adaptor::SurfaceSize( positionSize.width, positionSize.height ), orientation, forceUpdate );
+          handler->mWindow->mAdaptor->SurfaceResizeComplete( Adaptor::SurfaceSize( positionSize.width, positionSize.height ) );
+
+          DALI_LOG_INFO( gWindowLogFilter, Debug::General, "Window (%p) EcoreEventIgnoreOutputTransform [%d, %d]\n", handler->mEcoreWindow, positionSize.width, positionSize.height );
+        }
       }
     }
 
@@ -425,6 +440,8 @@ struct Window::EventHandler
   tizen_policy* mTizenPolicy;
   tizen_display_policy* mTizenDisplayPolicy;
 
+  int mScreenRotationAngle;
+
   int mNotificationLevel;
   uint32_t mNotificationChangeState;
   bool mNotificationLevelChangeDone;