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

Unified Diff: chromecast/media/cma/backend/alsa/post_processors/governor_unittest.cc

Issue 2958393002: Add a simple, safe gain PostProcessor. (Closed)
Patch Set: Combine cma/backend/alsa tests into a single target Created 3 years, 5 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
Index: chromecast/media/cma/backend/alsa/post_processors/governor_unittest.cc
diff --git a/chromecast/media/cma/backend/alsa/post_processors/governor_unittest.cc b/chromecast/media/cma/backend/alsa/post_processors/governor_unittest.cc
index 38cac53f21d75a1196a58cd169bb864aa0b73018..d91ccc8aa298ec7979ad7ddd4f2e8cbff8b296c6 100644
--- a/chromecast/media/cma/backend/alsa/post_processors/governor_unittest.cc
+++ b/chromecast/media/cma/backend/alsa/post_processors/governor_unittest.cc
@@ -13,8 +13,6 @@
#include "base/strings/stringprintf.h"
#include "chromecast/media/cma/backend/alsa/post_processors/governor.h"
#include "chromecast/media/cma/backend/alsa/post_processors/post_processor_unittest.h"
-#include "media/base/audio_bus.h"
-#include "media/base/audio_sample_types.h"
#include "testing/gtest/include/gtest/gtest.h"
namespace chromecast {
@@ -111,24 +109,24 @@ INSTANTIATE_TEST_CASE_P(GovernorClampVolumeTest,
::testing::Values(0.0f, 0.1f, 0.5f, 0.9f, 1.0f, 1.1f));
// Default tests from post_processor_test
-TEST_P(PostProcessorTest, TestDelay) {
+TEST_P(PostProcessorTest, GovernorDelay) {
std::string config = MakeConfigString(1.0, 1.0);
auto pp =
- base::WrapUnique(AudioPostProcessorShlib_Create(config, kNumChannels));
+ base::WrapUnique<AudioPostProcessor>(new Governor(config, kNumChannels));
kmackay 2017/07/06 22:16:28 Why not make this work the same way as saturated_g
bshaya 2017/07/06 22:38:10 Done.
TestDelay(pp.get(), sample_rate_);
}
-TEST_P(PostProcessorTest, TestRinging) {
+TEST_P(PostProcessorTest, GovernorRinging) {
std::string config = MakeConfigString(1.0, 1.0);
auto pp =
- base::WrapUnique(AudioPostProcessorShlib_Create(config, kNumChannels));
+ base::WrapUnique<AudioPostProcessor>(new Governor(config, kNumChannels));
TestRingingTime(pp.get(), sample_rate_);
}
-TEST_P(PostProcessorTest, TestPassthrough) {
+TEST_P(PostProcessorTest, GovernorPassthrough) {
std::string config = MakeConfigString(1.0, 1.0);
auto pp =
- base::WrapUnique(AudioPostProcessorShlib_Create(config, kNumChannels));
+ base::WrapUnique<AudioPostProcessor>(new Governor(config, kNumChannels));
TestPassthrough(pp.get(), sample_rate_);
}

Powered by Google App Engine
This is Rietveld 408576698