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

Unified Diff: media/mojo/services/mojo_audio_decoder.cc

Issue 1797393007: Connect MojoAudioDecoder to the service (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created AudioDecoder in MediaInterfaceProvider Created 4 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
« no previous file with comments | « media/mojo/services/mojo_audio_decoder.h ('k') | media/mojo/services/mojo_decoder_factory.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/mojo/services/mojo_audio_decoder.cc
diff --git a/media/mojo/services/mojo_audio_decoder.cc b/media/mojo/services/mojo_audio_decoder.cc
index 14a7a000e4c29375420ca8376088ebd81ff8c820..1c6ad0c616e1de1574d7d57681d9082b28487ef3 100644
--- a/media/mojo/services/mojo_audio_decoder.cc
+++ b/media/mojo/services/mojo_audio_decoder.cc
@@ -12,7 +12,10 @@
namespace media {
-MojoAudioDecoder::MojoAudioDecoder() {
+MojoAudioDecoder::MojoAudioDecoder(
+ scoped_refptr<base::SingleThreadTaskRunner> task_runner,
+ interfaces::AudioDecoderPtr remote_decoder)
+ : task_runner_(task_runner), remote_decoder_(std::move(remote_decoder)) {
DVLOG(1) << __FUNCTION__;
}
@@ -29,17 +32,19 @@ void MojoAudioDecoder::Initialize(const AudioDecoderConfig& config,
const InitCB& init_cb,
const OutputCB& output_cb) {
DVLOG(1) << __FUNCTION__;
- task_runner_ = base::ThreadTaskRunnerHandle::Get();
+ DCHECK(task_runner_->BelongsToCurrentThread());
NOTIMPLEMENTED();
- // Pretend to be able to decode anything.
- task_runner_->PostTask(FROM_HERE, base::Bind(init_cb, true));
+ // Fail initialization while not implemented.
+ task_runner_->PostTask(FROM_HERE, base::Bind(init_cb, false));
}
void MojoAudioDecoder::Decode(const scoped_refptr<DecoderBuffer>& buffer,
const DecodeCB& decode_cb) {
DVLOG(3) << __FUNCTION__;
+ DCHECK(task_runner_->BelongsToCurrentThread());
+
NOTIMPLEMENTED();
// Actually we can't decode anything.
@@ -48,11 +53,15 @@ void MojoAudioDecoder::Decode(const scoped_refptr<DecoderBuffer>& buffer,
void MojoAudioDecoder::Reset(const base::Closure& closure) {
DVLOG(2) << __FUNCTION__;
+ DCHECK(task_runner_->BelongsToCurrentThread());
+
NOTIMPLEMENTED();
}
bool MojoAudioDecoder::NeedsBitstreamConversion() const {
DVLOG(1) << __FUNCTION__;
+ DCHECK(task_runner_->BelongsToCurrentThread());
+
NOTIMPLEMENTED();
return false;
}
« no previous file with comments | « media/mojo/services/mojo_audio_decoder.h ('k') | media/mojo/services/mojo_decoder_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698