[ros-diffs] [navaraf] 14342: - Rewrite NtUserGetUpdateRect. It should return TRUE when the window needs to be pained, not only when the client update rect is non-empty.

- Rewrite NtUserGetUpdateRect. It should return TRUE when the window
needs to be pained, not only when the client update rect is non-empty.
- Return TRUE for windows with pending WM_NCPAINT message in
IntIsWindowDirty.
Modified: trunk/reactos/subsys/win32k/ntuser/painting.c
  _____  

Modified: trunk/reactos/subsys/win32k/ntuser/painting.c
--- trunk/reactos/subsys/win32k/ntuser/painting.c	2005-03-27
01:57:26 UTC (rev 14341)
+++ trunk/reactos/subsys/win32k/ntuser/painting.c	2005-03-27
08:29:25 UTC (rev 14342)
@@ -537,7 +537,8 @@

 {
    return (Window->Style & WS_VISIBLE) &&
       ((Window->UpdateRegion != NULL) ||
-       (Window->Flags & WINDOWOBJECT_NEED_INTERNALPAINT));
+       (Window->Flags & WINDOWOBJECT_NEED_INTERNALPAINT) ||
+       (Window->Flags & WINDOWOBJECT_NEED_NCPAINT));
 }
 
 HWND STDCALL
@@ -873,6 +874,7 @@
       return ERROR;
    }
     
+   IntLockWindowUpdate(Window);
    if (Window->UpdateRegion == NULL)
    {
       RegionType = (NtGdiSetRectRgn(hRgn, 0, 0, 0, 0) ? NULLREGION :
ERROR);
@@ -885,6 +887,7 @@
          Window->WindowRect.left - Window->ClientRect.left,
          Window->WindowRect.top - Window->ClientRect.top);
    }
+   IntUnLockWindowUpdate(Window);
 
    IntReleaseWindowObject(Window);
 
@@ -904,49 +907,56 @@
  */
 
 BOOL STDCALL
-NtUserGetUpdateRect(HWND Wnd, LPRECT UnsafeRect, BOOL Erase)
+NtUserGetUpdateRect(HWND hWnd, LPRECT UnsafeRect, BOOL bErase)
 {
-  RECT Rect;
-  HRGN Rgn;
-  PROSRGNDATA RgnData;
-  NTSTATUS Status;
+   PWINDOW_OBJECT Window;
+   RECT Rect;
+   INT RegionType;
+   PROSRGNDATA RgnData;
+   BOOL AlwaysPaint;
+   NTSTATUS Status;
 
-  Rgn = NtGdiCreateRectRgn(0, 0, 0, 0);
-  if (NULL == Rgn)
-    {
-      NtGdiDeleteObject(Rgn);
-      SetLastWin32Error(ERROR_NO_SYSTEM_RESOURCES);
-      return FALSE;
-    }
-  NtUserGetUpdateRgn(Wnd, Rgn, Erase);
-  RgnData = RGNDATA_LockRgn(Rgn);
-  if (NULL == RgnData)
-    {
-      NtGdiDeleteObject(Rgn);
-      SetLastWin32Error(ERROR_NO_SYSTEM_RESOURCES);
-      return FALSE;
-    }
-  if (ERROR == UnsafeIntGetRgnBox(RgnData, &Rect))
-    {
-      RGNDATA_UnlockRgn(Rgn);
-      NtGdiDeleteObject(Rgn);
-      SetLastWin32Error(ERROR_NO_SYSTEM_RESOURCES);
-      return FALSE;
-    }
-  RGNDATA_UnlockRgn(Rgn);
-  NtGdiDeleteObject(Rgn);
+   if (!(Window = IntGetWindowObject(hWnd)))
+   {
+      SetLastWin32Error(ERROR_INVALID_WINDOW_HANDLE);
+      return ERROR;
+   }
+    
+   IntLockWindowUpdate(Window);
+   if (Window->UpdateRegion == NULL)
+   {
+      Rect.left = Rect.top = Rect.right = Rect.bottom = 0;
+   }
+   else
+   {
+      RgnData = RGNDATA_LockRgn(Window->UpdateRegion);
+      ASSERT(RgnData != NULL);
+      RegionType = UnsafeIntGetRgnBox(RgnData, &Rect);
+      ASSERT(RegionType != ERROR);
+      RGNDATA_UnlockRgn(Window->UpdateRegion);
+   }
+   AlwaysPaint = (Window->Flags & WINDOWOBJECT_NEED_NCPAINT) ||
+                 (Window->Flags & WINDOWOBJECT_NEED_INTERNALPAINT);
+   IntUnLockWindowUpdate(Window);
 
-  if (UnsafeRect != NULL)
-    {
+   IntReleaseWindowObject(Window);
+
+   if (bErase && Rect.left < Rect.right && Rect.top < Rect.bottom)
+   {
+      NtUserRedrawWindow(hWnd, NULL, NULL, RDW_ERASENOW |
RDW_NOCHILDREN);
+   }
+  
+   if (UnsafeRect != NULL)
+   {
       Status = MmCopyToCaller(UnsafeRect, &Rect, sizeof(RECT));
-      if (! NT_SUCCESS(Status))
-        {
-          SetLastWin32Error(ERROR_INVALID_PARAMETER);
-          return FALSE;
-        }
-    }
+      if (!NT_SUCCESS(Status))
+      {
+         SetLastWin32Error(ERROR_INVALID_PARAMETER);
+         return FALSE;
+      }
+   }
 
-  return Rect.left < Rect.right && Rect.top < Rect.bottom;
+   return (Rect.left < Rect.right && Rect.top < Rect.bottom) ||
AlwaysPaint;
 }
 
 /*