Clean up WebSourceBufferClient interface

media/blink/websourcebuffer_impl.cc

Index: media/blink/websourcebuffer_impl.cc
diff --git a/media/blink/websourcebuffer_impl.cc b/media/blink/websourcebuffer_impl.cc
index 9386f5222eca6a9145c7f5e53e6853ebebfa37ca..feb8c5b5adcf07a74d479f32ffb68d73e89a7d3f 100644
--- a/media/blink/websourcebuffer_impl.cc
+++ b/media/blink/websourcebuffer_impl.cc
@@ -179,15 +179,16 @@ void WebSourceBufferImpl::InitSegmentReceived(scoped_ptr<MediaTracks> tracks) {
 
   std::vector<blink::WebSourceBufferClient::MediaTrackInfo> trackInfoVector;
   for (const auto& track : tracks->tracks()) {
-    trackInfoVector.push_back(
-        std::make_tuple(mediaTrackTypeToBlink(track->type()),
-                        blink::WebString::fromUTF8(track->id()),
-                        blink::WebString::fromUTF8(track->kind()),
-                        blink::WebString::fromUTF8(track->label()),
-                        blink::WebString::fromUTF8(track->language())));
+    blink::WebSourceBufferClient::MediaTrackInfo trackInfo;
+    trackInfo.trackType = mediaTrackTypeToBlink(track->type());
+    trackInfo.byteStreamTrackId = blink::WebString::fromUTF8(track->id());
+    trackInfo.kind = blink::WebString::fromUTF8(track->kind());
+    trackInfo.label = blink::WebString::fromUTF8(track->label());
+    trackInfo.language = blink::WebString::fromUTF8(track->language());
+    trackInfoVector.push_back(trackInfo);
   }
 
-  std::vector<blink::WebMediaPlayer::TrackId> blinkTrackIds =
+  blink::WebVector<blink::WebMediaPlayer::TrackId> blinkTrackIds =

[esprehn, 2016/04/15 20:56:00]

Whats the purpose of this if we just ignore the value?

[servolk, 2016/04/15 21:02:32]

This information is necessary to be able to map blink track id values to
DemuxerStream* pointers in media::Pipeline. The previous CL that added this code
implemented plumbing to get media track info to blink and then this set of track
ids out of blink. The next CL that uses this set of track ids has not landed yet
(although it will hopefully land soon, see
https://codereview.chromium.org/1812543003/)

[haraken, 2016/04/16 01:06:02]

I'd suggest you add the |blinkTrackIds| in the next CL that actually uses it.

       client_->initializationSegmentReceived(trackInfoVector);
 }