Bug 1945834 - Fix handling of direct position on wayland popups after bug 581863. r=stransky

The code was relying on mBounds being set sync, which was only true for
popups before that change, and is no longer the case now.

Differential Revision: https://phabricator.services.mozilla.com/D236734

Author: emilio

widget/gtk/nsWindow.cpp

@@ -2115,7 +2115,7 @@ bool nsWindow::IsPopupDirectionRTL() {
 // It's used when we position noautihode popup and we don't use xdg_positioner.
 // See Bug 1718867
 void nsWindow::WaylandPopupSetDirectPosition() {
-  GdkPoint topLeft = DevicePixelsToGdkPointRoundDown(mBounds.TopLeft());
+  GdkPoint topLeft = DevicePixelsToGdkPointRoundDown(mLastMoveRequest);
   GdkRectangle size = DevicePixelsToGdkSizeRoundUp(mLastSizeRequest);
 
   LOG("nsWindow::WaylandPopupSetDirectPosition %d,%d -> %d x %d\n", topLeft.x,
@@ -2197,7 +2197,7 @@ bool nsWindow::WaylandPopupFitsToplevelWindow(bool aMove) {
   LOG("  parent size %d x %d", parentWidth, parentHeight);
 
   GdkPoint topLeft = aMove ? mPopupPosition
-                           : DevicePixelsToGdkPointRoundDown(mBounds.TopLeft());
+                           : DevicePixelsToGdkPointRoundDown(mLastMoveRequest);
   GdkRectangle size = DevicePixelsToGdkSizeRoundUp(mLastSizeRequest);
   LOG("  popup topleft %d, %d size %d x %d", topLeft.x, topLeft.y, size.width,
       size.height);
@@ -2210,7 +2210,7 @@ bool nsWindow::WaylandPopupFitsToplevelWindow(bool aMove) {
 }
 
 void nsWindow::NativeMoveResizeWaylandPopup(bool aMove, bool aResize) {
-  GdkPoint topLeft = DevicePixelsToGdkPointRoundDown(mBounds.TopLeft());
+  GdkPoint topLeft = DevicePixelsToGdkPointRoundDown(mLastMoveRequest);
   GdkRectangle size = DevicePixelsToGdkSizeRoundUp(mLastSizeRequest);
 
   LOG("nsWindow::NativeMoveResizeWaylandPopup Bounds %d,%d -> %d x %d move %d "