* The move/resize window protocol now uses absolute coordinates rather than

relative ones. This fixes bugs #2658, and #3213; in BWindow::ScreenChanged()
  the window does not yet know that it moved on the new screen (when it already
  had a position on that workspace).


git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@28908 a95241bf-73f2-0310-859d-f6bbb57e9c96

src/kits/interface/Window.cpp

@@ -2251,89 +2251,87 @@ BWindow::LastMouseMovedView() const
 void
 BWindow::MoveBy(float dx, float dy)
 {
-	if ((dx == 0.0 && dy == 0.0) || !Lock())
+	if ((dx != 0.0f || dy != 0.0f) && Lock()) {
-		return;
+		MoveTo(fFrame.left + dx, fFrame.top + dy);
-
+		Unlock();
-	fLink->StartMessage(AS_WINDOW_MOVE);
+	}
-	fLink->Attach<float>(dx);
-	fLink->Attach<float>(dy);
-
-	status_t status;
-	if (fLink->FlushWithReply(status) == B_OK && status == B_OK)
-		fFrame.OffsetBy(dx, dy);
-
-	Unlock();
 }
 
 
 void
 BWindow::MoveTo(BPoint point)
 {
-	if (!Lock())
+	MoveTo(point.x, point.y);
-		return;
-
-	point.x = roundf(point.x);
-	point.y = roundf(point.y);
-
-	if (fFrame.left != point.x || fFrame.top != point.y) {
-		float xOffset = point.x - fFrame.left;
-		float yOffset = point.y - fFrame.top;
-
-		MoveBy(xOffset, yOffset);
-	}
-
-	Unlock();
 }
 
 
 void
 BWindow::MoveTo(float x, float y)
 {
-	MoveTo(BPoint(x, y));
+	if (!Lock())
+		return;
+
+	x = roundf(x);
+	y = roundf(y);
+
+	if (fFrame.left != x || fFrame.top != y) {
+		fLink->StartMessage(AS_WINDOW_MOVE);
+		fLink->Attach<float>(x);
+		fLink->Attach<float>(y);
+
+		status_t status;
+		if (fLink->FlushWithReply(status) == B_OK && status == B_OK)
+			fFrame.OffsetTo(x, y);
+	}
+
+	Unlock();
 }
 
 
 void
 BWindow::ResizeBy(float dx, float dy)
 {
-	if (!Lock())
+	if (Lock()) {
-		return;
+		ResizeTo(fFrame.Width() + dx, fFrame.Height() + dy);
-
+		Unlock();
-	dx = roundf(dx);
-	dy = roundf(dy);
-
-	// stay in minimum & maximum frame limits
-	if (fFrame.Width() + dx < fMinWidth)
-		dx = fMinWidth - fFrame.Width();
-	if (fFrame.Width() + dx > fMaxWidth)
-		dx = fMaxWidth - fFrame.Width();
-	if (fFrame.Height() + dy < fMinHeight)
-		dy = fMinHeight - fFrame.Height();
-	if (fFrame.Height() + dy > fMaxHeight)
-		dy = fMaxHeight - fFrame.Height();
-
-	if (dx != 0.0 || dy != 0.0) {
-		fLink->StartMessage(AS_WINDOW_RESIZE);
-		fLink->Attach<float>(dx);
-		fLink->Attach<float>(dy);
-
-		status_t status;
-		if (fLink->FlushWithReply(status) == B_OK && status == B_OK) {
-			fFrame.SetRightBottom(fFrame.RightBottom() + BPoint(dx, dy));
-			_AdoptResize();
-		}
 	}
-	Unlock();
 }
 
 
 void
 BWindow::ResizeTo(float width, float height)
 {
-	if (Lock()) {
+	if (!Lock())
-		ResizeBy(width - fFrame.Width(), height - fFrame.Height());
+		return;
-		Unlock();
+
+	width = roundf(width);
+	height = roundf(height);
+
+	// stay in minimum & maximum frame limits
+	if (width < fMinWidth)
+		width = fMinWidth;
+	else if (width > fMaxWidth)
+		width = fMaxWidth;
+
+	if (height < fMinHeight)
+		height = fMinHeight;
+	else if (height > fMaxHeight)
+		height = fMaxHeight;
+
+	if (width != fFrame.Width() || height != fFrame.Height()) {
+		fLink->StartMessage(AS_WINDOW_RESIZE);
+		fLink->Attach<float>(width);
+		fLink->Attach<float>(height);
+
+		status_t status;
+		if (fLink->FlushWithReply(status) == B_OK && status == B_OK) {
+			fFrame.right = fFrame.left + width;
+			fFrame.bottom = fFrame.top + height;
+			_AdoptResize();
+		}
 	}
+
+	Unlock();
 }
 
 

src/servers/app/ServerWindow.cpp

@@ -938,14 +938,14 @@ ServerWindow::_DispatchMessage(int32 code, BPrivate::LinkReceiver &link)
 		}
 		case AS_WINDOW_RESIZE:
 		{
-			float xResizeBy;
+			float xResizeTo;
-			float yResizeBy;
+			float yResizeTo;
-			link.Read<float>(&xResizeBy);
+			link.Read<float>(&xResizeTo);
-			if (link.Read<float>(&yResizeBy) != B_OK)
+			if (link.Read<float>(&yResizeTo) != B_OK)
 				break;
 
 			DTRACE(("ServerWindow %s: Message AS_WINDOW_RESIZE %.1f, %.1f\n",
-				Title(), xResizeBy, yResizeBy));
+				Title(), xResizeTo, yResizeTo));
 
 			// comment this code for the time being, as some apps rely
 			// on the programmatically resize behavior during user resize
@@ -955,7 +955,9 @@ ServerWindow::_DispatchMessage(int32 code, BPrivate::LinkReceiver &link)
 //				fLink.StartMessage(B_BUSY);
 //			} else {
 //fDesktop->UnlockSingleWindow();
-				fDesktop->ResizeWindowBy(fWindow, xResizeBy, yResizeBy);
+				fDesktop->ResizeWindowBy(fWindow,
+					xResizeTo - fWindow->Frame().Width(),
+					yResizeTo - fWindow->Frame().Height());
 //fDesktop->LockSingleWindow();
 				fLink.StartMessage(B_OK);
 //			}
@@ -964,14 +966,14 @@ ServerWindow::_DispatchMessage(int32 code, BPrivate::LinkReceiver &link)
 		}
 		case AS_WINDOW_MOVE:
 		{
-			float xMoveBy;
+			float xMoveTo;
-			float yMoveBy;
+			float yMoveTo;
-			link.Read<float>(&xMoveBy);
+			link.Read<float>(&xMoveTo);
-			if (link.Read<float>(&yMoveBy) != B_OK)
+			if (link.Read<float>(&yMoveTo) != B_OK)
 				break;
 
 			DTRACE(("ServerWindow %s: Message AS_WINDOW_MOVE: %.1f, %.1f\n",
-				Title(), xMoveBy, yMoveBy));
+				Title(), xMoveTo, yMoveTo));
 
 			if (fWindow->IsDragging()) {
 				// While the user moves the window, we ignore
@@ -979,7 +981,8 @@ ServerWindow::_DispatchMessage(int32 code, BPrivate::LinkReceiver &link)
 				fLink.StartMessage(B_BUSY);
 			} else {
 //fDesktop->UnlockSingleWindow();
-				fDesktop->MoveWindowBy(fWindow, xMoveBy, yMoveBy);
+				fDesktop->MoveWindowBy(fWindow, xMoveTo - fWindow->Frame().left,
+					yMoveTo - fWindow->Frame().top);
 //fDesktop->LockSingleWindow();
 				fLink.StartMessage(B_OK);
 			}