Do not call RequestUpdateOnce, instead Deiconify

We can only assume that Deiconify signal can assume that window is valid.
Other API is just kind of request to window server, so we could not send
rendering forcibly.

Change-Id: I6d6b9348d4661efb241cb6aa24209346d908f5de
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

diff --git a/dali/internal/window-system/common/window-impl.cpp b/dali/internal/window-system/common/window-impl.cpp
index 2687a32e5d65ebf3ba3aaef616fbc96cdcc8b020..006b7a959cee5995e8166ff0304defdb0c8f1458 100644 (file)
--- a/dali/internal/window-system/common/window-impl.cpp
+++ b/dali/internal/window-system/common/window-impl.cpp
@@ -577,12 +577,6 @@ void Window::Show()
 
   mSurface->SetFullSwapNextFrame();
 
-  // Need to update/render once if surface set full swaped after adaptor call ProcessCoreEvents().
-  if(DALI_LIKELY(mAdaptor))
-  {
-    mAdaptor->RequestUpdateOnce();
-  }
-
   DALI_LOG_RELEASE_INFO("Window (%p), WinId (%d), Show(): iconified = %d, visible = %d\n", this, mNativeWindowId, mIconified, mVisible);
 }
 
@@ -771,12 +765,6 @@ void Window::SetSize(Dali::Window::WindowSize size)
   }
 
   mSurface->SetFullSwapNextFrame();
-
-  // Need to update/render once if surface set full swaped after adaptor call ProcessCoreEvents().
-  if(DALI_LIKELY(mAdaptor))
-  {
-    mAdaptor->RequestUpdateOnce();
-  }
 }
 
 Dali::Window::WindowSize Window::GetSize() const
@@ -888,12 +876,6 @@ void Window::SetPositionSize(PositionSize positionSize)
   }
 
   mSurface->SetFullSwapNextFrame();
-
-  // Need to update/render once if surface set full swaped after adaptor call ProcessCoreEvents().
-  if(DALI_LIKELY(mAdaptor))
-  {
-    mAdaptor->RequestUpdateOnce();
-  }
 }
 
 void Window::SetLayout(unsigned int numCols, unsigned int numRows, unsigned int column, unsigned int row, unsigned int colSpan, unsigned int rowSpan)
@@ -1013,7 +995,7 @@ void Window::OnIconifyChanged(bool iconified)
   mSurface->SetFullSwapNextFrame();
 
   // Need to update/render once if surface set full swaped after adaptor call ProcessCoreEvents().
-  if(DALI_LIKELY(mAdaptor))
+  if(DALI_LIKELY(mAdaptor) && !iconified)
   {
     mAdaptor->RequestUpdateOnce();
   }
@@ -1152,12 +1134,6 @@ void Window::OnUpdatePositionSize(Dali::PositionSize& positionSize)
   }
 
   mSurface->SetFullSwapNextFrame();
-
-  // Need to update/render once if surface set full swaped after adaptor call ProcessCoreEvents().
-  if(DALI_LIKELY(mAdaptor))
-  {
-    mAdaptor->RequestUpdateOnce();
-  }
 }
 
 void Window::OnTouchPoint(Dali::Integration::Point& point, int timeStamp)
@@ -1255,12 +1231,6 @@ void Window::OnResume()
   }
 
   mSurface->SetFullSwapNextFrame();
-
-  // Need to update/render once if surface set full swaped after adaptor call ProcessCoreEvents().
-  if(DALI_LIKELY(mAdaptor))
-  {
-    mAdaptor->RequestUpdateOnce();
-  }
 }
 
 void Window::OnAuxiliaryMessage(const std::string& key, const std::string& value, const Property::Array& options)