Spec compliant video constraints for getUserMedia behind flag.

content/renderer/pepper/pepper_media_stream_video_track_host.cc

Index: content/renderer/pepper/pepper_media_stream_video_track_host.cc
diff --git a/content/renderer/pepper/pepper_media_stream_video_track_host.cc b/content/renderer/pepper/pepper_media_stream_video_track_host.cc
index bbded169a288bbc254843f120c2534356fd0a6f5..3daeb5a4fecf1b0fe251be83da9ce19c4130d857 100644
--- a/content/renderer/pepper/pepper_media_stream_video_track_host.cc
+++ b/content/renderer/pepper/pepper_media_stream_video_track_host.cc
@@ -443,6 +443,17 @@ class PepperMediaStreamVideoTrackHost::VideoSource final
   }
 
  private:
+  base::Optional<media::VideoCaptureFormat> GetCurrentFormatImpl()
+      const override {
+    DCHECK(!IsOldVideoConstraints());
+    if (host_) {
+      return media::VideoCaptureFormat(
+          host_->plugin_frame_size_, kDefaultOutputFrameRate,
+          ToPixelFormat(host_->plugin_frame_format_));
+    }
+    return base::Optional<media::VideoCaptureFormat>();
+  }

[hbos_chromium, 2017/04/05 12:37:28]

Is this used?

[Guido Urdaneta, 2017/04/05 16:17:17]

Yes. This is an override of a virtual method.
This value is stored in a variable in the original code, but the new code uses
the virtual method.

+
   const base::WeakPtr<PepperMediaStreamVideoTrackHost> host_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoSource);
@@ -522,14 +533,11 @@ void PepperMediaStreamVideoTrackHost::InitBlinkTrack() {
   webkit_source.setExtraData(source);  // Takes ownership of |source|.
 
   const bool enabled = true;
-  blink::WebMediaConstraints constraints;
-  constraints.initialize();
   track_ = MediaStreamVideoTrack::CreateVideoTrack(
-       source, constraints,
-       base::Bind(
-           &PepperMediaStreamVideoTrackHost::OnTrackStarted,
-           base::Unretained(this)),
-       enabled);
+      source,
+      base::Bind(&PepperMediaStreamVideoTrackHost::OnTrackStarted,
+                 base::Unretained(this)),
+      enabled);
   // Note: The call to CreateVideoTrack() returned a track that holds a
   // ref-counted reference to |webkit_source| (and, implicitly, |source|).
 }