Changeset 42646 in vbox for trunk/src/VBox/Main/src-client

Timestamp:

Aug 7, 2012 7:12:56 AM (13 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

79843

Message:

Main/src-client/DisplayImpl: Code improvement for video recording of SDL session.

File:

• trunk/src/VBox/Main/src-client/DisplayImpl.cpp (modified) (3 diffs)

trunk/src/VBox/Main/src-client/DisplayImpl.cpp

@@ -3256 +3256 @@
          && pFBInfo->u32ResizeStatus==ResizeStatus_Void)
     {
+        HRESULT rc;
         ULONG ulPixelFormat = 0;
-        pFBInfo->pFramebuffer->COMGETTER(PixelFormat)(&ulPixelFormat);
+        rc = pFBInfo->pFramebuffer->COMGETTER(PixelFormat)(&ulPixelFormat);
+        AssertComRC (rc);
 
         ULONG ulBitsPerPixel;
-        pFBInfo->pFramebuffer->COMGETTER(BitsPerPixel)(&ulBitsPerPixel);
-
-        ULONG ulFrameHeight = 0;
-        pFBInfo->pFramebuffer->COMGETTER(Height)(&ulFrameHeight);
-
-        ULONG ulFrameWidth = 0;
-        pFBInfo->pFramebuffer->COMGETTER(Width)(&ulFrameWidth);
+        rc = pFBInfo->pFramebuffer->COMGETTER(BitsPerPixel)(&ulBitsPerPixel);
+        AssertComRC (rc);
+
+        ULONG ulGuestHeight = 0;
+        rc = pFBInfo->pFramebuffer->COMGETTER(Height)(&ulGuestHeight);
+        AssertComRC (rc);
+
+        ULONG ulGuestWidth = 0;
+        rc = pFBInfo->pFramebuffer->COMGETTER(Width)(&ulGuestWidth);
+        AssertComRC (rc);
 
         BYTE *address = NULL;
-        HRESULT hrc = pFBInfo->pFramebuffer->COMGETTER(Address) (&address);
+        rc = pFBInfo->pFramebuffer->COMGETTER(Address) (&address);
+        AssertComRC (rc);
+
+        ULONG ulBytesPerLine = 0;
+        rc = pFBInfo->pFramebuffer->COMGETTER(BytesPerLine) (&ulBytesPerLine);
+        AssertComRC (rc);
 
 
@@ -3306 +3316 @@
         }
 
-        LogFlow(("cp to IntBuf orgx=%d orgy=%d cx=%d cy=%d bpp=%d\n",
-                   pFBInfo->xOrigin, pFBInfo->yOrigin,  pDrv->IConnector.cx, pDrv->IConnector.cy,
-                   pDrv->IConnector.cBits));
-
-
         if (u32VideoRecImgFormat != VPX_IMG_FMT_NONE && address != NULL)
         {
             VideoRecCopyToIntBuffer(pVideoRecContext, pFBInfo->xOrigin, pFBInfo->yOrigin,
-                              u32PixelFormat, ulBitsPerPixel, pDrv->IConnector.cbScanline,
-                              ulFrameWidth, ulFrameHeight,address);
+                              u32PixelFormat, ulBitsPerPixel, ulBytesPerLine,
+                              ulGuestWidth, ulGuestHeight, address);
 
 
             LogFlow(("RGB:YUV\n"));
-            VideoRecDoRGBToYUV(ulFrameWidth, ulFrameHeight, u32VideoRecImgFormat,
-                            pVideoRecContext->pu8TempYUVBuffer,
-                            pVideoRecContext->pu8TempRGBBuffer);
+            VideoRecDoRGBToYUV(pVideoRecContext, u32VideoRecImgFormat);
             LogFlow(("Encode\n"));
-            VideoRecEncodeAndWrite(pVideoRecContext, ulFrameWidth,
-                                    ulFrameHeight, pVideoRecContext->pu8TempYUVBuffer);
+            VideoRecEncodeAndWrite(pVideoRecContext, ulGuestWidth, ulGuestHeight);
         }
     }
@@ -4315 +4317 @@
 
     if(res == VINF_SUCCESS)
-        res = VideoRecContextInit(pVideoRecContext, "test.webm", 800, 720);
+        res = VideoRecContextInit(pVideoRecContext, "test.webm", 640, 480);
 
 #endif