Chromium Code Reviews| Index: media/mp3/mp3_stream_parser_unittest.cc |
| diff --git a/media/mp3/mp3_stream_parser_unittest.cc b/media/mp3/mp3_stream_parser_unittest.cc |
| index 2e2b12e1246435071e72c6bf8b872a823cefa2d7..533a471de309b2681e7dd6eb9b56d8c389876cf2 100644 |
| --- a/media/mp3/mp3_stream_parser_unittest.cc |
| +++ b/media/mp3/mp3_stream_parser_unittest.cc |
| @@ -7,6 +7,7 @@ |
| #include "media/base/decoder_buffer.h" |
| #include "media/base/stream_parser_buffer.h" |
| #include "media/base/test_data_util.h" |
| +#include "media/base/text_track_config.h" |
| #include "media/base/video_decoder_config.h" |
| #include "media/mp3/mp3_stream_parser.h" |
| #include "testing/gtest/include/gtest/gtest.h" |
| @@ -44,7 +45,8 @@ class MP3StreamParserTest : public testing::Test { |
| } |
| bool OnNewConfig(const AudioDecoderConfig& audio_config, |
| - const VideoDecoderConfig& video_config) { |
| + const VideoDecoderConfig& video_config, |
| + const TextTrackConfigMap& text_config) { |
| DVLOG(1) << __FUNCTION__ << "(" << audio_config.IsValidConfig() << ", " |
| << video_config.IsValidConfig() << ")"; |
| EXPECT_TRUE(audio_config.IsValidConfig()); |
| @@ -79,7 +81,7 @@ class MP3StreamParserTest : public testing::Test { |
| return true; |
| } |
| - bool OnNewTextBuffers(TextTrack* text_track, |
| + bool OnNewTextBuffers(int text_track, |
|
acolwell GONE FROM CHROMIUM
2013/10/29 21:14:19
nit: Remove since you are removing the reference b
Matthew Heaney (Chromium)
2013/11/05 04:52:40
Done.
|
| const StreamParser::BufferQueue& buffers) { |
| return true; |
| } |
| @@ -89,12 +91,6 @@ class MP3StreamParserTest : public testing::Test { |
| DVLOG(1) << __FUNCTION__ << "(" << type << ", " << init_data.size() << ")"; |
| } |
| - scoped_ptr<TextTrack> OnAddTextTrack(TextKind kind, |
| - const std::string& label, |
| - const std::string& language) { |
| - return scoped_ptr<TextTrack>(); |
| - } |
| - |
| void OnNewSegment() { |
| DVLOG(1) << __FUNCTION__; |
| results_stream_ << "NewSegment"; |
| @@ -110,11 +106,8 @@ class MP3StreamParserTest : public testing::Test { |
| base::Bind(&MP3StreamParserTest::OnInitDone, base::Unretained(this)), |
| base::Bind(&MP3StreamParserTest::OnNewConfig, base::Unretained(this)), |
| base::Bind(&MP3StreamParserTest::OnNewBuffers, base::Unretained(this)), |
| - base::Bind(&MP3StreamParserTest::OnNewTextBuffers, |
| - base::Unretained(this)), |
| + StreamParser::NewTextBuffersCB(), |
| base::Bind(&MP3StreamParserTest::OnKeyNeeded, base::Unretained(this)), |
| - base::Bind(&MP3StreamParserTest::OnAddTextTrack, |
| - base::Unretained(this)), |
| base::Bind(&MP3StreamParserTest::OnNewSegment, base::Unretained(this)), |
| base::Bind(&MP3StreamParserTest::OnEndOfSegment, |
| base::Unretained(this)), |