From 8689f7d81f4926423c6c40af10dac2b07a5ce997 Mon Sep 17 00:00:00 2001
From: "bzbarsky%mit.edu" <bzbarsky%mit.edu>
Date: Fri, 20 Jan 2006 21:31:29 +0000
Subject: [PATCH] Fix bug 297541.  Patch by roc, r=pavlov, sr=bzbarsky

---
 widget/src/gtk2/nsWindow.cpp | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/widget/src/gtk2/nsWindow.cpp b/widget/src/gtk2/nsWindow.cpp
index f405d0621a4d..5e41638f0357 100644
--- a/widget/src/gtk2/nsWindow.cpp
+++ b/widget/src/gtk2/nsWindow.cpp
@@ -551,6 +551,8 @@ nsWindow::Move(PRInt32 aX, PRInt32 aY)
         mWindowType != eWindowType_popup)
         return NS_OK;
 
+    // XXX Should we do some AreBoundsSane check here?
+
     mBounds.x = aX;
     mBounds.y = aY;
 
@@ -570,8 +572,12 @@ nsWindow::Move(PRInt32 aX, PRInt32 aY)
             // actually placed the window somewhere.  If no placement
             // has taken place, we just let the window manager Do The
             // Right Thing.
-            if (mPlaced)
+            // XXX Uhhh ... aX and aY are in twips, not screen coordinates
+            // XXX mPlaced is always PR_TRUE here, see above
+            if (mPlaced) {
+                NS_WARNING("BOGUS code reached!");
                 gtk_window_move(GTK_WINDOW(mShell), aX, aY);
+            }
         }
     }
     else if (mDrawingarea) {
@@ -2275,6 +2281,13 @@ nsWindow::NativeCreate(nsIWidget        *aParent,
 
     // save our bounds
     mBounds = aRect;
+    if (mWindowType != eWindowType_child) {
+        // The window manager might place us. Indicate that if we're
+        // shown, we want to go through
+        // nsWindow::NativeResize(x,y,w,h) to maybe set our own
+        // position.
+        mNeedsMove = PR_TRUE;
+    }
 
     // figure out our parent window
     MozDrawingarea *parentArea = nsnull;