OLD | NEW |
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 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 "media/renderers/audio_renderer_impl.h" | 5 #include "media/renderers/audio_renderer_impl.h" |
6 | 6 |
7 #include <memory> | 7 #include <memory> |
8 #include <utility> | 8 #include <utility> |
9 | 9 |
10 #include "base/bind.h" | 10 #include "base/bind.h" |
11 #include "base/callback_helpers.h" | 11 #include "base/callback_helpers.h" |
12 #include "base/format_macros.h" | 12 #include "base/format_macros.h" |
13 #include "base/macros.h" | 13 #include "base/macros.h" |
14 #include "base/run_loop.h" | 14 #include "base/run_loop.h" |
15 #include "base/single_thread_task_runner.h" | 15 #include "base/single_thread_task_runner.h" |
16 #include "base/strings/stringprintf.h" | 16 #include "base/strings/stringprintf.h" |
17 #include "base/test/simple_test_tick_clock.h" | 17 #include "base/test/simple_test_tick_clock.h" |
18 #include "media/base/audio_buffer_converter.h" | 18 #include "media/base/audio_buffer_converter.h" |
19 #include "media/base/audio_splicer.h" | |
20 #include "media/base/fake_audio_renderer_sink.h" | 19 #include "media/base/fake_audio_renderer_sink.h" |
21 #include "media/base/gmock_callback_support.h" | 20 #include "media/base/gmock_callback_support.h" |
22 #include "media/base/media_util.h" | 21 #include "media/base/media_util.h" |
23 #include "media/base/mock_filters.h" | 22 #include "media/base/mock_filters.h" |
24 #include "media/base/test_helpers.h" | 23 #include "media/base/test_helpers.h" |
25 #include "testing/gtest/include/gtest/gtest.h" | 24 #include "testing/gtest/include/gtest/gtest.h" |
26 | 25 |
27 using ::base::TimeDelta; | 26 using ::base::TimeDelta; |
28 using ::testing::_; | 27 using ::testing::_; |
29 using ::testing::Return; | 28 using ::testing::Return; |
(...skipping 333 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
363 | 362 |
364 void force_config_change() { | 363 void force_config_change() { |
365 renderer_->OnConfigChange(); | 364 renderer_->OnConfigChange(); |
366 } | 365 } |
367 | 366 |
368 InputFrames converter_input_frames_left() const { | 367 InputFrames converter_input_frames_left() const { |
369 return InputFrames( | 368 return InputFrames( |
370 renderer_->buffer_converter_->input_frames_left_for_testing()); | 369 renderer_->buffer_converter_->input_frames_left_for_testing()); |
371 } | 370 } |
372 | 371 |
373 bool splicer_has_next_buffer() const { | |
374 return renderer_->splicer_->HasNextBuffer(); | |
375 } | |
376 | |
377 base::TimeDelta CurrentMediaTime() { | 372 base::TimeDelta CurrentMediaTime() { |
378 return renderer_->CurrentMediaTime(); | 373 return renderer_->CurrentMediaTime(); |
379 } | 374 } |
380 | 375 |
381 bool ended() const { return ended_; } | 376 bool ended() const { return ended_; } |
382 | 377 |
383 // Fixture members. | 378 // Fixture members. |
384 AudioParameters hardware_params_; | 379 AudioParameters hardware_params_; |
385 base::MessageLoop message_loop_; | 380 base::MessageLoop message_loop_; |
386 std::unique_ptr<AudioRendererImpl> renderer_; | 381 std::unique_ptr<AudioRendererImpl> renderer_; |
(...skipping 279 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
666 } | 661 } |
667 | 662 |
668 TEST_F(AudioRendererImplTest, InitializeThenDestroy) { | 663 TEST_F(AudioRendererImplTest, InitializeThenDestroy) { |
669 InitializeAndDestroy(); | 664 InitializeAndDestroy(); |
670 } | 665 } |
671 | 666 |
672 TEST_F(AudioRendererImplTest, InitializeThenDestroyDuringDecoderInit) { | 667 TEST_F(AudioRendererImplTest, InitializeThenDestroyDuringDecoderInit) { |
673 InitializeAndDestroyDuringDecoderInit(); | 668 InitializeAndDestroyDuringDecoderInit(); |
674 } | 669 } |
675 | 670 |
| 671 // |
| 672 // TODO- CONFIG change no longer drains converter. Anything to test here? |
| 673 // |
676 TEST_F(AudioRendererImplTest, ConfigChangeDrainsConverter) { | 674 TEST_F(AudioRendererImplTest, ConfigChangeDrainsConverter) { |
677 Initialize(); | 675 Initialize(); |
678 Preroll(); | 676 Preroll(); |
679 StartTicking(); | 677 StartTicking(); |
680 | 678 |
681 // Drain internal buffer, we should have a pending read. | 679 // Drain internal buffer, we should have a pending read. |
682 EXPECT_TRUE(ConsumeBufferedData(frames_buffered())); | 680 EXPECT_TRUE(ConsumeBufferedData(frames_buffered())); |
683 WaitForPendingRead(); | 681 WaitForPendingRead(); |
684 | 682 |
685 // Deliver a little bit of data. Use an odd data size to ensure there is data | 683 // Deliver a little bit of data. Use an odd data size to ensure there is data |
686 // left in the AudioBufferConverter. Ensure no buffers are in the splicer. | 684 // left in the AudioBufferConverter. Ensure no buffers are in the splicer. |
687 SatisfyPendingRead(InputFrames(2053)); | 685 SatisfyPendingRead(InputFrames(2053)); |
688 EXPECT_FALSE(splicer_has_next_buffer()); | 686 // EXPECT_FALSE(splicer_has_next_buffer()); |
689 EXPECT_GT(converter_input_frames_left().value, 0); | 687 EXPECT_GT(converter_input_frames_left().value, 0); |
690 | 688 |
691 // Force a config change and then ensure all buffered data has been put into | 689 // Force a config change and then ensure all buffered data has been put into |
692 // the splicer. | 690 // the splicer. |
693 force_config_change(); | 691 force_config_change(); |
694 EXPECT_TRUE(splicer_has_next_buffer()); | 692 // EXPECT_TRUE(splicer_has_next_buffer()); |
695 EXPECT_EQ(0, converter_input_frames_left().value); | 693 EXPECT_EQ(0, converter_input_frames_left().value); |
696 } | 694 } |
697 | 695 |
698 TEST_F(AudioRendererImplTest, CurrentMediaTimeBehavior) { | 696 TEST_F(AudioRendererImplTest, CurrentMediaTimeBehavior) { |
699 Initialize(); | 697 Initialize(); |
700 Preroll(); | 698 Preroll(); |
701 StartTicking(); | 699 StartTicking(); |
702 | 700 |
703 AudioTimestampHelper timestamp_helper(kOutputSamplesPerSecond); | 701 AudioTimestampHelper timestamp_helper(kOutputSamplesPerSecond); |
704 timestamp_helper.SetBaseTimestamp(base::TimeDelta()); | 702 timestamp_helper.SetBaseTimestamp(base::TimeDelta()); |
(...skipping 315 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
1020 // Advance far enough that we shouldn't be clamped to current time (tested | 1018 // Advance far enough that we shouldn't be clamped to current time (tested |
1021 // already above). | 1019 // already above). |
1022 tick_clock_->Advance(kOneSecond); | 1020 tick_clock_->Advance(kOneSecond); |
1023 EXPECT_EQ( | 1021 EXPECT_EQ( |
1024 current_time + timestamp_helper.GetFrameDuration(frames_to_consume.value), | 1022 current_time + timestamp_helper.GetFrameDuration(frames_to_consume.value), |
1025 CurrentMediaWallClockTime(&is_time_moving)); | 1023 CurrentMediaWallClockTime(&is_time_moving)); |
1026 EXPECT_TRUE(is_time_moving); | 1024 EXPECT_TRUE(is_time_moving); |
1027 } | 1025 } |
1028 | 1026 |
1029 } // namespace media | 1027 } // namespace media |
OLD | NEW |