Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 1 // Copyright 2013 The Chromium Authors. All rights reserved. | |
| 2 // Use of this source code is governed by a BSD-style license that can be | |
| 3 // found in the LICENSE file. | |
| 4 | |
| 5 #include "content/renderer/media/media_stream_video_track.h" | |
| 6 | |
| 7 #include "content/renderer/media/webrtc/webrtc_video_sink_adapter.h" | |
| 8 | |
| 9 namespace content { | |
| 10 | |
| 11 // Wrapper which allows to use std::find_if() when adding and removing | |
| 12 // sinks to/from |sinks_|. | |
| 13 struct SinkWrapper { | |
| 14 explicit SinkWrapper(MediaStreamVideoSink* sink) : sink_(sink) {} | |
| 15 bool operator()( | |
| 16 const WebRtcVideoSinkAdapter* owner) { | |
| 17 return owner->sink() == sink_; | |
| 18 } | |
| 19 MediaStreamVideoSink* sink_; | |
| 20 }; | |
| 21 | |
| 22 MediaStreamVideoTrack::MediaStreamVideoTrack(webrtc::VideoTrackInterface* track) | |
| 23 : MediaStreamTrackExtraData(track), | |
| 24 video_track_(track) { | |
| 25 } | |
| 26 | |
| 27 MediaStreamVideoTrack::~MediaStreamVideoTrack() { | |
| 28 DCHECK(sinks_.empty()); | |
| 29 } | |
| 30 | |
| 31 void MediaStreamVideoTrack::AddSink(MediaStreamVideoSink* sink) { | |
| 32 DCHECK(thread_checker_.CalledOnValidThread()); | |
| 33 DCHECK(std::find_if(sinks_.begin(), sinks_.end(), | |
| 34 SinkWrapper(sink)) == sinks_.end()); | |
| 35 sinks_.push_back(new WebRtcVideoSinkAdapter(video_track_, sink)); | |
| 36 } | |
| 37 | |
| 38 void MediaStreamVideoTrack::RemoveSink(MediaStreamVideoSink* sink) { | |
| 39 DCHECK(thread_checker_.CalledOnValidThread()); | |
| 40 ScopedVector<WebRtcVideoSinkAdapter>::iterator it = | |
| 41 std::find_if(sinks_.begin(), sinks_.end(), SinkWrapper(sink)); | |
| 42 DCHECK(it != sinks_.end()); | |
| 43 sinks_.erase(it); | |
|
no longer working on chromium
2013/11/28 15:20:46
To be more specific on the racing concern, I think
perkj_chrome
2013/11/29 13:29:48
Why would I need that? sinks_ is a scoped_vector ?
| |
| 44 } | |
| 45 | |
| 46 } // namespace content | |
| OLD | NEW |