Changeset 50314 in vbox for trunk/src/VBox/HostServices/SharedOpenGL

Timestamp:

Feb 3, 2014 7:29:26 PM (11 years ago)

Author:

vboxsync

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

92023

Message:

crOpenGL/VideoRec: do hardweight screen capthure only when recorder is ready

File:

• trunk/src/VBox/HostServices/SharedOpenGL/crserver/crservice.cpp (modified) (2 diffs)

trunk/src/VBox/HostServices/SharedOpenGL/crserver/crservice.cpp

@@ -933 +933 @@
 }
 
+static void crScreenshotHandle(CRVBOXHGCMTAKESCREENSHOT *pScreenshot, uint32_t idScreen, uint64_t u64Now)
+{
+    if (!pScreenshot->pfnScreenshotBegin || pScreenshot->pfnScreenshotBegin(pScreenshot->pvContext, idScreen, u64Now))
+    {
+        CR_SCREENSHOT Screenshot;
+
+        int rc = crServerVBoxScreenshotGet(idScreen, &Screenshot);
+        if (RT_SUCCESS(rc))
+        {
+            pScreenshot->pfnScreenshotPerform(pScreenshot->pvContext, idScreen,
+                    0, 0, 32,
+                    Screenshot.Img.pitch, Screenshot.Img.width, Screenshot.Img.height,
+                    (uint8_t*)Screenshot.Img.pvData, u64Now);
+            crServerVBoxScreenshotRelease(&Screenshot);
+        }
+        else
+        {
+            Assert(rc == VERR_INVALID_STATE);
+        }
+
+        if (pScreenshot->pfnScreenshotEnd)
+            pScreenshot->pfnScreenshotEnd(pScreenshot->pvContext, idScreen, u64Now);
+    }
+}
+
 /*
  * We differentiate between a function handler for the guest and one for the host.
@@ -1225 +1250 @@
                 for (uint32_t i = 0; i < g_u32ScreenCount; ++i)
                 {
-                    CR_SCREENSHOT Screenshot;
-
-                    int rc = crServerVBoxScreenshotGet(i, &Screenshot);
-                    if (RT_SUCCESS(rc))
-                    {
-                        pScreenshot->pfnScreenshot(pScreenshot->pvContext, i,
-                                0, 0, 32,
-                                Screenshot.Img.pitch, Screenshot.Img.width, Screenshot.Img.height,
-                                (uint8_t*)Screenshot.Img.pvData, u64Now);
-                        crServerVBoxScreenshotRelease(&Screenshot);
-                    }
-                    else
-                    {
-                        Assert(rc == VERR_INVALID_STATE);
-                    }
+                    crScreenshotHandle(pScreenshot, i, u64Now);
                 }
             }
             else if (pScreenshot->u32Screen < g_u32ScreenCount)
             {
-                CR_SCREENSHOT Screenshot;
-
-                int rc = crServerVBoxScreenshotGet(pScreenshot->u32Screen, &Screenshot);
-                if (RT_SUCCESS(rc))
-                {
-                    pScreenshot->pfnScreenshot(pScreenshot->pvContext, pScreenshot->u32Screen,
-                            0, 0, 32,
-                            Screenshot.Img.pitch, Screenshot.Img.width, Screenshot.Img.height,
-                            (uint8_t*)Screenshot.Img.pvData, u64Now);
-                    crServerVBoxScreenshotRelease(&Screenshot);
-                }
-                else
-                {
-                    Assert(rc == VERR_INVALID_STATE);
-                }
+                crScreenshotHandle(pScreenshot, pScreenshot->u32Screen, u64Now);
             }
             else