[webkit-changes] [WebKit/WebKit] 523cb7: REGRESSION(263499 at main): [GStreamer] Several webrt...

Philippe Normand noreply at github.com
Tue May 2 04:02:15 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 523cb7fde9c6422fb077a53a9b58ea07e9a18a1e
      https://github.com/WebKit/WebKit/commit/523cb7fde9c6422fb077a53a9b58ea07e9a18a1e
  Author: Philippe Normand <philn at igalia.com>
  Date:   2023-05-02 (Tue, 02 May 2023)

  Changed paths:
    M LayoutTests/platform/glib/TestExpectations
    M Source/WebCore/platform/mediastream/RealtimeMediaSource.h
    M Source/WebCore/platform/mediastream/gstreamer/GStreamerMediaStreamSource.cpp
    M Source/WebCore/platform/mediastream/gstreamer/RealtimeIncomingSourceGStreamer.cpp
    M Source/WebCore/platform/mediastream/gstreamer/RealtimeIncomingSourceGStreamer.h

  Log Message:
  -----------
  REGRESSION(263499 at main): [GStreamer] Several webrtc tests timing out
https://bugs.webkit.org/show_bug.cgi?id=256127

Reviewed by Xabier Rodriguez-Calvar.

The timeouts were caused by the playback appsrc forwarding caps queries to the incoming source. Also
the incoming source shouldn't forward stream collection events to the playback appsrc because it is
already handling stream collections itself.

Driving by, the handleDownstreamEvent plumbing is no longer needed now that we know which appsrc is
related to the incoming source appsinks.

* LayoutTests/platform/glib/TestExpectations:
* Source/WebCore/platform/mediastream/RealtimeMediaSource.h:
* Source/WebCore/platform/mediastream/gstreamer/GStreamerMediaStreamSource.cpp:
* Source/WebCore/platform/mediastream/gstreamer/RealtimeIncomingSourceGStreamer.cpp:
(WebCore::RealtimeIncomingSourceGStreamer::RealtimeIncomingSourceGStreamer):
(WebCore::RealtimeIncomingSourceGStreamer::registerClient):
(WebCore::RealtimeIncomingSourceGStreamer::handleDownstreamEvent): Deleted.
* Source/WebCore/platform/mediastream/gstreamer/RealtimeIncomingSourceGStreamer.h:
(WebCore::RealtimeIncomingSourceGStreamer::dispatchSample):

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




More information about the webkit-changes mailing list