Chromium Code Reviews| Index: media/base/media_tracks.cc |
| diff --git a/media/base/media_tracks.cc b/media/base/media_tracks.cc |
| index 770ab8dc7d4ed62110e0682def544dbfb517612f..afc8174328975abbc4a550dc2e9aa2e6c6c91309 100644 |
| --- a/media/base/media_tracks.cc |
| +++ b/media/base/media_tracks.cc |
| @@ -15,30 +15,32 @@ MediaTracks::MediaTracks() {} |
| MediaTracks::~MediaTracks() {} |
| -void MediaTracks::AddAudioTrack(const AudioDecoderConfig& config, |
| - StreamParser::TrackId byteStreamTrackId, |
| - const std::string& kind, |
| - const std::string& label, |
| - const std::string& language) { |
| +MediaTrack* MediaTracks::AddAudioTrack(const AudioDecoderConfig& config, |
| + StreamParser::TrackId byteStreamTrackId, |
| + const std::string& kind, |
| + const std::string& label, |
| + const std::string& language) { |
| DCHECK(config.IsValidConfig()); |
| CHECK(audio_configs_.find(byteStreamTrackId) == audio_configs_.end()); |
| std::unique_ptr<MediaTrack> track = base::WrapUnique(new MediaTrack( |
| MediaTrack::Audio, byteStreamTrackId, kind, label, language)); |
| tracks_.push_back(std::move(track)); |
| audio_configs_[byteStreamTrackId] = config; |
| + return tracks_.back().get(); |
| } |
| -void MediaTracks::AddVideoTrack(const VideoDecoderConfig& config, |
| - StreamParser::TrackId byteStreamTrackId, |
| - const std::string& kind, |
| - const std::string& label, |
| - const std::string& language) { |
| +MediaTrack* MediaTracks::AddVideoTrack(const VideoDecoderConfig& config, |
| + StreamParser::TrackId byteStreamTrackId, |
| + const std::string& kind, |
| + const std::string& label, |
| + const std::string& language) { |
| DCHECK(config.IsValidConfig()); |
| CHECK(video_configs_.find(byteStreamTrackId) == video_configs_.end()); |
| std::unique_ptr<MediaTrack> track = base::WrapUnique(new MediaTrack( |
| MediaTrack::Video, byteStreamTrackId, kind, label, language)); |
| tracks_.push_back(std::move(track)); |
| video_configs_[byteStreamTrackId] = config; |
| + return tracks_.back().get(); |
|
xhwang
2016/06/09 06:06:02
nit: You can store a raw pointer of track.get() ab
servolk
2016/06/09 19:14:46
Done.
|
| } |
| const AudioDecoderConfig& MediaTracks::getAudioConfig( |