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

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

Issue 1858803002: [chromecast] Pass media task runner to MediaPipelineBackendManager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Minor changes according to comments. 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 MediaPipelineBackend* backend, 14 scoped_ptr<MediaPipelineBackend> backend,
15 int stream_type, 15 int stream_type,
16 float stream_type_volume) 16 float stream_type_volume,
17 : backend_(backend), 17 MediaPipelineBackendManager* backend_manager)
18 : backend_(std::move(backend)),
18 stream_type_(stream_type), 19 stream_type_(stream_type),
19 audio_decoder_wrapper_(nullptr), 20 audio_decoder_wrapper_(nullptr),
20 stream_type_volume_(stream_type_volume), 21 stream_type_volume_(stream_type_volume),
21 is_initialized_(false) { 22 is_initialized_(false),
23 backend_manager_(backend_manager) {
22 DCHECK(backend_); 24 DCHECK(backend_);
23 } 25 }
24 26
25 MediaPipelineBackendWrapper::~MediaPipelineBackendWrapper() { 27 MediaPipelineBackendWrapper::~MediaPipelineBackendWrapper() {
26 MediaPipelineBackendManager::OnMediaPipelineBackendDestroyed(this); 28 backend_manager_->OnMediaPipelineBackendDestroyed(this);
27 } 29 }
28 30
29 MediaPipelineBackend::AudioDecoder* 31 MediaPipelineBackend::AudioDecoder*
30 MediaPipelineBackendWrapper::CreateAudioDecoder() { 32 MediaPipelineBackendWrapper::CreateAudioDecoder() {
31 DCHECK(!is_initialized_); 33 DCHECK(!is_initialized_);
32 if (audio_decoder_wrapper_) 34 if (audio_decoder_wrapper_)
33 return nullptr; 35 return nullptr;
34 36
35 audio_decoder_wrapper_.reset( 37 audio_decoder_wrapper_.reset(
36 new AudioDecoderWrapper(backend_->CreateAudioDecoder())); 38 new AudioDecoderWrapper(backend_->CreateAudioDecoder()));
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
82 84
83 void MediaPipelineBackendWrapper::SetStreamTypeVolume( 85 void MediaPipelineBackendWrapper::SetStreamTypeVolume(
84 float stream_type_volume) { 86 float stream_type_volume) {
85 stream_type_volume_ = stream_type_volume; 87 stream_type_volume_ = stream_type_volume;
86 if (is_initialized_ && audio_decoder_wrapper_) 88 if (is_initialized_ && audio_decoder_wrapper_)
87 audio_decoder_wrapper_->SetStreamTypeVolume(stream_type_volume_); 89 audio_decoder_wrapper_->SetStreamTypeVolume(stream_type_volume_);
88 } 90 }
89 91
90 } // namespace media 92 } // namespace media
91 } // namespace chromecast 93 } // 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