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

Unified Diff: media/filters/source_buffer_stream_unittest.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_unittest.cc
diff --git a/media/filters/source_buffer_stream_unittest.cc b/media/filters/source_buffer_stream_unittest.cc
index 0fbd0425a51448bc993cf8266261a2e44f91cb42..40fb895eee3d4dc6ad966aabd3cd821516d132d1 100644
--- a/media/filters/source_buffer_stream_unittest.cc
+++ b/media/filters/source_buffer_stream_unittest.cc
@@ -232,7 +232,7 @@ class SourceBufferStreamTest : public testing::Test {
break;
if (expect_keyframe && current_position == starting_position)
- EXPECT_TRUE(buffer->IsKeyframe());
+ EXPECT_TRUE(buffer->is_keyframe());
if (expected_data) {
const uint8* actual_data = buffer->data();
@@ -295,7 +295,7 @@ class SourceBufferStreamTest : public testing::Test {
// Handle preroll buffers.
if (EndsWith(timestamps[i], "P", true)) {
- ASSERT_TRUE(buffer->IsKeyframe());
+ ASSERT_TRUE(buffer->is_keyframe());
scoped_refptr<StreamParserBuffer> preroll_buffer;
preroll_buffer.swap(buffer);
@@ -312,10 +312,10 @@ class SourceBufferStreamTest : public testing::Test {
preroll_buffer->GetDecodeTimestamp());
ASSERT_EQ(kInfiniteDuration(), preroll_buffer->discard_padding().first);
ASSERT_EQ(base::TimeDelta(), preroll_buffer->discard_padding().second);
- ASSERT_TRUE(buffer->IsKeyframe());
+ ASSERT_TRUE(buffer->is_keyframe());
ss << "P";
- } else if (buffer->IsKeyframe()) {
+ } else if (buffer->is_keyframe()) {
ss << "K";
}
@@ -2301,7 +2301,7 @@ TEST_F(SourceBufferStreamTest, PresentationTimestampIndependence) {
scoped_refptr<StreamParserBuffer> buffer;
ASSERT_EQ(stream_->GetNextBuffer(&buffer), SourceBufferStream::kSuccess);
- if (buffer->IsKeyframe()) {
+ if (buffer->is_keyframe()) {
EXPECT_EQ(DecodeTimestamp::FromPresentationTime(buffer->timestamp()),
buffer->GetDecodeTimestamp());
last_keyframe_idx = i;

Powered by Google App Engine
This is Rietveld 408576698