Fix VideoTrack VideoSinkWants for renderers.

webrtc/api/videotrack.cc

 /*
  *  Copyright 2011 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
 #include "webrtc/api/videotrack.h"
 
 #include <string>
 
 namespace webrtc {
 
 const char MediaStreamTrackInterface::kVideoKind[] = "video";
 
 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;

[nisse-webrtc, 2016/03/04 10:08:14]

nit: Could move the declaration and init a block under the if.

   if (video_source_)
-    video_source_->AddOrUpdateSink(&renderers_, rtc::VideoSinkWants());
+    video_source_->AddOrUpdateSink(&renderers_, wants);
 }
 
 VideoTrack::~VideoTrack() {
   if (video_source_)
     video_source_->RemoveSink(&renderers_);
 }
 
 std::string VideoTrack::kind() const {
   return kVideoKind;
 }
@@ skipping 20 matching lines @@
 
 rtc::scoped_refptr<VideoTrack> VideoTrack::Create(
     const std::string& id,
     VideoSourceInterface* source) {
   rtc::RefCountedObject<VideoTrack>* track =
       new rtc::RefCountedObject<VideoTrack>(id, source);
   return track;
 }
 
 }  // namespace webrtc