Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: e1dc7148f9e6b222c0012658b44ef27af1fdbdb8
      
https://github.com/WebKit/WebKit/commit/e1dc7148f9e6b222c0012658b44ef27af1fdbdb8
  Author: Youenn Fablet <[email protected]>
  Date:   2024-09-13 (Fri, 13 Sep 2024)

  Changed paths:
    A LayoutTests/fast/mediastream/getDisplayMedia-max-constraints5-expected.txt
    A LayoutTests/fast/mediastream/getDisplayMedia-max-constraints5.html
    M LayoutTests/platform/glib/TestExpectations
    M LayoutTests/platform/ios/TestExpectations
    M LayoutTests/platform/mac-site-isolation/TestExpectations
    M Source/WebKit/UIProcess/Cocoa/UserMediaCaptureManagerProxy.cpp

  Log Message:
  -----------
  getDisplayMedia track clone does not copy its video constraints
rdar://133611004
https://bugs.webkit.org/show_bug.cgi?id=279605

Reviewed by Jean-Yves Avenard.

When cloning, we were copying current settings and constraints, but not the 
video constraints themselves.
These video constraints are used in case of configuration change, for to apply 
max size constraints in case of source video frame size change.

Video frame size change is happening for real display sources as we do not know 
the actual size until getting the first video frame.
When cloning before receiving the first video frame, our size computation would 
be wrong.
And whenever a window resize happens, it would also be wrong.

To prevent this, we copy the video constraints to the clone track like we do 
for settings.
Covered by a layout test that clones a track, and then trigger a source 
configuration change.

* LayoutTests/fast/mediastream/getDisplayMedia-max-constraints5-expected.txt: 
Added.
* LayoutTests/fast/mediastream/getDisplayMedia-max-constraints5.html: Added.
* LayoutTests/platform/glib/TestExpectations:
* LayoutTests/platform/ios/TestExpectations:
* LayoutTests/platform/mac-site-isolation/TestExpectations:
* Source/WebKit/UIProcess/Cocoa/UserMediaCaptureManagerProxy.cpp:

Canonical link: https://commits.webkit.org/283612@main



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to