[webkit-changes] [WebKit/WebKit] 2f1c60: LibWebRTCSocket does no longer need to check for b...

youennf noreply at github.com
Tue Sep 5 10:25:26 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 2f1c609b53d2ae9b253324c9685f46cd5d5cb94a
      https://github.com/WebKit/WebKit/commit/2f1c609b53d2ae9b253324c9685f46cd5d5cb94a
  Author: Youenn Fablet <youennf at gmail.com>
  Date:   2023-09-05 (Tue, 05 Sep 2023)

  Changed paths:
    M Source/WebKit/WebProcess/Network/webrtc/LibWebRTCSocket.cpp
    M Source/WebKit/WebProcess/Network/webrtc/LibWebRTCSocket.h

  Log Message:
  -----------
  LibWebRTCSocket does no longer need to check for buffered data size
https://bugs.webkit.org/show_bug.cgi?id=261021
rdar://problem/114810113

Reviewed by Jean-Yves Avenard.

We used to check for buffered size to prevent excessive memory allocation in case the main thread is blocked.
We no longer need to do this since we are doing IPC straight from the networking thread.
The backpressure mechanism (via nw callback and libwebrtc SignalSentPacket) should be sufficient.
This might reduce a case where we would loose some UDP packets if there is a burst of packet sending on libwebrtc side and a slowdown in nw or network process.

* Source/WebKit/WebProcess/Network/webrtc/LibWebRTCSocket.cpp:
(WebKit::LibWebRTCSocket::signalSentPacket):
(WebKit::LibWebRTCSocket::SendTo):
(WebKit::LibWebRTCSocket::willSend): Deleted.
* Source/WebKit/WebProcess/Network/webrtc/LibWebRTCSocket.h:

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




More information about the webkit-changes mailing list