Remove old-style constraints from Chrome internals

content/renderer/media/webrtc/media_stream_video_webrtc_sink.cc

Index: content/renderer/media/webrtc/media_stream_video_webrtc_sink.cc
diff --git a/content/renderer/media/webrtc/media_stream_video_webrtc_sink.cc b/content/renderer/media/webrtc/media_stream_video_webrtc_sink.cc
index 5cb5bac30e838f57870f8e17bb3fd8cc560df0c1..87f325315fabf51be2b3413d9edd929ff0a6d3f5 100644
--- a/content/renderer/media/webrtc/media_stream_video_webrtc_sink.cc
+++ b/content/renderer/media/webrtc/media_stream_video_webrtc_sink.cc
@@ -10,20 +10,10 @@
 #include "base/synchronization/lock.h"
 #include "base/thread_task_runner_handle.h"
 #include "content/common/media/media_stream_options.h"
+#include "content/renderer/media/media_stream_constraints_util.h"
 #include "content/renderer/media/media_stream_video_track.h"
 #include "content/renderer/media/webrtc/peer_connection_dependency_factory.h"
 
-namespace {
-
-bool ConstraintKeyExists(const blink::WebMediaConstraints& constraints,
-                         const blink::WebString& name) {
-  blink::WebString value_str;
-  return constraints.getMandatoryConstraintValue(name, value_str) ||
-      constraints.getOptionalConstraintValue(name, value_str);
-}
-
-}  // anonymouse namespace
-
 namespace content {
 
 // Simple help class used for receiving video frames on the IO-thread from a
@@ -138,15 +128,16 @@ MediaStreamVideoWebRtcSink::MediaStreamVideoWebRtcSink(
   const blink::WebMediaConstraints& constraints =
       MediaStreamVideoTrack::GetVideoTrack(track)->constraints();
 
-  bool is_screencast = ConstraintKeyExists(
-      constraints, base::UTF8ToUTF16(kMediaStreamSource));
+  std::string value;
+  bool is_screencast = GetConstraintValueAsString(
+      constraints, &blink::WebMediaTrackConstraintSet::mediaStreamSource,
+      &value);

[tommi (sloooow) - chröme, 2016/03/14 18:37:47]

add a comment that explains why we don't use |value|?

[hta - Chromium, 2016/03/15 06:47:33]

Done.

   WebRtcVideoCapturerAdapter* capture_adapter =
       factory->CreateVideoCapturer(is_screencast);
 
   // |video_source| owns |capture_adapter|
   scoped_refptr<webrtc::VideoTrackSourceInterface> video_source(
-      factory->CreateVideoSource(capture_adapter,
-                                 track.source().constraints()));
+      factory->CreateVideoSource(capture_adapter));
 
   video_track_ = factory->CreateLocalVideoTrack(web_track_.id().utf8(),
                                                 video_source.get());