Chromium Code Reviews| Index: media/base/stream_parser_buffer.cc |
| diff --git a/media/base/stream_parser_buffer.cc b/media/base/stream_parser_buffer.cc |
| index e5cc9795c0318ca7e3e485fb4c553fb9cae0fa58..a4f3ee2079228f495c15ba98b908e5c1ba228fad 100644 |
| --- a/media/base/stream_parser_buffer.cc |
| +++ b/media/base/stream_parser_buffer.cc |
| @@ -19,21 +19,22 @@ static bool HasNestedFadeOutPreroll( |
| } |
| scoped_refptr<StreamParserBuffer> StreamParserBuffer::CreateEOSBuffer() { |
| - return make_scoped_refptr(new StreamParserBuffer(NULL, 0, NULL, 0, false)); |
| + return make_scoped_refptr(new StreamParserBuffer(NULL, 0, NULL, 0, false, |
| + kAudio /* meaningless */)); |
|
xhwang
2014/01/29 08:04:50
see above about the possibility of using UNKNOWN f
wolenetz
2014/02/05 02:49:53
Done.
|
| } |
| scoped_refptr<StreamParserBuffer> StreamParserBuffer::CopyFrom( |
| - const uint8* data, int data_size, bool is_keyframe) { |
| + const uint8* data, int data_size, bool is_keyframe, Type type) { |
| return make_scoped_refptr( |
| - new StreamParserBuffer(data, data_size, NULL, 0, is_keyframe)); |
| + new StreamParserBuffer(data, data_size, NULL, 0, is_keyframe, type)); |
| } |
| scoped_refptr<StreamParserBuffer> StreamParserBuffer::CopyFrom( |
| const uint8* data, int data_size, |
| - const uint8* side_data, int side_data_size, bool is_keyframe) { |
| + const uint8* side_data, int side_data_size, bool is_keyframe, Type type) { |
| return make_scoped_refptr( |
| new StreamParserBuffer(data, data_size, side_data, side_data_size, |
| - is_keyframe)); |
| + is_keyframe, type)); |
| } |
| base::TimeDelta StreamParserBuffer::GetDecodeTimestamp() const { |
| @@ -48,11 +49,14 @@ void StreamParserBuffer::SetDecodeTimestamp(const base::TimeDelta& timestamp) { |
| StreamParserBuffer::StreamParserBuffer(const uint8* data, int data_size, |
| const uint8* side_data, |
| - int side_data_size, bool is_keyframe) |
| + int side_data_size, bool is_keyframe, |
| + Type type) |
| : DecoderBuffer(data, data_size, side_data, side_data_size), |
| is_keyframe_(is_keyframe), |
| decode_timestamp_(kNoTimestamp()), |
| - config_id_(kInvalidConfigId) { |
| + config_id_(kInvalidConfigId), |
| + type_(type), |
| + text_track_number_(-1) { |
| // TODO(scherkus): Should DataBuffer constructor accept a timestamp and |
| // duration to force clients to set them? Today they end up being zero which |
| // is both a common and valid value and could lead to bugs. |