Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: aff28bc50d0e283250350647403aba6131fbd8fb
      
https://github.com/WebKit/WebKit/commit/aff28bc50d0e283250350647403aba6131fbd8fb
  Author: Philippe Normand <[email protected]>
  Date:   2023-03-22 (Wed, 22 Mar 2023)

  Changed paths:
    M LayoutTests/platform/glib/TestExpectations
    M LayoutTests/webrtc/routines.js
    M LayoutTests/webrtc/video-disabled-black.html
    M 
Source/WebCore/platform/mediastream/gstreamer/GStreamerMediaStreamSource.cpp

  Log Message:
  -----------
  [Gstreamer][WebRTC] webrtc/video-disabled-black.html fails
https://bugs.webkit.org/show_bug.cgi?id=254212

Reviewed by Xabier Rodriguez-Calvar.

Flushing a disabled video track would lead to a caps negotiation failure 
afterwards, so flush only
if the track is being re-enabled.

The webrtc/video-disabled-black.html test was also rewritten in order to avoid 
flakiness, we now
check the video at most 200 times instead of one, before failing the test.

Also drive-by in MediaStreamSource, rebuild black frame caps when needed, 
instead of attempting to
update caps that are not writable.

* LayoutTests/platform/glib/TestExpectations:
* LayoutTests/webrtc/routines.js:
* LayoutTests/webrtc/video-disabled-black.html:
* Source/WebCore/platform/mediastream/gstreamer/GStreamerMediaStreamSource.cpp:

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


_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to