Changeset 46465 in vbox for trunk/src/VBox/Main/src-server

Timestamp:

Jun 10, 2013 2:11:26 PM (12 years ago)

Author:

vboxsync

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

86308

Message:

Main/VPX: introduced onVideoCaptureChange event

File:

• trunk/src/VBox/Main/src-server/MachineImpl.cpp (modified) (9 diffs)

trunk/src/VBox/Main/src-server/MachineImpl.cpp

@@ -1721 +1721 @@
 STDMETHODIMP Machine::COMSETTER(VideoCaptureEnabled)(BOOL fEnabled)
 {
-    AutoCaller autoCaller(this);
-    if (FAILED(autoCaller.rc())) return autoCaller.rc();
-
+    HRESULT rc = S_OK;
+
+    AutoCaller autoCaller(this);
+    if (FAILED(autoCaller.rc())) return autoCaller.rc();
     AutoWriteLock alock(this COMMA_LOCKVAL_SRC_POS);
+
+    rc = checkStateDependency(MutableStateDep);
+    if (FAILED(rc)) return rc;
+
+    setModified(IsModified_MachineData);
+    mHWData.backup();
+
     mHWData->mVideoCaptureEnabled = fEnabled;
-    return S_OK;
+
+    return rc;
 }
 
@@ -1732 +1741 @@
 {
     CheckComArgOutSafeArrayPointerValid(aScreens);
+
+    AutoCaller autoCaller(this);
+    if (FAILED(autoCaller.rc())) return autoCaller.rc();
+
+    AutoReadLock alock(this COMMA_LOCKVAL_SRC_POS);
 
     SafeArray<BOOL> screens(mHWData->mMonitorCount);
@@ -1744 +1758 @@
     SafeArray<BOOL> screens(ComSafeArrayInArg(aScreens));
     AssertReturn(screens.size() <= RT_ELEMENTS(mHWData->maVideoCaptureScreens), E_INVALIDARG);
+    bool fChanged = false;
+
+    AutoWriteLock alock(this COMMA_LOCKVAL_SRC_POS);
+
     for (unsigned i = 0; i < screens.size(); i++)
-        mHWData->maVideoCaptureScreens[i] = RT_BOOL(screens[i]);
+    {
+        if (mHWData->maVideoCaptureScreens[i] != RT_BOOL(screens[i]))
+        {
+            mHWData->maVideoCaptureScreens[i] = RT_BOOL(screens[i]);
+            fChanged = true;
+        }
+    }
+    if (fChanged)
+    {
+        alock.release();
+        HRESULT rc = onVideoCaptureChange();
+        alock.acquire();
+        if (FAILED(rc)) return rc;
+        setModified(IsModified_MachineData);
+        if (Global::IsOnline(mData->mMachineState))
+            saveSettings(NULL);
+    }
+
     return S_OK;
 }
@@ -1766 +1801 @@
 
     AutoWriteLock alock(this COMMA_LOCKVAL_SRC_POS);
+
+    HRESULT rc = checkStateDependency(MutableStateDep);
+    if (FAILED(rc)) return rc; 
+
     if (strFile.isEmpty())
        strFile = "VideoCap.webm";
+
+    setModified(IsModified_MachineData);
+    mHWData.backup();
     mHWData->mVideoCaptureFile = strFile;
+
     return S_OK;
 }
@@ -1785 +1828 @@
 {
     AutoCaller autoCaller(this);
-    if (FAILED(autoCaller.rc()))
-    {
-        LogFlow(("Autolocked failed\n"));
-        return autoCaller.rc();
-    }
+    if (FAILED(autoCaller.rc())) return autoCaller.rc();
 
     AutoWriteLock alock(this COMMA_LOCKVAL_SRC_POS);
+
+    HRESULT rc = checkStateDependency(MutableStateDep);
+    if (FAILED(rc)) return rc; 
+
+    setModified(IsModified_MachineData);
+    mHWData.backup();
     mHWData->mVideoCaptureWidth = aHorzRes;
+
     return S_OK;
 }
@@ -1811 +1857 @@
     if (FAILED(autoCaller.rc())) return autoCaller.rc();
 
-    AutoReadLock alock(this COMMA_LOCKVAL_SRC_POS);
+    AutoWriteLock alock(this COMMA_LOCKVAL_SRC_POS);
+
+    HRESULT rc = checkStateDependency(MutableStateDep);
+    if (FAILED(rc)) return rc; 
+
+    setModified(IsModified_MachineData);
+    mHWData.backup();
     mHWData->mVideoCaptureHeight = aVertRes;
+
     return S_OK;
 }
@@ -1831 +1884 @@
     if (FAILED(autoCaller.rc())) return autoCaller.rc();
 
-    AutoReadLock alock(this COMMA_LOCKVAL_SRC_POS);
+    AutoWriteLock alock(this COMMA_LOCKVAL_SRC_POS);
+
+    HRESULT rc = checkStateDependency(MutableStateDep);
+    if (FAILED(rc)) return rc;
+
+    setModified(IsModified_MachineData);
+    mHWData.backup();
     mHWData->mVideoCaptureRate = aRate;
+
     return S_OK;
 }
@@ -1851 +1911 @@
     if (FAILED(autoCaller.rc())) return autoCaller.rc();
 
-    AutoReadLock alock(this COMMA_LOCKVAL_SRC_POS);
+    AutoWriteLock alock(this COMMA_LOCKVAL_SRC_POS);
+
+    HRESULT rc = checkStateDependency(MutableStateDep);
+    if (FAILED(rc)) return rc;
+
+    setModified(IsModified_MachineData);
+    mHWData.backup();
     mHWData->mVideoCaptureFps = aFps;
+
     return S_OK;
 }
@@ -13798 +13865 @@
 
 /**
+ * @note Locks this object for reading.
+ */
+HRESULT SessionMachine::onVideoCaptureChange()
+{
+    LogFlowThisFunc(("\n"));
+
+    AutoCaller autoCaller(this);
+    AssertComRCReturn(autoCaller.rc(), autoCaller.rc());
+
+    ComPtr<IInternalSessionControl> directControl;
+    {
+        AutoReadLock alock(this COMMA_LOCKVAL_SRC_POS);
+        directControl = mData->mSession.mDirectControl;
+    }
+
+    /* ignore notifications sent after #OnSessionEnd() is called */
+    if (!directControl)
+        return S_OK;
+
+    return directControl->OnVideoCaptureChange();
+}
+
+/**
  *  @note Locks this object for reading.
  */