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

Unified Diff: media/filters/source_buffer_stream.cc

Issue 712593003: Move key frame flag from StreamParserBuffer to DecoderBuffer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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/filters/source_buffer_stream.cc
diff --git a/media/filters/source_buffer_stream.cc b/media/filters/source_buffer_stream.cc
index 5a2de0b49b84e92b1d745cba057d114c4572ec58..c025f1da249cf5206ba22afef985e5f6dd5adca4 100644
--- a/media/filters/source_buffer_stream.cc
+++ b/media/filters/source_buffer_stream.cc
@@ -189,7 +189,7 @@ bool SourceBufferStream::Append(const BufferQueue& buffers) {
DCHECK(!end_of_stream_);
// New media segments must begin with a keyframe.
- if (new_media_segment_ && !buffers.front()->IsKeyframe()) {
+ if (new_media_segment_ && !buffers.front()->is_keyframe()) {
MEDIA_LOG(log_cb_) << "Media segment did not begin with keyframe.";
return false;
}
@@ -206,7 +206,7 @@ bool SourceBufferStream::Append(const BufferQueue& buffers) {
}
if (!IsNextTimestampValid(buffers.front()->GetDecodeTimestamp(),
- buffers.front()->IsKeyframe())) {
+ buffers.front()->is_keyframe())) {
MEDIA_LOG(log_cb_) << "Invalid same timestamp construct detected at time "
<< buffers.front()->GetDecodeTimestamp().InSecondsF();
@@ -227,7 +227,7 @@ bool SourceBufferStream::Append(const BufferQueue& buffers) {
if (range_for_next_append_ != ranges_.end()) {
(*range_for_next_append_)->AppendBuffersToEnd(buffers);
last_appended_buffer_timestamp_ = buffers.back()->GetDecodeTimestamp();
- last_appended_buffer_is_keyframe_ = buffers.back()->IsKeyframe();
+ last_appended_buffer_is_keyframe_ = buffers.back()->is_keyframe();
} else {
DecodeTimestamp new_range_start_time = std::min(
media_segment_start_time_, buffers.front()->GetDecodeTimestamp());
@@ -242,7 +242,7 @@ bool SourceBufferStream::Append(const BufferQueue& buffers) {
BufferQueue::const_iterator itr = buffers.begin();
// Scan past all the non-keyframes.
- while (itr != buffers.end() && !(*itr)->IsKeyframe()) {
+ while (itr != buffers.end() && !(*itr)->is_keyframe()) {
++itr;
}
@@ -250,7 +250,7 @@ bool SourceBufferStream::Append(const BufferQueue& buffers) {
// buffer state and return.
if (itr == buffers.end()) {
last_appended_buffer_timestamp_ = buffers.back()->GetDecodeTimestamp();
- last_appended_buffer_is_keyframe_ = buffers.back()->IsKeyframe();
+ last_appended_buffer_is_keyframe_ = buffers.back()->is_keyframe();
return true;
} else if (itr != buffers.begin()) {
// Copy the first keyframe and everything after it into
@@ -272,7 +272,7 @@ bool SourceBufferStream::Append(const BufferQueue& buffers) {
last_appended_buffer_timestamp_ =
buffers_for_new_range->back()->GetDecodeTimestamp();
last_appended_buffer_is_keyframe_ =
- buffers_for_new_range->back()->IsKeyframe();
+ buffers_for_new_range->back()->is_keyframe();
}
new_media_segment_ = false;
@@ -456,7 +456,7 @@ bool SourceBufferStream::IsMonotonicallyIncreasing(
for (BufferQueue::const_iterator itr = buffers.begin();
itr != buffers.end(); ++itr) {
DecodeTimestamp current_timestamp = (*itr)->GetDecodeTimestamp();
- bool current_is_keyframe = (*itr)->IsKeyframe();
+ bool current_is_keyframe = (*itr)->is_keyframe();
DCHECK(current_timestamp != kNoDecodeTimestamp());
DCHECK((*itr)->duration() >= base::TimeDelta())
<< "Packet with invalid duration."
@@ -740,7 +740,7 @@ void SourceBufferStream::PrepareRangesForNextAppend(
DecodeTimestamp prev_timestamp = last_appended_buffer_timestamp_;
bool prev_is_keyframe = last_appended_buffer_is_keyframe_;
DecodeTimestamp next_timestamp = new_buffers.front()->GetDecodeTimestamp();
- bool next_is_keyframe = new_buffers.front()->IsKeyframe();
+ bool next_is_keyframe = new_buffers.front()->is_keyframe();
if (prev_timestamp != kNoDecodeTimestamp() &&
prev_timestamp != next_timestamp) {

Powered by Google App Engine
This is Rietveld 408576698