admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Removed outdated comment of Notifier

Browse Source 
Notifier is thread-hostile, and we have added a SequenceChecker
on https://webrtc-review.googlesource.com/c/src/+/252520 ,
so this comment is no longer needed.

Bug: None
Change-Id: I39f7f75a786dd27d2f6299d85676e7182d9032eb
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/288180
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Auto-Submit: Daniel.L (Byoungchan) Lee <daniel.l@hpcnt.com>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#38899}
This commit is contained in:
Byoungchan Lee 2022-12-15 12:31:29 +09:00 committed by WebRTC LUCI CQ
parent f00483206d
commit 4a680f11ae
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
sdk/android/src/jni/android_video_track_source.cc
View File

@ -65,8 +65,6 @@ void AndroidVideoTrackSource::SetState(JNIEnv* env,
if (rtc::Thread::Current() == signaling_thread_) { if (rtc::Thread::Current() == signaling_thread_) {
FireOnChanged(); FireOnChanged();
} else { } else {
// TODO(sakal): Is this even necessary, does FireOnChanged have to be
// called from signaling thread?
signaling_thread_->PostTask([this] { FireOnChanged(); }); signaling_thread_->PostTask([this] { FireOnChanged(); });
} }
} }