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

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

Issue 2712883006: [Chromecast] Add new volume control API to CastMediaShlib (Closed)
Patch Set: rebase Created 3 years, 9 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/media_pipeline_backend_wrapper.h
diff --git a/chromecast/media/cma/backend/media_pipeline_backend_wrapper.h b/chromecast/media/cma/backend/media_pipeline_backend_wrapper.h
index ad158913e6c396823e84b69593ed51662798b5ef..24b8b65d4d7441a5a84d21f429aa0d0e74314cb1 100644
--- a/chromecast/media/cma/backend/media_pipeline_backend_wrapper.h
+++ b/chromecast/media/cma/backend/media_pipeline_backend_wrapper.h
@@ -12,7 +12,6 @@
#include "base/macros.h"
#include "base/memory/ref_counted.h"
#include "base/time/time.h"
-#include "chromecast/media/cma/backend/audio_decoder_wrapper.h"
#include "chromecast/public/media/media_pipeline_backend.h"
#include "chromecast/public/media/media_pipeline_device_params.h"
@@ -23,9 +22,7 @@ class MediaPipelineBackendManager;
class MediaPipelineBackendWrapper : public MediaPipelineBackend {
public:
- MediaPipelineBackendWrapper(std::unique_ptr<MediaPipelineBackend> backend,
- int stream_type,
- float stream_type_volume,
+ MediaPipelineBackendWrapper(const media::MediaPipelineDeviceParams& params,
MediaPipelineBackendManager* backend_manager);
~MediaPipelineBackendWrapper() override;
@@ -40,18 +37,17 @@ class MediaPipelineBackendWrapper : public MediaPipelineBackend {
int64_t GetCurrentPts() override;
bool SetPlaybackRate(float rate) override;
- int GetStreamType() const;
- void SetStreamTypeVolume(float stream_type_volume);
-
private:
- std::unique_ptr<MediaPipelineBackend> backend_;
- const int stream_type_;
- std::unique_ptr<AudioDecoderWrapper> audio_decoder_wrapper_;
- float stream_type_volume_;
- bool is_initialized_;
- bool have_video_decoder_;
+ void SetPlaying(bool playing);
+
+ const std::unique_ptr<MediaPipelineBackend> backend_;
MediaPipelineBackendManager* const backend_manager_;
+ bool sfx_backend_;
+ bool have_audio_decoder_;
+ bool have_video_decoder_;
+ bool playing_;
+
DISALLOW_COPY_AND_ASSIGN(MediaPipelineBackendWrapper);
};

Powered by Google App Engine
This is Rietveld 408576698