Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 9671d6e9c9773473d974c053cd8c7f1bc6a9788c
      
https://github.com/WebKit/WebKit/commit/9671d6e9c9773473d974c053cd8c7f1bc6a9788c
  Author: Youenn Fablet <[email protected]>
  Date:   2025-09-29 (Mon, 29 Sep 2025)

  Changed paths:
    M 
LayoutTests/imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-stop-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-stop.html
    R 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-stop-expected.txt
    M Source/WebCore/Modules/mediarecorder/MediaRecorder.cpp

  Log Message:
  -----------
  Limit the events fired by a stopped MediaRecorder in case of change of track 
states
rdar://161124260
https://bugs.webkit.org/show_bug.cgi?id=299601

Reviewed by Jean-Yves Avenard.

When a MediaRecorder is stopped just after its stream has changed (track 
stopped or track addition/removal),
we were previously firing error events, even though the actual recording stop 
was triggered by the web page.

We are now exiting early if the recorder is stopped while we process the stream 
changes.
This aligns with Chrome behavior.
After the patch, we get the same behavior for Firefox in one case and get 
closer to Firefox in another case.

* 
LayoutTests/imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-stop-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-stop.html:
* 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-stop-expected.txt:
 Removed.
* Source/WebCore/Modules/mediarecorder/MediaRecorder.cpp:
(WebCore::MediaRecorder::handleTrackChange):
(WebCore::MediaRecorder::trackEnded):

Canonical link: https://commits.webkit.org/300682@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