Wip - cl for implemening VideoTrackSource::ApplyConstraints(constraints, VideoTrack) and adopt safe…

content/renderer/media/remote_media_stream_impl.cc

 // Copyright (c) 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.
 
 #include "content/renderer/media/remote_media_stream_impl.h"
 
 #include <stddef.h>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 142 matching lines @@
   }
 
  private:
   void InitializeWebkitVideoTrack(std::unique_ptr<TrackObserver> observer,
                                   bool enabled) {
     DCHECK(main_thread_->BelongsToCurrentThread());
     std::unique_ptr<MediaStreamRemoteVideoSource> video_source(
         new MediaStreamRemoteVideoSource(std::move(observer)));
     InitializeWebkitTrack(blink::WebMediaStreamSource::TypeVideo);
     webkit_track()->source().setExtraData(video_source.get());
+    MediaStreamVideoTrack::InitializeTrack(video_source.release(),
+                                           *webkit_track());
     // Initial constraints must be provided to a MediaStreamVideoTrack. But
     // no constraints are available initially on a remote video track.
-    blink::WebMediaConstraints constraints;
+/*    blink::WebMediaConstraints constraints;
     constraints.initialize();
     MediaStreamVideoTrack* media_stream_track =
         new MediaStreamVideoTrack(video_source.release(), constraints,
             MediaStreamVideoSource::ConstraintsCallback(), enabled);
-    webkit_track()->setTrackData(media_stream_track);
+    webkit_track()->setTrackData(media_stream_track);*/
   }
 };
 
 // RemoteAudioTrackAdapter is responsible for listening on state
 // change notifications on a remote webrtc audio MediaStreamTracks and notify
 // WebKit.
 class RemoteAudioTrackAdapter
     : public RemoteMediaStreamTrackAdapter<webrtc::AudioTrackInterface>,
       public webrtc::ObserverInterface {
  public:
@@ skipping 242 matching lines @@
   // Unregister all the audio track observers that were not used.
   // We need to do this before destruction since the observers can't unregister
   // from within the dtor due to a race.
   for (auto& track : *audio_tracks.get()) {
     if (track.get())
       track->Unregister();
   }
 }
 
 }  // namespace content