Spec compliant video constraints for getUserMedia behind flag.

content/renderer/media/media_stream_center.cc

 // Copyright (c) 2012 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.
 
 #include "content/renderer/media/media_stream_center.h"
 
 #include <stddef.h>
 
 #include <string>
 
@@ skipping 52 matching lines @@
 
 void CreateNativeVideoMediaStreamTrack(
     const blink::WebMediaStreamTrack& track) {
   DCHECK(track.getTrackData() == NULL);
   blink::WebMediaStreamSource source = track.source();
   DCHECK_EQ(source.getType(), blink::WebMediaStreamSource::TypeVideo);
   MediaStreamVideoSource* native_source =
       MediaStreamVideoSource::GetVideoSource(source);
   DCHECK(native_source);
   blink::WebMediaStreamTrack writable_track(track);
-  // TODO(perkj): The constraints to use here should be passed from blink when
-  // a new track is created. For cloning, it should be the constraints of the
-  // cloned track and not the originating source.
-  // Also - source.constraints() returns an uninitialized constraint if the
-  // source is coming from a remote video track. See http://crbug/287805.
-  blink::WebMediaConstraints constraints = source.constraints();
-  if (constraints.isNull())
-    constraints.initialize();
-  writable_track.setTrackData(new MediaStreamVideoTrack(
-      native_source, constraints, MediaStreamVideoSource::ConstraintsCallback(),
-      track.isEnabled()));
+  if (IsOldVideoConstraints()) {
+    // TODO(perkj): The constraints to use here should be passed from blink when
+    // a new track is created. For cloning, it should be the constraints of the
+    // cloned track and not the originating source.
+    // Also - source.constraints() returns an uninitialized constraint if the
+    // source is coming from a remote video track. See http://crbug/287805.
+    blink::WebMediaConstraints constraints = source.constraints();
+    if (constraints.isNull())
+      constraints.initialize();
+    writable_track.setTrackData(new MediaStreamVideoTrack(
+        native_source, constraints,
+        MediaStreamVideoSource::ConstraintsCallback(), track.isEnabled()));
+  } else {
+    // TODO(guidou): All existing uses are without constraints except for the
+    // case of cloning a track. To fix this, create a new function in
+    // MediaStreamCenter specifically for cloning.
+    writable_track.setTrackData(new MediaStreamVideoTrack(
+        native_source, MediaStreamVideoSource::ConstraintsCallback(),
+        track.isEnabled()));
+  }
 }
 
 }  // namespace
 
 MediaStreamCenter::MediaStreamCenter(
     blink::WebMediaStreamCenterClient* client,
     PeerConnectionDependencyFactory* factory) {}
 
 MediaStreamCenter::~MediaStreamCenter() {}
 
@@ skipping 103 matching lines @@
 
 bool MediaStreamCenter::didRemoveMediaStreamTrack(
     const blink::WebMediaStream& stream,
     const blink::WebMediaStreamTrack& track) {
   DVLOG(1) << "MediaStreamCenter::didRemoveMediaStreamTrack";
   MediaStream* native_stream = MediaStream::GetMediaStream(stream);
   return native_stream->RemoveTrack(track);
 }
 
 }  // namespace content