Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(306)

Unified Diff: media/base/stream_parser_buffer.cc

Issue 149153002: MSE: Add StreamParser buffer remuxing utility and tests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased, fixed comments from PS3, and prepped for TrackId unification work. Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..bceafbda5b7d1fddd94275200e62921c676552b3 100644
--- a/media/base/stream_parser_buffer.cc
+++ b/media/base/stream_parser_buffer.cc
@@ -19,21 +19,25 @@ 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,
+ DemuxerStream::UNKNOWN, 0));
}
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,
+ TrackId track_id) {
return make_scoped_refptr(
- new StreamParserBuffer(data, data_size, NULL, 0, is_keyframe));
+ new StreamParserBuffer(data, data_size, NULL, 0, is_keyframe, type,
+ track_id));
}
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, TrackId track_id) {
return make_scoped_refptr(
new StreamParserBuffer(data, data_size, side_data, side_data_size,
- is_keyframe));
+ is_keyframe, type, track_id));
}
base::TimeDelta StreamParserBuffer::GetDecodeTimestamp() const {
@@ -48,11 +52,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, TrackId track_id)
: DecoderBuffer(data, data_size, side_data, side_data_size),
is_keyframe_(is_keyframe),
decode_timestamp_(kNoTimestamp()),
- config_id_(kInvalidConfigId) {
+ config_id_(kInvalidConfigId),
+ type_(type),
+ track_id_(track_id) {
// 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.

Powered by Google App Engine
This is Rietveld 408576698