66 lines
3.3 KiB
Diff
66 lines
3.3 KiB
Diff
From: Byron Campen <docfaraday@gmail.com>
|
|
Date: Thu, 20 Jul 2023 14:24:00 +0000
|
|
Subject: Bug 1838080: Use the current TaskQueue, instead of the current
|
|
thread, to init this. r=pehrsons,webrtc-reviewers
|
|
|
|
There are situations where the current thread is not set, but the current
|
|
TaskQueue is (but not vice versa).
|
|
|
|
Differential Revision: https://phabricator.services.mozilla.com/D180736
|
|
Mercurial Revision: https://hg.mozilla.org/mozilla-central/rev/62e71a2f745c4b98d5ee7ce9e6386aa1b657be9b
|
|
---
|
|
.../rtp_video_stream_receiver_frame_transformer_delegate.cc | 3 +--
|
|
.../rtp_video_stream_receiver_frame_transformer_delegate.h | 5 ++---
|
|
video/rtp_video_stream_receiver2.cc | 2 +-
|
|
3 files changed, 4 insertions(+), 6 deletions(-)
|
|
|
|
diff --git a/modules/rtp_rtcp/source/rtp_video_stream_receiver_frame_transformer_delegate.cc b/modules/rtp_rtcp/source/rtp_video_stream_receiver_frame_transformer_delegate.cc
|
|
index 918143bc43..c736cc647d 100644
|
|
--- a/modules/rtp_rtcp/source/rtp_video_stream_receiver_frame_transformer_delegate.cc
|
|
+++ b/modules/rtp_rtcp/source/rtp_video_stream_receiver_frame_transformer_delegate.cc
|
|
@@ -123,8 +123,7 @@ RtpVideoStreamReceiverFrameTransformerDelegate::
|
|
RtpVideoFrameReceiver* receiver,
|
|
Clock* clock,
|
|
rtc::scoped_refptr<FrameTransformerInterface> frame_transformer,
|
|
- rtc::Thread* network_thread,
|
|
- uint32_t ssrc)
|
|
+ TaskQueueBase* network_thread, uint32_t ssrc)
|
|
: receiver_(receiver),
|
|
frame_transformer_(std::move(frame_transformer)),
|
|
network_thread_(network_thread),
|
|
diff --git a/modules/rtp_rtcp/source/rtp_video_stream_receiver_frame_transformer_delegate.h b/modules/rtp_rtcp/source/rtp_video_stream_receiver_frame_transformer_delegate.h
|
|
index f08fc692dd..02f2e53923 100644
|
|
--- a/modules/rtp_rtcp/source/rtp_video_stream_receiver_frame_transformer_delegate.h
|
|
+++ b/modules/rtp_rtcp/source/rtp_video_stream_receiver_frame_transformer_delegate.h
|
|
@@ -41,8 +41,7 @@ class RtpVideoStreamReceiverFrameTransformerDelegate
|
|
RtpVideoFrameReceiver* receiver,
|
|
Clock* clock,
|
|
rtc::scoped_refptr<FrameTransformerInterface> frame_transformer,
|
|
- rtc::Thread* network_thread,
|
|
- uint32_t ssrc);
|
|
+ TaskQueueBase* network_thread, uint32_t ssrc);
|
|
|
|
void Init();
|
|
void Reset();
|
|
@@ -71,7 +70,7 @@ class RtpVideoStreamReceiverFrameTransformerDelegate
|
|
RtpVideoFrameReceiver* receiver_ RTC_GUARDED_BY(network_sequence_checker_);
|
|
rtc::scoped_refptr<FrameTransformerInterface> frame_transformer_
|
|
RTC_GUARDED_BY(network_sequence_checker_);
|
|
- rtc::Thread* const network_thread_;
|
|
+ TaskQueueBase* const network_thread_;
|
|
const uint32_t ssrc_;
|
|
Clock* const clock_;
|
|
bool short_circuit_ RTC_GUARDED_BY(network_sequence_checker_) = false;
|
|
diff --git a/video/rtp_video_stream_receiver2.cc b/video/rtp_video_stream_receiver2.cc
|
|
index f99e0d4b45..13bdba2583 100644
|
|
--- a/video/rtp_video_stream_receiver2.cc
|
|
+++ b/video/rtp_video_stream_receiver2.cc
|
|
@@ -341,7 +341,7 @@ RtpVideoStreamReceiver2::RtpVideoStreamReceiver2(
|
|
frame_transformer_delegate_ =
|
|
rtc::make_ref_counted<RtpVideoStreamReceiverFrameTransformerDelegate>(
|
|
this, &env_.clock(), std::move(frame_transformer),
|
|
- rtc::Thread::Current(), config_.rtp.remote_ssrc);
|
|
+ TaskQueueBase::Current(), config_.rtp.remote_ssrc);
|
|
frame_transformer_delegate_->Init();
|
|
}
|
|
}
|