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

Unified Diff: media/base/stream_parser_buffer.cc

Issue 276573002: Add gapless playback support for AAC playback. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix msvc error. Created 6 years, 7 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
« no previous file with comments | « media/base/stream_parser_buffer.h ('k') | media/filters/chunk_demuxer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/stream_parser_buffer.cc
diff --git a/media/base/stream_parser_buffer.cc b/media/base/stream_parser_buffer.cc
index 1d3440235488a9a241d75ac5530dc11e9cfcf11b..e9d64272c4d84670c4b2e7e1fe739433e856412b 100644
--- a/media/base/stream_parser_buffer.cc
+++ b/media/base/stream_parser_buffer.cc
@@ -67,8 +67,10 @@ base::TimeDelta StreamParserBuffer::GetDecodeTimestamp() const {
return decode_timestamp_;
}
-void StreamParserBuffer::SetDecodeTimestamp(const base::TimeDelta& timestamp) {
+void StreamParserBuffer::SetDecodeTimestamp(base::TimeDelta timestamp) {
decode_timestamp_ = timestamp;
+ if (preroll_buffer_)
+ preroll_buffer_->SetDecodeTimestamp(timestamp);
}
StreamParserBuffer::StreamParserBuffer(const uint8* data, int data_size,
@@ -97,6 +99,8 @@ int StreamParserBuffer::GetConfigId() const {
void StreamParserBuffer::SetConfigId(int config_id) {
config_id_ = config_id;
+ if (preroll_buffer_)
+ preroll_buffer_->SetConfigId(config_id);
}
void StreamParserBuffer::ConvertToSpliceBuffer(
@@ -118,6 +122,12 @@ void StreamParserBuffer::ConvertToSpliceBuffer(
// that implies EOS care must be taken to ensure there are no clients relying
// on that behavior.
+ // Move over any preroll from this buffer.
+ if (preroll_buffer_) {
+ DCHECK(!overlapping_buffer->preroll_buffer_);
+ overlapping_buffer->preroll_buffer_.swap(preroll_buffer_);
+ }
+
// Rewrite |this| buffer as a splice buffer.
SetDecodeTimestamp(first_splice_buffer->GetDecodeTimestamp());
SetConfigId(first_splice_buffer->GetConfigId());
@@ -141,6 +151,7 @@ void StreamParserBuffer::ConvertToSpliceBuffer(
++it) {
const scoped_refptr<StreamParserBuffer>& buffer = *it;
DCHECK(!buffer->end_of_stream());
+ DCHECK(!buffer->preroll_buffer());
DCHECK(buffer->splice_buffers().empty());
splice_buffers_.push_back(CopyBuffer(*buffer));
splice_buffers_.back()->set_splice_timestamp(splice_timestamp());
@@ -149,4 +160,32 @@ void StreamParserBuffer::ConvertToSpliceBuffer(
splice_buffers_.push_back(overlapping_buffer);
}
+void StreamParserBuffer::SetPrerollBuffer(
+ const scoped_refptr<StreamParserBuffer>& preroll_buffer) {
+ DCHECK(!preroll_buffer_);
+ DCHECK(!end_of_stream());
+ DCHECK(!preroll_buffer->end_of_stream());
+ DCHECK(!preroll_buffer->preroll_buffer_);
+ DCHECK(preroll_buffer->splice_timestamp() == kNoTimestamp());
+ DCHECK(preroll_buffer->splice_buffers().empty());
+ DCHECK(preroll_buffer->timestamp() <= timestamp());
+ DCHECK(preroll_buffer->discard_padding() == DecoderBuffer::DiscardPadding());
+ DCHECK_EQ(preroll_buffer->type(), type());
+ DCHECK_EQ(preroll_buffer->track_id(), track_id());
+
+ preroll_buffer_ = preroll_buffer;
+ preroll_buffer_->set_timestamp(timestamp());
+ preroll_buffer_->SetDecodeTimestamp(GetDecodeTimestamp());
+
+ // Mark the entire buffer for discard.
+ preroll_buffer_->set_discard_padding(
+ std::make_pair(kInfiniteDuration(), base::TimeDelta()));
+}
+
+void StreamParserBuffer::set_timestamp(base::TimeDelta timestamp) {
+ DecoderBuffer::set_timestamp(timestamp);
+ if (preroll_buffer_)
+ preroll_buffer_->set_timestamp(timestamp);
+}
+
} // namespace media
« no previous file with comments | « media/base/stream_parser_buffer.h ('k') | media/filters/chunk_demuxer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698