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

Side by Side Diff: chromecast/media/cma/backend/media_pipeline_backend_wrapper.cc

Issue 1855633002: Revert of [chromecast] Pass media task runner to MediaPipelineBackendManager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Created 4 years, 8 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 unified diff | Download patch
« no previous file with comments | « chromecast/media/cma/backend/media_pipeline_backend_wrapper.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "chromecast/media/cma/backend/media_pipeline_backend_wrapper.h" 5 #include "chromecast/media/cma/backend/media_pipeline_backend_wrapper.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "chromecast/media/cma/backend/media_pipeline_backend_manager.h" 8 #include "chromecast/media/cma/backend/media_pipeline_backend_manager.h"
9 9
10 namespace chromecast { 10 namespace chromecast {
11 namespace media { 11 namespace media {
12 12
13 MediaPipelineBackendWrapper::MediaPipelineBackendWrapper( 13 MediaPipelineBackendWrapper::MediaPipelineBackendWrapper(
14 scoped_ptr<MediaPipelineBackend> backend, 14 MediaPipelineBackend* backend,
15 int stream_type, 15 int stream_type,
16 float stream_type_volume, 16 float stream_type_volume)
17 MediaPipelineBackendManager* backend_manager) 17 : backend_(backend),
18 : backend_(std::move(backend)),
19 stream_type_(stream_type), 18 stream_type_(stream_type),
20 audio_decoder_wrapper_(nullptr), 19 audio_decoder_wrapper_(nullptr),
21 stream_type_volume_(stream_type_volume), 20 stream_type_volume_(stream_type_volume),
22 is_initialized_(false), 21 is_initialized_(false) {
23 backend_manager_(backend_manager) {
24 DCHECK(backend_); 22 DCHECK(backend_);
25 } 23 }
26 24
27 MediaPipelineBackendWrapper::~MediaPipelineBackendWrapper() { 25 MediaPipelineBackendWrapper::~MediaPipelineBackendWrapper() {
28 backend_manager_->OnMediaPipelineBackendDestroyed(this); 26 MediaPipelineBackendManager::OnMediaPipelineBackendDestroyed(this);
29 } 27 }
30 28
31 MediaPipelineBackend::AudioDecoder* 29 MediaPipelineBackend::AudioDecoder*
32 MediaPipelineBackendWrapper::CreateAudioDecoder() { 30 MediaPipelineBackendWrapper::CreateAudioDecoder() {
33 DCHECK(!is_initialized_); 31 DCHECK(!is_initialized_);
34 if (audio_decoder_wrapper_) 32 if (audio_decoder_wrapper_)
35 return nullptr; 33 return nullptr;
36 34
37 audio_decoder_wrapper_.reset( 35 audio_decoder_wrapper_.reset(
38 new AudioDecoderWrapper(backend_->CreateAudioDecoder())); 36 new AudioDecoderWrapper(backend_->CreateAudioDecoder()));
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 82
85 void MediaPipelineBackendWrapper::SetStreamTypeVolume( 83 void MediaPipelineBackendWrapper::SetStreamTypeVolume(
86 float stream_type_volume) { 84 float stream_type_volume) {
87 stream_type_volume_ = stream_type_volume; 85 stream_type_volume_ = stream_type_volume;
88 if (is_initialized_ && audio_decoder_wrapper_) 86 if (is_initialized_ && audio_decoder_wrapper_)
89 audio_decoder_wrapper_->SetStreamTypeVolume(stream_type_volume_); 87 audio_decoder_wrapper_->SetStreamTypeVolume(stream_type_volume_);
90 } 88 }
91 89
92 } // namespace media 90 } // namespace media
93 } // namespace chromecast 91 } // namespace chromecast
OLDNEW
« no previous file with comments | « chromecast/media/cma/backend/media_pipeline_backend_wrapper.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698