Removed unnecessary VideoTrack creation code in MediaStreamCenter::didCreateMediaStream

content/renderer/media/media_stream_center.cc

Index: content/renderer/media/media_stream_center.cc
diff --git a/content/renderer/media/media_stream_center.cc b/content/renderer/media/media_stream_center.cc
index f5f5e7ad6d92a344fb83a66f118a0137f97f158d..ef76b054c098241b113a0b6ca10f6a31f9e986cb 100644
--- a/content/renderer/media/media_stream_center.cc
+++ b/content/renderer/media/media_stream_center.cc
@@ -165,16 +165,8 @@ void MediaStreamCenter::didStopLocalMediaStream(
 void MediaStreamCenter::didCreateMediaStream(blink::WebMediaStream& stream) {
   DVLOG(1) << "MediaStreamCenter::didCreateMediaStream";
   blink::WebMediaStream writable_stream(stream);
-  MediaStream* native_stream(
-      new MediaStream(stream));
+  MediaStream* native_stream(new MediaStream(stream));

[tommi (sloooow) - chröme, 2015/11/18 12:22:19]

nit: could even skip this variable.
writable_stream.setExtraData(new MediaStream(stream));

[perkj_chrome, 2015/11/19 09:39:18]

No, if a track later is added through didAddMediaStreamTrack, it must exist. 

[tommi (sloooow) - chröme, 2015/11/19 09:46:50]

I don't understand?  I'm just saying:

MediaStream* native_stream(new MediaStream(stream));
writable_stream.setExtraData(native_stream);
->
writable_stream.setExtraData(new MediaStream(stream));

Besides saving a line and reducing the scope of the temporary, what's the
difference?

   writable_stream.setExtraData(native_stream);
-
-  blink::WebVector<blink::WebMediaStreamTrack> video_tracks;
-  stream.videoTracks(video_tracks);
-  for (size_t i = 0; i < video_tracks.size(); ++i) {
-    if (!MediaStreamTrack::GetTrack(video_tracks[i]))
-      CreateNativeMediaStreamTrack(video_tracks[i], rtc_factory_);
-  }
 }
 
 bool MediaStreamCenter::didAddMediaStreamTrack(