Commit 165f26ee authored by Sergey Volk's avatar Sergey Volk Committed by Commit Bot

Add a lock in ChunkDemuxer::SetStreamStatusChangeCB

This method accesses audio/video_streams_ collections and must acquire
the |lock_| since those collections might be modified from a different
thread.

BUG=726526

Change-Id: Ibb9ab848612f9bace1321aed56a4d22706ab7bea
Reviewed-on: https://chromium-review.googlesource.com/517226Reviewed-by: default avatarXiaohan Wang <xhwang@chromium.org>
Reviewed-by: default avatarMatthew Wolenetz <wolenetz@chromium.org>
Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org>
Commit-Queue: Sergey Volk <servolk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#476462}
parent 0d87c5aa
......@@ -534,6 +534,7 @@ std::vector<DemuxerStream*> ChunkDemuxer::GetAllStreams() {
}
void ChunkDemuxer::SetStreamStatusChangeCB(const StreamStatusChangeCB& cb) {
base::AutoLock auto_lock(lock_);
DCHECK(!cb.is_null());
for (const auto& stream : audio_streams_)
stream->SetStreamStatusChangeCB(cb);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment