New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ensure DirectX capturer is supported. #13543
Conversation
@MarshallOfSound I haven't succeeded in packaging a Windows build to test on my Win7 machine, but this works as before on Win10 and I suspect will resolve the Win7 crash. Want to merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suspect will resolve the Win7 crash
Can you please actually test you changes on Windows 7?
I'd love to, but haven't been able to produce a distributable package on Windows to test on my Win7 machine. Is there any way to have CI produce one? |
|
@ajmacd I'll try building this on my local today and send you a build on Slack 👍 |
@MarshallOfSound any progress on this? |
Sorry for the delay. Finally got a working build on a Win7 machine and verified this PR resolves the crash in #13426. The display IDs don't match between the desktopCapturer and screen APIs, but that's not a regression. Let's merge this! |
@alexeykuzmin Good to merge? |
/trop run backport-to 3-0-x |
The backport process for this PR has been manually initiated, sending your 1's and 0's to "3-0-x" here we go! :D |
We have automatically backported this PR to "3-0-x", please check out #13738 |
/trop run backport-to 2-1-x |
The backport process for this PR has been manually initiated, sending your 1's and 0's to "2-1-x" here we go! :D |
An error occurred while attempting to backport this PR to "2-1-x", you will need to perform this backport manually |
Oop. My first guess is this failed because #13986 needed to be merged first.. let's try again. |
/trop run backport-to 2-1-x |
The backport process for this PR has been manually initiated, sending your 1's and 0's to "2-1-x" here we go! :D |
An error occurred while attempting to backport this PR to "2-1-x", you will need to perform this backport manually |
Bah. |
Ensure DirectX capturer is supported.
* Merge PR #13543 from electron/ajm-win7-crash Ensure DirectX capturer is supported. * chore: remove unnecessary whitespace change * refactor: update desktop capturer backport to old content namespace
Should resolve #13426