Let getSettings() return the constrained track resolution, not source.

content/renderer/media/media_stream_video_track.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_RENDERER_MEDIA_MEDIA_STREAM_VIDEO_TRACK_H_
 #define CONTENT_RENDERER_MEDIA_MEDIA_STREAM_VIDEO_TRACK_H_
 
 #include <vector>
 
 #include "base/compiler_specific.h"
@@ skipping 43 matching lines @@
   void SetEnabled(bool enabled) override;
   void SetContentHint(
       blink::WebMediaStreamTrack::ContentHintType content_hint) override;
   void Stop() override;
   void getSettings(blink::WebMediaStreamTrack::Settings& settings) override;
 
   void OnReadyStateChanged(blink::WebMediaStreamSource::ReadyState state);
 
   const blink::WebMediaConstraints& constraints() const { return constraints_; }
 
+  // Setting information about the track size.
+  // Called from MediaStreamVideoSource at track initialization.
+  void SetTargetSize(int width, int height) {

[Guido Urdaneta, 2017/03/13 14:12:37]

Is there a reason why the track cannot set these values itself during
initialization?

[hta - Chromium, 2017/03/13 15:04:35]

Yes - when it is created, I don't think the resolution is decided yet, and after
creation, it can't easily navigate the hierarchy of objects involved.

It is better for the track to be told by upstream what the resolution is than to
have the track ask the upstream about it.

The way it is set, these variables are empty if not set.
I also wanted to have the opportunty to change them if something changes
upstream (such as calling applyConstraints) - which requires a function anyway.

+    width_ = width;
+    height_ = height;
+  }
+
  private:
   // MediaStreamVideoSink is a friend to allow it to call AddSink() and
   // RemoveSink().
   friend class MediaStreamVideoSink;
   FRIEND_TEST_ALL_PREFIXES(MediaStreamRemoteVideoSourceTest, StartTrack);
   FRIEND_TEST_ALL_PREFIXES(MediaStreamRemoteVideoSourceTest, RemoteTrackStop);
   FRIEND_TEST_ALL_PREFIXES(PepperToVideoTrackAdapterTest, PutFrame);
 
   // Add |sink| to receive state changes on the main render thread and video
   // frames in the |callback| method on the IO-thread.
@@ skipping 14 matching lines @@
   const scoped_refptr<FrameDeliverer> frame_deliverer_;
 
   const blink::WebMediaConstraints constraints_;
 
   // Weak ref to the source this tracks is connected to.
   base::WeakPtr<MediaStreamVideoSource> source_;
 
   // This is used for tracking if all connected video sinks are secure.
   SecureDisplayLinkTracker<MediaStreamVideoSink> secure_tracker_;
 
+  // Remembering our desired video size.
+  int width_ = 0;
+  int height_ = 0;
+
   DISALLOW_COPY_AND_ASSIGN(MediaStreamVideoTrack);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_MEDIA_MEDIA_STREAM_VIDEO_TRACK_H_