-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[iOS] Fail camera capture if there is no media environment #24873
[iOS] Fail camera capture if there is no media environment #24873
Conversation
EWS run on previous version of this PR (hash c1716da) |
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.
Nit: s/Updte/Update/ in the commit message
c1716da
to
fabc035
Compare
EWS run on previous version of this PR (hash fabc035) |
fabc035
to
4ea61cb
Compare
EWS run on current version of this PR (hash 4ea61cb) |
https://bugs.webkit.org/show_bug.cgi?id=269837 rdar://121770895 Reviewed by Eric Carlson. Update UserMediaCaptureManagerProxy::startProducingData to fail capture early if there is the browser entitlement, capture is for camera and there is no media environment. We add an ASSERT to MockRealtimeVideoSource::startProducingData and logging in AVVideoCaptureSource::setupSession. * Source/WebCore/platform/mediastream/RealtimeMediaSourceCenter.cpp: (WebCore::RealtimeMediaSourceCenter::setCurrentMediaEnvironment): * Source/WebCore/platform/mediastream/RealtimeMediaSourceCenter.h: * Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.mm: (WebCore::AVVideoCaptureSource::setupSession): * Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp: (WebCore::MockRealtimeVideoSource::startProducingData): * Source/WebKit/GPUProcess/GPUConnectionToWebProcess.cpp: * Source/WebKit/UIProcess/Cocoa/UserMediaCaptureManagerProxy.cpp: (WebKit::UserMediaCaptureManagerProxy::startProducingData): * Source/WebKit/UIProcess/Cocoa/UserMediaCaptureManagerProxy.h: (WebKit::UserMediaCaptureManagerProxy::ConnectionProxy::setCurrentMediaEnvironment): * Tools/TestWebKitAPI/Tests/WebKit/GetUserMedia.mm: (TestWebKitAPI::TEST): Canonical link: https://commits.webkit.org/275169@main
4ea61cb
to
857b349
Compare
Committed 275169@main (857b349): https://commits.webkit.org/275169@main Reviewed commits have been landed. Closing PR #24873 and removing active labels. |
857b349
4ea61cb
π§ͺ wpe-wk2π§ͺ gtk-wk2π§ͺ mac-AS-debug-wk2π§ͺ api-gtk