[ros-diffs] [jimtabor] 56838: [User32] - Sync up with wine NCActivate.

[jimtabor]

Author: jimtabor
Date: Thu Jul  5 21:39:21 2012
New Revision: 56838

URL: http://svn.reactos.org/svn/reactos?rev=56838&view=rev
Log:
[User32]
- Sync up with wine NCActivate.

Modified:
    trunk/reactos/win32ss/user/user32/windows/defwnd.c
    trunk/reactos/win32ss/user/user32/windows/nonclient.c

Modified: trunk/reactos/win32ss/user/user32/windows/defwnd.c
URL: 
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/user32/windows/defwnd.c?rev=56838&r1=56837&r2=56838&view=diff
==============================================================================
--- trunk/reactos/win32ss/user/user32/windows/defwnd.c [iso-8859-1] (original)
+++ trunk/reactos/win32ss/user/user32/windows/defwnd.c [iso-8859-1] Thu Jul  5 
21:39:21 2012
@@ -18,7 +18,7 @@
 
 LRESULT DefWndNCPaint(HWND hWnd, HRGN hRgn, BOOL Active);
 LRESULT DefWndNCCalcSize(HWND hWnd, BOOL CalcSizeStruct, RECT *Rect);
-LRESULT DefWndNCActivate(HWND hWnd, WPARAM wParam);
+LRESULT DefWndNCActivate(HWND hWnd, WPARAM wParam, LPARAM lParam);
 LRESULT DefWndNCHitTest(HWND hWnd, POINT Point);
 LRESULT DefWndNCLButtonDown(HWND hWnd, WPARAM wParam, LPARAM lParam);
 LRESULT DefWndNCLButtonDblClk(HWND hWnd, WPARAM wParam, LPARAM lParam);
@@ -1053,7 +1053,7 @@
 
         case WM_NCACTIVATE:
         {
-            return DefWndNCActivate(hWnd, wParam);
+            return DefWndNCActivate(hWnd, wParam, lParam);
         }
 
         case WM_NCHITTEST:

Modified: trunk/reactos/win32ss/user/user32/windows/nonclient.c
URL: 
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/user32/windows/nonclient.c?rev=56838&r1=56837&r2=56838&view=diff
==============================================================================
--- trunk/reactos/win32ss/user/user32/windows/nonclient.c [iso-8859-1] 
(original)
+++ trunk/reactos/win32ss/user/user32/windows/nonclient.c [iso-8859-1] Thu Jul  
5 21:39:21 2012
@@ -495,7 +495,7 @@
    if (hRgn != HRGN_WINDOW)
       DeleteObject(hRgn); // We use DCX_KEEPCLIPRGN
 
-   return 0;
+   return 0; // For WM_NCPAINT message, return 0.
 }
 
 LRESULT
@@ -652,9 +652,35 @@
 }
 
 LRESULT
-DefWndNCActivate(HWND hWnd, WPARAM wParam)
-{
-   DefWndNCPaint(hWnd, HRGN_WINDOW, wParam);
+DefWndNCActivate(HWND hWnd, WPARAM wParam, LPARAM lParam)
+{
+   PWND Wnd = ValidateHwnd(hWnd);
+
+   if (!Wnd) return 0;
+
+  /* Lotus Notes draws menu descriptions in the caption of its main
+   * window. When it wants to restore original "system" view, it just
+   * sends WM_NCACTIVATE message to itself. Any optimizations here in
+   * attempt to minimize redrawings lead to a not restored caption.   
+   */
+   if (wParam)
+      NtUserxSetWindowState(Wnd, WNDSACTIVEFRAME);
+   else
+      NtUserxClearWindowState(Wnd, WNDSACTIVEFRAME);
+
+   if (Wnd->state & WNDS_NONCPAINT)
+      return 0;
+
+   /* This isn't documented but is reproducible in at least XP SP2 and
+    * Outlook 2007 depends on it
+    */
+   // MSDN:
+   // If this parameter is set to -1, DefWindowProc does not repaint the
+   // nonclient area to reflect the state change.
+   if (lParam != -1)
+   {
+      DefWndNCPaint(hWnd, HRGN_WINDOW, wParam);
+   }
    return TRUE;
 }
 
@@ -930,6 +956,7 @@
    {
       if (GetMessageW(&Msg, 0, WM_MOUSEFIRST, WM_MOUSELAST) <= 0)
          break;
+      if (CallMsgFilterW( &Msg, MSGF_MAX )) continue;
 
       if (Msg.message == WM_LBUTTONUP)
          break;
@@ -948,7 +975,7 @@
    ReleaseCapture();
    ReleaseDC(hWnd, WindowDC);
    if (Pressed)
-      SendMessageW(hWnd, WM_SYSCOMMAND, SCMsg, 0);
+      SendMessageW(hWnd, WM_SYSCOMMAND, SCMsg, MAKELONG(Msg.pt.x,Msg.pt.y));
 }