Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Commit 509171c

Browse files
committed
Bug 1892299 - Vendor libwebrtc from d99499abba
Essentially a no-op since we're going to see this change reverted when we vendor in 7098d110e1. Upstream commit: https://webrtc.googlesource.com/src/+/d99499abbae94793a02944a1f28f7015816447f5 p2p: separate ICE tie breaker and foundation seed BUG=webrtc:14626 Change-Id: I189a708192c9cef0b50c3fcbe798b30376d3b547 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/338982 Reviewed-by: Tomas Gunnarsson <[email protected]> Commit-Queue: Philipp Hancke <[email protected]> Reviewed-by: Harald Alvestrand <[email protected]> Cr-Commit-Position: refs/heads/main@{#41806}
1 parent 9726885 commit 509171c

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

112 files changed

+45831
-45532
lines changed

third_party/libwebrtc/README.moz-ff-commit

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29172,3 +29172,6 @@ c7a4b2a7eb
2917229172
# MOZ_LIBWEBRTC_SRC=/Users/ng/dev/elm/.moz-fast-forward/moz-libwebrtc MOZ_LIBWEBRTC_BRANCH=mozpatches bash dom/media/webrtc/third_party_build/fast-forward-libwebrtc.sh
2917329173
# base of lastest vendoring
2917429174
7c5f9cf47f
29175+
# MOZ_LIBWEBRTC_SRC=/Users/ng/dev/elm/.moz-fast-forward/moz-libwebrtc MOZ_LIBWEBRTC_BRANCH=mozpatches bash dom/media/webrtc/third_party_build/fast-forward-libwebrtc.sh
29176+
# base of lastest vendoring
29177+
d99499abba

third_party/libwebrtc/README.mozilla

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19472,3 +19472,5 @@ libwebrtc updated from /Users/ng/dev/elm/.moz-fast-forward/moz-libwebrtc commit
1947219472
libwebrtc updated from /Users/ng/dev/elm/.moz-fast-forward/moz-libwebrtc commit mozpatches on 2024-04-26T08:09:05.396473.
1947319473
# ./mach python dom/media/webrtc/third_party_build/vendor-libwebrtc.py --from-local /Users/ng/dev/elm/.moz-fast-forward/moz-libwebrtc --commit mozpatches libwebrtc
1947419474
libwebrtc updated from /Users/ng/dev/elm/.moz-fast-forward/moz-libwebrtc commit mozpatches on 2024-04-26T08:10:12.207928.
19475+
# ./mach python dom/media/webrtc/third_party_build/vendor-libwebrtc.py --from-local /Users/ng/dev/elm/.moz-fast-forward/moz-libwebrtc --commit mozpatches libwebrtc
19476+
libwebrtc updated from /Users/ng/dev/elm/.moz-fast-forward/moz-libwebrtc commit mozpatches on 2024-04-26T16:16:56.695304.

third_party/libwebrtc/moz-patch-stack/0001.patch

Lines changed: 270 additions & 1670 deletions
Large diffs are not rendered by default.

third_party/libwebrtc/moz-patch-stack/0002.patch

Lines changed: 1677 additions & 27 deletions
Large diffs are not rendered by default.
Lines changed: 37 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,34 +1,43 @@
1-
From: Michael Froman <mfroman@mozilla.com>
2-
Date: Fri, 9 Jul 2021 18:14:00 -0500
3-
Subject: Bug 1654112 - mutex changes to fix tsan errors. r=ng
1+
From: Nico Grunbaum <na-g@nostrum.com>
2+
Date: Sat, 13 Feb 2021 04:20:00 -0800
3+
Subject: Bug 1654112 - Add pid_t to desktop_capture_types.h; r=pehrsons
44

5-
Differential Revision: https://phabricator.services.mozilla.com/D119674
6-
Mercurial Revision: https://hg.mozilla.org/mozilla-central/rev/3d5503acf9a4b22e02c4300f29e4fbfed406ea2c
5+
Upstreaming bug 1697385
6+
7+
Also includes:
8+
Bug 1654112 - Clarifying prev. rev that moved pid_t into the global namespace; r=dminor
9+
10+
Differential Revision: https://phabricator.services.mozilla.com/D107897
11+
Mercurial Revision: https://hg.mozilla.org/mozilla-central/rev/539b69f0e21118a1898f1ef689577c7193ff7be7
712
---
8-
rtc_base/logging.cc | 5 ++++-
9-
1 file changed, 4 insertions(+), 1 deletion(-)
13+
modules/desktop_capture/desktop_capture_types.h | 4 ++++
14+
modules/video_capture/video_capture.h | 1 +
15+
2 files changed, 5 insertions(+)
1016

11-
diff --git a/rtc_base/logging.cc b/rtc_base/logging.cc
12-
index ad2303735e..9c2d3b0a39 100644
13-
--- a/rtc_base/logging.cc
14-
+++ b/rtc_base/logging.cc
15-
@@ -275,8 +275,8 @@ void LogMessage::LogTimestamps(bool on) {
16-
}
17+
diff --git a/modules/desktop_capture/desktop_capture_types.h b/modules/desktop_capture/desktop_capture_types.h
18+
index a4e3e897fd..e777a45f92 100644
19+
--- a/modules/desktop_capture/desktop_capture_types.h
20+
+++ b/modules/desktop_capture/desktop_capture_types.h
21+
@@ -13,6 +13,10 @@
22+
23+
#include <stdint.h>
1724

18-
void LogMessage::LogToDebug(LoggingSeverity min_sev) {
19-
- g_dbg_sev = min_sev;
20-
webrtc::MutexLock lock(&GetLoggingLock());
21-
+ g_dbg_sev = min_sev;
22-
UpdateMinLogSeverity();
23-
}
25+
+#ifdef XP_WIN // Moving this into the global namespace
26+
+typedef int pid_t; // matching what used to be in
27+
+#endif // video_capture_defines.h
28+
+
29+
namespace webrtc {
2430

25-
@@ -460,6 +460,9 @@ void LogMessage::OutputToDebug(const LogLineRef& log_line) {
31+
enum class CaptureType { kWindow, kScreen, kAnyScreenContent };
32+
diff --git a/modules/video_capture/video_capture.h b/modules/video_capture/video_capture.h
33+
index f31b239c71..7279bed476 100644
34+
--- a/modules/video_capture/video_capture.h
35+
+++ b/modules/video_capture/video_capture.h
36+
@@ -13,6 +13,7 @@
2637

27-
// static
28-
bool LogMessage::IsNoop(LoggingSeverity severity) {
29-
+ // Added MutexLock to fix tsan warnings on accessing g_dbg_sev. (mjf)
30-
+ // See https://bugs.chromium.org/p/chromium/issues/detail?id=1228729
31-
+ webrtc::MutexLock lock(&GetLoggingLock());
32-
if (severity >= g_dbg_sev || severity >= g_min_sev)
33-
return false;
34-
return streams_empty_.load(std::memory_order_relaxed);
38+
#include "api/video/video_rotation.h"
39+
#include "api/video/video_sink_interface.h"
40+
+#include "modules/desktop_capture/desktop_capture_types.h"
41+
#include "modules/video_capture/raw_video_sink_interface.h"
42+
#include "modules/video_capture/video_capture_defines.h"
43+
#include <set>
Lines changed: 28 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -1,46 +1,34 @@
1-
From: Dan Minor <dminor@mozilla.com>
2-
Date: Wed, 18 Nov 2020 13:33:00 -0500
3-
Subject: Bug 1654112 - Suppress -Wclass-varargs warning in logging.h. r=ng
1+
From: Michael Froman <mfroman@mozilla.com>
2+
Date: Fri, 9 Jul 2021 18:14:00 -0500
3+
Subject: Bug 1654112 - mutex changes to fix tsan errors. r=ng
44

5-
This needs some investigation to see why we get this warning when it is not
6-
present upstream.
7-
8-
Since both were doing the same thing for different compiler chains,
9-
also includes:
10-
Bug 1654112 - linux build fix (pragmas) for base-toolchains* . r=ng
11-
12-
Differential Revision: https://phabricator.services.mozilla.com/D130086
13-
Mercurial Revision: https://hg.mozilla.org/mozilla-central/rev/8d832e832ffe513246c0763a56376a8022b2447b
5+
Differential Revision: https://phabricator.services.mozilla.com/D119674
6+
Mercurial Revision: https://hg.mozilla.org/mozilla-central/rev/3d5503acf9a4b22e02c4300f29e4fbfed406ea2c
147
---
15-
rtc_base/logging.h | 13 +++++++++++++
16-
1 file changed, 13 insertions(+)
8+
rtc_base/logging.cc | 5 ++++-
9+
1 file changed, 4 insertions(+), 1 deletion(-)
1710

18-
diff --git a/rtc_base/logging.h b/rtc_base/logging.h
19-
index df7f173f58..a3733d7543 100644
20-
--- a/rtc_base/logging.h
21-
+++ b/rtc_base/logging.h
22-
@@ -48,6 +48,14 @@
23-
#ifndef RTC_BASE_LOGGING_H_
24-
#define RTC_BASE_LOGGING_H_
25-
26-
+#pragma GCC diagnostic push
27-
+#pragma GCC diagnostic ignored "-Wvarargs"
28-
+
29-
+#if defined(__clang__)
30-
+# pragma clang diagnostic push
31-
+# pragma clang diagnostic ignored "-Wclass-varargs"
32-
+#endif
33-
+
34-
#include <errno.h>
11+
diff --git a/rtc_base/logging.cc b/rtc_base/logging.cc
12+
index ad2303735e..9c2d3b0a39 100644
13+
--- a/rtc_base/logging.cc
14+
+++ b/rtc_base/logging.cc
15+
@@ -275,8 +275,8 @@ void LogMessage::LogTimestamps(bool on) {
16+
}
3517

36-
#include <atomic>
37-
@@ -764,4 +772,9 @@ inline const char* AdaptString(const std::string& str) {
18+
void LogMessage::LogToDebug(LoggingSeverity min_sev) {
19+
- g_dbg_sev = min_sev;
20+
webrtc::MutexLock lock(&GetLoggingLock());
21+
+ g_dbg_sev = min_sev;
22+
UpdateMinLogSeverity();
23+
}
3824

39-
} // namespace rtc
25+
@@ -460,6 +460,9 @@ void LogMessage::OutputToDebug(const LogLineRef& log_line) {
4026

41-
+#pragma GCC diagnostic pop
42-
+#if defined(__clang__)
43-
+# pragma clang diagnostic pop
44-
+#endif
45-
+
46-
#endif // RTC_BASE_LOGGING_H_
27+
// static
28+
bool LogMessage::IsNoop(LoggingSeverity severity) {
29+
+ // Added MutexLock to fix tsan warnings on accessing g_dbg_sev. (mjf)
30+
+ // See https://bugs.chromium.org/p/chromium/issues/detail?id=1228729
31+
+ webrtc::MutexLock lock(&GetLoggingLock());
32+
if (severity >= g_dbg_sev || severity >= g_min_sev)
33+
return false;
34+
return streams_empty_.load(std::memory_order_relaxed);
Lines changed: 37 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -1,49 +1,46 @@
11
From: Dan Minor <[email protected]>
2-
Date: Tue, 27 Mar 2018 15:43:00 -0400
3-
Subject: Bug 1376873 - Disable Mid support in RtpDemuxer; r=mjf
2+
Date: Wed, 18 Nov 2020 13:33:00 -0500
3+
Subject: Bug 1654112 - Suppress -Wclass-varargs warning in logging.h. r=ng
44

5-
The only use of Mid in the current webrtc.org code is in the unit tests.
6-
RtpStreamReceiverController only allows adding sinks using SSRCs. Because
7-
of this, we'll end up dropping packets in the RtpDemuxer with the current
8-
code as none of our Mids will be recognized.
5+
This needs some investigation to see why we get this warning when it is not
6+
present upstream.
97

10-
Tip of webrtc.org fully supports using Mids, so we'll be able to enable this
11-
code again after the next update.
8+
Since both were doing the same thing for different compiler chains,
9+
also includes:
10+
Bug 1654112 - linux build fix (pragmas) for base-toolchains* . r=ng
1211

13-
Differential Revision: https://phabricator.services.mozilla.com/D7442
14-
Mercurial Revision: https://hg.mozilla.org/mozilla-central/rev/b3ba8452e77105c72f6ddbc49cbe5a53dbea1507
12+
Differential Revision: https://phabricator.services.mozilla.com/D130086
13+
Mercurial Revision: https://hg.mozilla.org/mozilla-central/rev/8d832e832ffe513246c0763a56376a8022b2447b
1514
---
16-
call/rtp_demuxer.cc | 7 ++++++-
17-
1 file changed, 6 insertions(+), 1 deletion(-)
15+
rtc_base/logging.h | 13 +++++++++++++
16+
1 file changed, 13 insertions(+)
1817

19-
diff --git a/call/rtp_demuxer.cc b/call/rtp_demuxer.cc
20-
index 841d7e3b94..f5c4186871 100644
21-
--- a/call/rtp_demuxer.cc
22-
+++ b/call/rtp_demuxer.cc
23-
@@ -285,13 +285,17 @@ RtpPacketSinkInterface* RtpDemuxer::ResolveSink(
24-
// RSID and RRID are routed to the same sinks. If an RSID is specified on a
25-
// repair packet, it should be ignored and the RRID should be used.
26-
std::string packet_mid, packet_rsid;
27-
- bool has_mid = use_mid_ && packet.GetExtension<RtpMid>(&packet_mid);
28-
+ //bool has_mid = use_mid_ && packet.GetExtension<RtpMid>(&packet_mid);
29-
bool has_rsid = packet.GetExtension<RepairedRtpStreamId>(&packet_rsid);
30-
if (!has_rsid) {
31-
has_rsid = packet.GetExtension<RtpStreamId>(&packet_rsid);
32-
}
33-
uint32_t ssrc = packet.Ssrc();
18+
diff --git a/rtc_base/logging.h b/rtc_base/logging.h
19+
index df7f173f58..a3733d7543 100644
20+
--- a/rtc_base/logging.h
21+
+++ b/rtc_base/logging.h
22+
@@ -48,6 +48,14 @@
23+
#ifndef RTC_BASE_LOGGING_H_
24+
#define RTC_BASE_LOGGING_H_
3425

35-
+ // Mid support is half-baked in branch 64. RtpStreamReceiverController only
36-
+ // supports adding sinks by ssrc, so our mids will never show up in
37-
+ // known_mids_, causing us to drop packets here.
38-
+#if 0
39-
// The BUNDLE spec says to drop any packets with unknown MIDs, even if the
40-
// SSRC is known/latched.
41-
if (has_mid && known_mids_.find(packet_mid) == known_mids_.end()) {
42-
@@ -365,6 +369,7 @@ RtpPacketSinkInterface* RtpDemuxer::ResolveSink(
43-
}
44-
}
26+
+#pragma GCC diagnostic push
27+
+#pragma GCC diagnostic ignored "-Wvarargs"
28+
+
29+
+#if defined(__clang__)
30+
+# pragma clang diagnostic push
31+
+# pragma clang diagnostic ignored "-Wclass-varargs"
32+
+#endif
33+
+
34+
#include <errno.h>
35+
36+
#include <atomic>
37+
@@ -764,4 +772,9 @@ inline const char* AdaptString(const std::string& str) {
38+
39+
} // namespace rtc
4540

41+
+#pragma GCC diagnostic pop
42+
+#if defined(__clang__)
43+
+# pragma clang diagnostic pop
4644
+#endif
47-
// We trust signaled SSRC more than payload type which is likely to conflict
48-
// between streams.
49-
const auto ssrc_sink_it = sink_by_ssrc_.find(ssrc);
45+
+
46+
#endif // RTC_BASE_LOGGING_H_

0 commit comments

Comments
 (0)