[ros-diffs] [navaraf] 16751: Yet another fix to WinPosActivateOtherWindow, we should search only for top-level windows next to the window we're about to deactivate.

Yet another fix to WinPosActivateOtherWindow, we should search only for
top-level windows next to the window we're about to deactivate.
Modified: trunk/reactos/subsys/win32k/ntuser/winpos.c
  _____  

Modified: trunk/reactos/subsys/win32k/ntuser/winpos.c
--- trunk/reactos/subsys/win32k/ntuser/winpos.c	2005-07-26 17:46:48 UTC
(rev 16750)
+++ trunk/reactos/subsys/win32k/ntuser/winpos.c	2005-07-26 19:02:31 UTC
(rev 16751)
@@ -114,6 +114,8 @@

     IntSetFocusMessageQueue(NULL);
     return;
   }
+
+  /* If this is popup window, try to activate the owner first. */
   if ((Window->Style & WS_POPUP) && (Wnd = IntGetOwner(Window)))
   {
     for(;;)
@@ -135,50 +137,21 @@
 
     IntReleaseWindowObject(Wnd);
   }
+
+  /* Pick a next top-level window. */
+  /* FIXME: Search for non-tooltip windows first. */
   Wnd = Window;
-  for(;;)
+  while (Wnd != NULL)
   {
-    HWND *List, *phWnd;
-
     Old = Wnd;
-    Wnd = IntGetParentObject(Wnd);
-    if(Old != Window)
-    {
+    IntLockRelatives(Old);
+    Wnd = IntGetWindowObject(Old->NextSibling->Self);
+    IntUnLockRelatives(Old);
+    if (Old != Window)
       IntReleaseWindowObject(Old);
-    }
-    if(!Wnd)
-    {
-      IntSetFocusMessageQueue(NULL);
-      return;
-    }
-
-    if((List = IntWinListChildren(Wnd)))
-    {
-      /* FIXME: We should scan for non-tooltip windows first. */
-      for(phWnd = List; *phWnd; phWnd++)
-      {
-        PWINDOW_OBJECT Child;
-
-        if((*phWnd) == Window->Self)
-        {
-          continue;
-        }
-
-        if((Child = IntGetWindowObject(*phWnd)))
-        {
-          if ((Child->Style & (WS_DISABLED | WS_VISIBLE)) == WS_VISIBLE
&&
-              (Child->Style & (WS_POPUP | WS_CHILD)) != WS_CHILD)
-          {
-            ExFreePool(List);
-            IntReleaseWindowObject(Wnd);
-            Wnd = Child;
-            goto done;
-          }
-          IntReleaseWindowObject(Child);
-        }
-      }
-      ExFreePool(List);
-    }
+    if ((Wnd->Style & (WS_DISABLED | WS_VISIBLE)) == WS_VISIBLE &&
+        (Wnd->Style & (WS_POPUP | WS_CHILD)) != WS_CHILD)
+      break;
   }
 
 done:
@@ -193,7 +166,8 @@
   }
   if (!IntSetActiveWindow(Wnd))
     IntSetActiveWindow(0);
-  IntReleaseWindowObject(Wnd);
+  if (Wnd)
+    IntReleaseWindowObject(Wnd);
 }