zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix VideoTrack VideoSinkWants for renderers.

Browse Source 
This temporarily fixes a probem where renderers causes VideoSinkWants.rotation_applied=true.
The problem was introduced by https://codereview.webrtc.org/1759473003/ where VideTrackRenderes are registered to the cricket::VideoCapturer with default VideoSinkWants.

BUG=webrtc:5621

Review URL: https://codereview.webrtc.org/1764693004

Cr-Commit-Position: refs/heads/master@{#11871}
This commit is contained in:
perkj
2016-03-04 03:09:14 -08:00
committed by Commit bot
parent a2a49d9d9c
commit 78417cf7c0
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
webrtc/api/videotrack.cc
View File

@ -20,8 +20,14 @@ VideoTrack::VideoTrack(const std::string& label,
VideoSourceInterface* video_source)
: MediaStreamTrack<VideoTrackInterface>(label),
video_source_(video_source) {
// TODO(perkj): Sinks should register directly to the source so that
// VideoSinkWants can be applied correctly per sink. For now, |renderers_|
// must be able to apply rotation. Note that this is only actual renderers,
// not sinks that connect directly to cricket::VideoCapture.
rtc::VideoSinkWants wants;
wants.rotation_applied = false;
if (video_source_)
video_source_->AddOrUpdateSink(&renderers_, rtc::VideoSinkWants());
video_source_->AddOrUpdateSink(&renderers_, wants);
}
VideoTrack::~VideoTrack() {