diff options
| author | wolenetz <wolenetz@chromium.org> | 2015-02-17 18:16:05 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-02-18 02:17:06 +0000 |
| commit | 186b656e47b8199822f71a5201a3a31bf43748ab (patch) | |
| tree | 56a40e0c2b49c2c1e1d1be645808e597e94f4f31 /media/formats/common | |
| parent | e0add0fd70c1f79154926fa02941e713cb36faeb (diff) | |
| download | chromium_src-186b656e47b8199822f71a5201a3a31bf43748ab.zip chromium_src-186b656e47b8199822f71a5201a3a31bf43748ab.tar.gz chromium_src-186b656e47b8199822f71a5201a3a31bf43748ab.tar.bz2 | |
MSE: Remove unused |success| StreamParser::InitCB argument
The only current source of a false |success| argument to
StreamParser::InitCB is MPEGAudioStreamParserBase::ParseFrame(), which
should not even run InitCB in that case. This CL removes this redundant
InitCB parameter, preparing for some further refactoring of InitCB and
NewConfigCB for compliant initialization segment received algorithm
implementation.
BUG=249428
TEST=No media_unittest or MSE layout test regressions locally on Linux
Review URL: https://codereview.chromium.org/935763002
Cr-Commit-Position: refs/heads/master@{#316744}
Diffstat (limited to 'media/formats/common')
| -rw-r--r-- | media/formats/common/stream_parser_test_base.cc | 4 | ||||
| -rw-r--r-- | media/formats/common/stream_parser_test_base.h | 2 |
2 files changed, 2 insertions, 4 deletions
diff --git a/media/formats/common/stream_parser_test_base.cc b/media/formats/common/stream_parser_test_base.cc index 41892d1..3a8b5f2 100644 --- a/media/formats/common/stream_parser_test_base.cc +++ b/media/formats/common/stream_parser_test_base.cc @@ -73,11 +73,9 @@ bool StreamParserTestBase::AppendDataInPieces(const uint8* data, } void StreamParserTestBase::OnInitDone( - bool success, const StreamParser::InitParameters& params) { EXPECT_TRUE(params.auto_update_timestamp_offset); - DVLOG(1) << __FUNCTION__ << "(" << success << ", " - << params.duration.InMilliseconds() << ", " + DVLOG(1) << __FUNCTION__ << "(" << params.duration.InMilliseconds() << ", " << params.auto_update_timestamp_offset << ")"; } diff --git a/media/formats/common/stream_parser_test_base.h b/media/formats/common/stream_parser_test_base.h index eb31562..1e3cd73 100644 --- a/media/formats/common/stream_parser_test_base.h +++ b/media/formats/common/stream_parser_test_base.h @@ -50,7 +50,7 @@ class StreamParserTestBase { private: bool AppendDataInPieces(const uint8* data, size_t length, size_t piece_size); - void OnInitDone(bool success, const StreamParser::InitParameters& params); + void OnInitDone(const StreamParser::InitParameters& params); bool OnNewConfig(const AudioDecoderConfig& audio_config, const VideoDecoderConfig& video_config, const StreamParser::TextTrackConfigMap& text_config); |