Changeset 50258 in vbox for trunk

Timestamp:

Jan 28, 2014 2:16:08 PM (11 years ago)

Author:

vboxsync

Message:

crOpenGL: resize bugfixes

Location:

trunk/src/VBox/HostServices/SharedOpenGL/crserverlib

Files:

• server_main.c (modified) (2 diffs)
• server_presenter.cpp (modified) (2 diffs)

trunk/src/VBox/HostServices/SharedOpenGL/crserverlib/server_main.c

@@ -2845 +2845 @@
     CRScreenViewportInfo *pViewport;
     RTRECT NewRect;
-    GLboolean fChanged;
     int rc;
 
@@ -2862 +2861 @@
 
     pViewport = &cr_server.screenVieport[sIndex];
-
-    fChanged = !!memcmp(&NewRect, &pViewport->Rect, sizeof (NewRect));
-    if (!fChanged)
-    {
-        crDebug("crVBoxServerSetScreenViewport: no changes");
-        return VINF_SUCCESS;
-    }
-
+    /*always do viewport updates no matter whether the rectangle actually changes,
+     * this is needed to ensure window is adjusted properly on OSX */
     pViewport->Rect = NewRect;
     rc = CrPMgrViewportUpdate((uint32_t)sIndex);

trunk/src/VBox/HostServices/SharedOpenGL/crserverlib/server_presenter.cpp

@@ -1582 +1582 @@
 
         LOG(("CrWIN: Pos [%d ; %d]", x, y));
-
-        if (x != mxPos || y != myPos)
+//      always do WindowPosition to ensure window is adjusted properly
+//        if (x != mxPos || y != myPos)
         {
             mxPos = x;
@@ -1905 +1905 @@
         }
 
-        if (pViewportRect->xLeft != mViewportRect.xLeft || pViewportRect->yTop != mViewportRect.yTop)
+// always call SetPosition to ensure window is adjustep properly
+//        if (pViewportRect->xLeft != mViewportRect.xLeft || pViewportRect->yTop != mViewportRect.yTop)
         {
             const RTRECT* pRect = getRect();