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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chromecast/media/cma/backend/alsa/stream_mixer_alsa.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
« chromecast/media/cma/backend/alsa/post_processors/governor_unittest.cc ('K') | « chromecast/media/cma/backend/alsa/post_processors/saturated_gain_unittest.cc ('k') | chromecast/media/cma/backend/alsa/volume_control.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromecast/media/cma/backend/alsa/stream_mixer_alsa.cc
diff --git a/chromecast/media/cma/backend/alsa/stream_mixer_alsa.cc b/chromecast/media/cma/backend/alsa/stream_mixer_alsa.cc
index daab40ef1703d9ed8c0d542120759bffd42c0acc..5836c9805db04f46b0f6ce331202b0d88d96a436 100644
--- a/chromecast/media/cma/backend/alsa/stream_mixer_alsa.cc
+++ b/chromecast/media/cma/backend/alsa/stream_mixer_alsa.cc
@@ -24,6 +24,7 @@
#include "chromecast/base/chromecast_switches.h"
#include "chromecast/media/base/audio_device_ids.h"
#include "chromecast/media/cma/backend/alsa/alsa_wrapper.h"
+#include "chromecast/media/cma/backend/alsa/cast_audio_json.h"
#include "chromecast/media/cma/backend/alsa/filter_group.h"
#include "chromecast/media/cma/backend/alsa/post_processing_pipeline_parser.h"
#include "chromecast/media/cma/backend/alsa/stream_mixer_alsa_input_impl.h"
@@ -277,7 +278,7 @@ void StreamMixerAlsa::CreatePostProcessors(
<< "media/audio/audio_device_description.cc";
CHECK(used_streams.insert(stream_type).second)
<< "Multiple instances of stream type '" << stream_type << "' in "
- << pipeline_parser->GetFilePath() << ".";
+ << kCastAudioJsonFilePath << ".";
}
filter_groups_.push_back(base::MakeUnique<FilterGroup>(
kNumInputChannels, false /* mono_mixer */,
« chromecast/media/cma/backend/alsa/post_processors/governor_unittest.cc ('K') | « chromecast/media/cma/backend/alsa/post_processors/saturated_gain_unittest.cc ('k') | chromecast/media/cma/backend/alsa/volume_control.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698