Remove MediaTracks::getFirstAudio/VideoConfig

media/formats/mp4/mp4_stream_parser_unittest.cc

Index: media/formats/mp4/mp4_stream_parser_unittest.cc
diff --git a/media/formats/mp4/mp4_stream_parser_unittest.cc b/media/formats/mp4/mp4_stream_parser_unittest.cc
index 7109c20ddf6ac118f815d2c4b08634574448b46b..046ac996fab40ba34cf4266dafb26be3b24ee0cd 100644
--- a/media/formats/mp4/mp4_stream_parser_unittest.cc
+++ b/media/formats/mp4/mp4_stream_parser_unittest.cc
@@ -117,12 +117,26 @@ class MP4StreamParserTest : public testing::Test {
                   const StreamParser::TextTrackConfigMap& tc) {
     configs_received_ = true;
     CHECK(tracks.get());
+    DVLOG(1) << "NewConfigF: got " << tracks->tracks().size() << " tracks";
+    for (const auto& track : tracks->tracks()) {
+      const auto& track_id = track->bytestream_track_id();
+      if (track->type() == MediaTrack::Audio) {
+        const auto& audio_config = tracks->getAudioConfig(track_id);

[wolenetz, 2016/06/16 20:20:36]

nit: why not drop this local var and just assign to |audio_decoder_config_|
here? (and likewise for video, below)

[servolk, 2016/06/16 21:14:07]

Done.

+        DVLOG(1) << "Audio track " << track_id
+                 << " config=" << (audio_config.IsValidConfig()
+                                       ? audio_config.AsHumanReadableString()
+                                       : "INVALID");
+        audio_decoder_config_ = audio_config;
+      } else if (track->type() == MediaTrack::Video) {
+        const auto& video_config = tracks->getVideoConfig(track_id);
+        DVLOG(1) << "Video track " << track_id
+                 << " config=" << (video_config.IsValidConfig()
+                                       ? video_config.AsHumanReadableString()
+                                       : "INVALID");
+        video_decoder_config_ = video_config;
+      }
+    }
     media_tracks_ = std::move(tracks);
-    audio_decoder_config_ = media_tracks_->getFirstAudioConfig();
-    video_decoder_config_ = media_tracks_->getFirstVideoConfig();
-    DVLOG(1) << "NewConfigF: track count=" << media_tracks_->tracks().size()
-             << " audio=" << audio_decoder_config_.IsValidConfig()
-             << " video=" << video_decoder_config_.IsValidConfig();
     return true;
   }