[Bug]: Capturing the screen is broken in the newer version (desktopCapturer)

[YouDJapp]

Preflight Checklist

• I have read the Contributing Guidelines for this project.
• I agree to follow the Code of Conduct that this project adheres to.
• I have searched the issue tracker for a bug report that matches the one I want to file, without success.

Electron Version

30.0.1

What operating system are you using?

macOS

Operating System Version

MacOS Sonoma 14.1.1

What arch are you using?

x64

Last Known Working Electron version

25.2.0

Expected Behavior

Capturing the screen of the app (desktopCapturer.getSources / mediaDevices.getUserMedia) and recording it (MediaRecorder) works well, like in v25.2.0, here is a working video : https://youtu.be/0fmZNy6Jo84.

var video_settings = { mandatory: { chromeMediaSource: 'desktop'}};
video_settings.mandatory.chromeMediaSourceId = video_sourceid; // ID from the desktopCapturer electron API

navigator.mediaDevices.getUserMedia({audio: false, video: video_settings}).then(function(stream) {
recorder_chunks = [];
var recorder = new MediaRecorder(stream, { mimeType: 'video/webm; codecs=vp9', audioBitsPerSecond : 192e3, videoBitsPerSecond : 8e6});
recorder.ondataavailable = function(e) { recorder_chunks.push(e.data); };
recorder.start(100);
});

Actual Behavior

Updating my app to v30, the screen capture is frozen at the first frame, but only the mouse is moving. Same code... Here is a video of the recording showing the issue : https://youtu.be/hHx2q9JqMNM.

Testcase Gist URL

No response

Additional Information

No response

[electron-issue-triage]

Hello @YouDJapp. Thanks for reporting this and helping to make Electron better!

Would it be possible for you to make a standalone testcase with only the code necessary to reproduce the issue? For example, Electron Fiddle is a great tool for making small test cases and makes it easy to publish your test case to a gist that Electron maintainers can use.

Stand-alone test cases make fixing issues go more smoothly: it ensure everyone's looking at the same issue, it removes all unnecessary variables from the equation, and it can also provide the basis for automated regression tests.

Now adding the blocked/need-repro Needs a test case to reproduce the bug label for this reason. After you make a test case, please link to it in a followup comment. This issue will be closed in 10 days if the above is not addressed.

[YouDJapp]

nevermind, I was sending not sending the wrong source ID, it works now

[VerteDinde]

Glad it works! Thanks for letting us know!