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

Side by Side Diff: media/base/audio_discard_helper_unittest.cc

Issue 414603002: Add support for partial append window end trimming. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Comments. Created 6 years, 4 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « media/base/audio_discard_helper.cc ('k') | media/filters/chunk_demuxer_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/memory/scoped_ptr.h" 5 #include "base/memory/scoped_ptr.h"
6 #include "media/base/audio_buffer.h" 6 #include "media/base/audio_buffer.h"
7 #include "media/base/audio_bus.h" 7 #include "media/base/audio_bus.h"
8 #include "media/base/audio_discard_helper.h" 8 #include "media/base/audio_discard_helper.h"
9 #include "media/base/buffers.h" 9 #include "media/base/buffers.h"
10 #include "media/base/decoder_buffer.h" 10 #include "media/base/decoder_buffer.h"
(...skipping 307 matching lines...) Expand 10 before | Expand all | Expand 10 after
318 const int kTestFrames = discard_helper.TimeDeltaToFrames(kDuration); 318 const int kTestFrames = discard_helper.TimeDeltaToFrames(kDuration);
319 319
320 scoped_refptr<DecoderBuffer> encoded_buffer = 320 scoped_refptr<DecoderBuffer> encoded_buffer =
321 CreateEncodedBuffer(kTimestamp, kDuration); 321 CreateEncodedBuffer(kTimestamp, kDuration);
322 scoped_refptr<AudioBuffer> decoded_buffer = CreateDecodedBuffer(kTestFrames); 322 scoped_refptr<AudioBuffer> decoded_buffer = CreateDecodedBuffer(kTestFrames);
323 323
324 // Set a discard padding equivalent to half of the buffer. 324 // Set a discard padding equivalent to half of the buffer.
325 encoded_buffer->set_discard_padding( 325 encoded_buffer->set_discard_padding(
326 std::make_pair(kDuration / 2, base::TimeDelta())); 326 std::make_pair(kDuration / 2, base::TimeDelta()));
327 327
328 // All of the first buffer should be discarded. 328 // All of the first buffer should be discarded, half from the inital delay and
329 // another half from the front discard padding.
330 //
331 // Encoded Discard Delay
332 // |--------| |---------| |----|
333 // |AAAAAAAA| --> |....|AAAA| --> |AAAA| -------> NULL
334 // |--------| |---------| |----|
335 // Decoded Discard Front Padding
336 //
329 ASSERT_FALSE(discard_helper.ProcessBuffers(encoded_buffer, decoded_buffer)); 337 ASSERT_FALSE(discard_helper.ProcessBuffers(encoded_buffer, decoded_buffer));
330 ASSERT_TRUE(discard_helper.initialized()); 338 ASSERT_TRUE(discard_helper.initialized());
331 339
332 // Processing another buffer (with the same discard padding) should discard 340 // Processing another buffer that has front discard set to half the buffer's
333 // the back half of the buffer since kDecoderDelay is half a buffer. 341 // duration should discard the back half of the buffer since kDecoderDelay is
334 encoded_buffer->set_timestamp(kTimestamp + kDuration); 342 // half a buffer. The end padding should not be discarded until another
343 // buffer is processed. kDuration / 4 is chosen for the end discard since it
344 // will force the end discard to start after position zero within the next
345 // decoded buffer.
346 //
347 // Encoded Discard Front Padding (from B)
348 // |--------| |---------| |----|
349 // |BBBBBBBB| --> |AAAA|BBBB| ----------> |AAAA|
350 // |--------| |---------| |----|
351 // Decoded
352 // (includes carryover from A)
353 //
354 encoded_buffer->set_timestamp(encoded_buffer->timestamp() + kDuration);
355 encoded_buffer->set_discard_padding(
356 std::make_pair(kDuration / 2, kDuration / 4));
335 decoded_buffer = CreateDecodedBuffer(kTestFrames); 357 decoded_buffer = CreateDecodedBuffer(kTestFrames);
336 ASSERT_FLOAT_EQ(0.0f, ExtractDecodedData(decoded_buffer, 0)); 358 ASSERT_FLOAT_EQ(0.0f, ExtractDecodedData(decoded_buffer, 0));
337 ASSERT_NEAR(kDecoderDelay * kDataStep, 359 ASSERT_NEAR(kDecoderDelay * kDataStep,
338 ExtractDecodedData(decoded_buffer, kDecoderDelay), 360 ExtractDecodedData(decoded_buffer, kDecoderDelay),
339 kDataStep * 1000); 361 kDataStep / 1000);
340 ASSERT_TRUE(discard_helper.ProcessBuffers(encoded_buffer, decoded_buffer)); 362 ASSERT_TRUE(discard_helper.ProcessBuffers(encoded_buffer, decoded_buffer));
341 EXPECT_EQ(kTimestamp, decoded_buffer->timestamp()); 363 EXPECT_EQ(kTimestamp, decoded_buffer->timestamp());
342 EXPECT_EQ(kDuration / 2, decoded_buffer->duration()); 364 EXPECT_EQ(kDuration / 2, decoded_buffer->duration());
343 EXPECT_EQ(kTestFrames / 2, decoded_buffer->frame_count()); 365 EXPECT_EQ(kTestFrames / 2, decoded_buffer->frame_count());
344 366
345 // Verify it was actually the latter half of the buffer that was removed. 367 // Verify it was actually the latter half of the buffer that was removed.
346 ASSERT_FLOAT_EQ(0.0f, ExtractDecodedData(decoded_buffer, 0)); 368 ASSERT_FLOAT_EQ(0.0f, ExtractDecodedData(decoded_buffer, 0));
369
370 // Verify the end discard padding is carried over to the next buffer. Use
371 // kDuration / 2 for the end discard padding so that the next buffer has its
372 // start entirely discarded.
373 //
374 // Encoded Discard End Padding (from B)
375 // |--------| |---------| |-------|
376 // |CCCCCCCC| --> |BBBB|CCCC| ----------> |BB|CCCC|
377 // |--------| |---------| |-------|
378 // Decoded
379 // (includes carryover from B)
380 //
381 encoded_buffer->set_timestamp(encoded_buffer->timestamp() + kDuration);
382 encoded_buffer->set_discard_padding(
383 std::make_pair(base::TimeDelta(), kDuration / 2));
384 decoded_buffer = CreateDecodedBuffer(kTestFrames);
385 ASSERT_TRUE(discard_helper.ProcessBuffers(encoded_buffer, decoded_buffer));
386 EXPECT_EQ(kTimestamp + kDuration / 2, decoded_buffer->timestamp());
387 EXPECT_EQ(3 * kDuration / 4, decoded_buffer->duration());
388 EXPECT_EQ(3 * kTestFrames / 4, decoded_buffer->frame_count());
389
390 // Verify it was actually the second quarter of the buffer that was removed.
391 const int kDiscardFrames = kTestFrames / 4;
392 ASSERT_FLOAT_EQ(0.0f, ExtractDecodedData(decoded_buffer, 0));
393 ASSERT_FLOAT_EQ(
394 kDiscardFrames * 2 * kDataStep,
395 ExtractDecodedData(decoded_buffer, kDecoderDelay - kDiscardFrames));
396
397 // One last test to ensure carryover discard from the start works.
398 //
399 // Encoded Discard End Padding (from C)
400 // |--------| |---------| |----|
401 // |DDDDDDDD| --> |CCCC|DDDD| ----------> |DDDD|
402 // |--------| |---------| |----|
403 // Decoded
404 // (includes carryover from C)
405 //
406 encoded_buffer->set_timestamp(encoded_buffer->timestamp() + kDuration);
407 encoded_buffer->set_discard_padding(DecoderBuffer::DiscardPadding());
408 decoded_buffer = CreateDecodedBuffer(kTestFrames);
409 ASSERT_FLOAT_EQ(0.0f, ExtractDecodedData(decoded_buffer, 0));
410 ASSERT_TRUE(discard_helper.ProcessBuffers(encoded_buffer, decoded_buffer));
411 EXPECT_EQ(kTimestamp + kDuration / 2 + 3 * kDuration / 4,
412 decoded_buffer->timestamp());
413 EXPECT_EQ(kDuration / 2, decoded_buffer->duration());
414 EXPECT_EQ(kTestFrames / 2, decoded_buffer->frame_count());
415 ASSERT_FLOAT_EQ(kTestFrames / 2 * kDataStep,
416 ExtractDecodedData(decoded_buffer, 0));
347 } 417 }
348 418
349 TEST(AudioDiscardHelperTest, DelayedDiscardInitialDiscardAndDiscardPadding) { 419 TEST(AudioDiscardHelperTest, DelayedDiscardInitialDiscardAndDiscardPadding) {
350 AudioDiscardHelper discard_helper(kSampleRate, 0); 420 AudioDiscardHelper discard_helper(kSampleRate, 0);
351 ASSERT_FALSE(discard_helper.initialized()); 421 ASSERT_FALSE(discard_helper.initialized());
352 422
353 const base::TimeDelta kTimestamp = base::TimeDelta(); 423 const base::TimeDelta kTimestamp = base::TimeDelta();
354 const base::TimeDelta kDuration = base::TimeDelta::FromMilliseconds(10); 424 const base::TimeDelta kDuration = base::TimeDelta::FromMilliseconds(10);
355 const int kTestFrames = discard_helper.TimeDeltaToFrames(kDuration); 425 const int kTestFrames = discard_helper.TimeDeltaToFrames(kDuration);
356 426
(...skipping 115 matching lines...) Expand 10 before | Expand all | Expand 10 after
472 decoded_buffer = CreateDecodedBuffer(kTestFrames * 2); 542 decoded_buffer = CreateDecodedBuffer(kTestFrames * 2);
473 ASSERT_TRUE(discard_helper.ProcessBuffers(encoded_buffer, decoded_buffer)); 543 ASSERT_TRUE(discard_helper.ProcessBuffers(encoded_buffer, decoded_buffer));
474 EXPECT_EQ(kTimestamp, decoded_buffer->timestamp()); 544 EXPECT_EQ(kTimestamp, decoded_buffer->timestamp());
475 EXPECT_EQ(kDuration * 2 - kDuration / 2, decoded_buffer->duration()); 545 EXPECT_EQ(kDuration * 2 - kDuration / 2, decoded_buffer->duration());
476 EXPECT_EQ(kTestFrames * 2 - kDecoderDelay, decoded_buffer->frame_count()); 546 EXPECT_EQ(kTestFrames * 2 - kDecoderDelay, decoded_buffer->frame_count());
477 ASSERT_FLOAT_EQ(kDecoderDelay * kDataStep, 547 ASSERT_FLOAT_EQ(kDecoderDelay * kDataStep,
478 ExtractDecodedData(decoded_buffer, 0)); 548 ExtractDecodedData(decoded_buffer, 0));
479 } 549 }
480 550
481 } // namespace media 551 } // namespace media
OLDNEW
« no previous file with comments | « media/base/audio_discard_helper.cc ('k') | media/filters/chunk_demuxer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698