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

Unified Diff: chromecast/media/cma/backend/audio_pipeline_device.h

Issue 1074383002: Introduce VideoConfig/AudioConfig class for CMA backend (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add TODO comments and fix unit test Created 5 years, 7 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/audio_pipeline_device.h
diff --git a/chromecast/media/cma/backend/audio_pipeline_device.h b/chromecast/media/cma/backend/audio_pipeline_device.h
index 932a807ac60e4f2a9d9131eda26ec3676e79b249..84c1af1fe1d9c1fb42bfe3db3a514b25f5eba4a5 100644
--- a/chromecast/media/cma/backend/audio_pipeline_device.h
+++ b/chromecast/media/cma/backend/audio_pipeline_device.h
@@ -8,13 +8,10 @@
#include "base/macros.h"
#include "chromecast/media/cma/backend/media_component_device.h"
-namespace media {
-class AudioDecoderConfig;
-}
-
namespace chromecast {
namespace media {
class AudioPipelineDeviceClient;
+struct AudioConfig;
class AudioPipelineDevice : public MediaComponentDevice {
public:
@@ -25,7 +22,7 @@ class AudioPipelineDevice : public MediaComponentDevice {
// Must be called before switching from |kStateUninitialized| to |kStateIdle|.
// Afterwards, this can be invoked any time the configuration changes.
// Returns true if the configuration is a supported configuration.
- virtual bool SetConfig(const ::media::AudioDecoderConfig& config) = 0;
+ virtual bool SetConfig(const AudioConfig& config) = 0;
// Sets the volume multiplier.
// The multiplier must be in the range [0.0, 1.0].

Powered by Google App Engine
This is Rietveld 408576698