Branch: refs/heads/main Home: https://github.com/WebKit/WebKit Commit: 862a01816580495be0447f4c9f069eda73bd79dd https://github.com/WebKit/WebKit/commit/862a01816580495be0447f4c9f069eda73bd79dd Author: Youenn Fablet <you...@apple.com> Date: 2025-03-26 (Wed, 26 Mar 2025)
Changed paths: M Source/WebCore/platform/mediastream/RealtimeMediaSourceCenter.cpp M Source/WebCore/platform/mediastream/RealtimeMediaSourceCenter.h M Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in M Source/WebKit/UIProcess/UserMediaPermissionRequestManagerProxy.cpp M Source/WebKit/UIProcess/UserMediaPermissionRequestManagerProxy.h M Source/WebKit/UIProcess/glib/UserMediaPermissionRequestManagerProxyGLib.cpp M Source/WebKit/WebProcess/cocoa/UserMediaCaptureManager.h M Source/WebKit/WebProcess/glib/UserMediaCaptureManager.cpp M Source/WebKit/WebProcess/glib/UserMediaCaptureManager.h M Source/WebKit/WebProcess/glib/UserMediaCaptureManager.messages.in Log Message: ----------- RealtimeMediaSourceCenter::validateRequestConstraints should take a single CompletionHandler rdar://147825053 https://bugs.webkit.org/show_bug.cgi?id=290378 Reviewed by Philippe Normand. Before the patch, validateRequestConstraints would take two functions, one for success and one for error case. We are merging both in a single CompletionHandler taking an Expected, as it is more standard nowadays and better overall. * Source/WebCore/platform/mediastream/RealtimeMediaSourceCenter.cpp: (WebCore::RealtimeMediaSourceCenter::validateRequestConstraints): (WebCore::RealtimeMediaSourceCenter::validateRequestConstraintsAfterEnumeration): * Source/WebCore/platform/mediastream/RealtimeMediaSourceCenter.h: * Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in: * Source/WebKit/UIProcess/UserMediaPermissionRequestManagerProxy.cpp: (WebKit::UserMediaPermissionRequestManagerProxy::grantRequest): (WebKit::UserMediaPermissionRequestManagerProxy::processUserMediaPermissionRequest): (WebKit::UserMediaPermissionRequestManagerProxy::validateUserMediaRequestConstraints): (WebKit::UserMediaPermissionRequestManagerProxy::platformValidateUserMediaRequestConstraints): Deleted. * Source/WebKit/UIProcess/UserMediaPermissionRequestManagerProxy.h: * Source/WebKit/UIProcess/glib/UserMediaPermissionRequestManagerProxyGLib.cpp: (WebKit::UserMediaPermissionRequestManagerProxy::validateUserMediaRequestConstraints): (WebKit::UserMediaPermissionRequestManagerProxy::platformValidateUserMediaRequestConstraints): Deleted. * Source/WebKit/WebProcess/cocoa/UserMediaCaptureManager.h: * Source/WebKit/WebProcess/glib/UserMediaCaptureManager.cpp: (WebKit::UserMediaCaptureManagerr:validateUserMediaRequestConstraints): (WebKit::UserMediaCaptureManager::validateUserMediaRequestConstraints): Deleted. * Source/WebKit/WebProcess/glib/UserMediaCaptureManager.h: * Source/WebKit/WebProcess/glib/UserMediaCaptureManager.messages.in: Canonical link: https://commits.webkit.org/292693@main To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications _______________________________________________ webkit-changes mailing list webkit-changes@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-changes