Chromium Code Reviews| Index: content/browser/media/android/browser_demuxer_android.cc |
| diff --git a/content/browser/media/android/browser_demuxer_android.cc b/content/browser/media/android/browser_demuxer_android.cc |
| index ed462032fdd7f6ad3d278996d36ae4009370ecca..ddaca7efa3cedbe025f748914a01b633141d1621 100644 |
| --- a/content/browser/media/android/browser_demuxer_android.cc |
| +++ b/content/browser/media/android/browser_demuxer_android.cc |
| @@ -4,7 +4,10 @@ |
| #include "content/browser/media/android/browser_demuxer_android.h" |
| +#include "base/command_line.h" |
| #include "content/common/media/media_player_messages_android.h" |
| +#include "media/base/android/media_codec_player.h" |
| +#include "media/base/media_switches.h" |
| namespace content { |
| @@ -52,21 +55,32 @@ class BrowserDemuxerAndroid::Internal : public media::DemuxerAndroid { |
| }; |
| BrowserDemuxerAndroid::BrowserDemuxerAndroid() |
| - : BrowserMessageFilter(MediaPlayerMsgStart) {} |
| + : BrowserMessageFilter(MediaPlayerMsgStart) { |
| + bool enable_media_thread = |
| + base::CommandLine::ForCurrentProcess()-> |
| + HasSwitch(switches::kEnableMediaThreadForMediaPlayback); |
| + |
| + task_runner_ = |
| + enable_media_thread ? |
| + media::GetMediaTaskRunner().get() : |
|
timav
2015/05/08 22:32:01
Without the BUILD.gn change in content the linker
|
| + BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI).get(); |
| + |
| +} |
| BrowserDemuxerAndroid::~BrowserDemuxerAndroid() {} |
| -void BrowserDemuxerAndroid::OverrideThreadForMessage( |
| - const IPC::Message& message, |
| - BrowserThread::ID* thread) { |
| +base::TaskRunner* BrowserDemuxerAndroid::OverrideTaskRunnerForMessage( |
| + const IPC::Message& message) { |
| + |
| switch (message.type()) { |
| case MediaPlayerHostMsg_DemuxerReady::ID: |
| case MediaPlayerHostMsg_ReadFromDemuxerAck::ID: |
| case MediaPlayerHostMsg_DurationChanged::ID: |
| case MediaPlayerHostMsg_DemuxerSeekDone::ID: |
| - *thread = BrowserThread::UI; |
| - return; |
| + return task_runner_; |
| } |
| + |
| + return nullptr; |
| } |
| bool BrowserDemuxerAndroid::OnMessageReceived(const IPC::Message& message) { |
| @@ -95,29 +109,46 @@ void BrowserDemuxerAndroid::AddDemuxerClient( |
| media::DemuxerAndroidClient* client) { |
| DVLOG(1) << __FUNCTION__ << " peer_pid=" << peer_pid() |
| << " demuxer_client_id=" << demuxer_client_id; |
| + DCHECK(task_runner_->BelongsToCurrentThread()); |
| + |
| demuxer_clients_.AddWithID(client, demuxer_client_id); |
| + |
| + if (pending_configs_.count(demuxer_client_id)) { |
| + client->OnDemuxerConfigsAvailable(pending_configs_[demuxer_client_id]); |
| + pending_configs_.erase(demuxer_client_id); |
| + } |
| } |
| void BrowserDemuxerAndroid::RemoveDemuxerClient(int demuxer_client_id) { |
| DVLOG(1) << __FUNCTION__ << " peer_pid=" << peer_pid() |
| << " demuxer_client_id=" << demuxer_client_id; |
| + DCHECK(task_runner_->BelongsToCurrentThread()); |
| + |
| demuxer_clients_.Remove(demuxer_client_id); |
| } |
| void BrowserDemuxerAndroid::OnDemuxerReady( |
| int demuxer_client_id, |
| const media::DemuxerConfigs& configs) { |
| + DCHECK(task_runner_->BelongsToCurrentThread()); |
| + |
| media::DemuxerAndroidClient* client = |
| demuxer_clients_.Lookup(demuxer_client_id); |
| + |
| if (client) |
| client->OnDemuxerConfigsAvailable(configs); |
| + else |
| + pending_configs_[demuxer_client_id] = configs; |
| } |
| void BrowserDemuxerAndroid::OnReadFromDemuxerAck( |
| int demuxer_client_id, |
| const media::DemuxerData& data) { |
| + DCHECK(task_runner_->BelongsToCurrentThread()); |
| + |
| media::DemuxerAndroidClient* client = |
| demuxer_clients_.Lookup(demuxer_client_id); |
| + |
| if (client) |
| client->OnDemuxerDataAvailable(data); |
| } |
| @@ -125,18 +156,26 @@ void BrowserDemuxerAndroid::OnReadFromDemuxerAck( |
| void BrowserDemuxerAndroid::OnDemuxerSeekDone( |
| int demuxer_client_id, |
| const base::TimeDelta& actual_browser_seek_time) { |
| + DCHECK(task_runner_->BelongsToCurrentThread()); |
| + |
| media::DemuxerAndroidClient* client = |
| demuxer_clients_.Lookup(demuxer_client_id); |
| + |
| if (client) |
| client->OnDemuxerSeekDone(actual_browser_seek_time); |
| } |
| void BrowserDemuxerAndroid::OnDurationChanged(int demuxer_client_id, |
| const base::TimeDelta& duration) { |
| + DCHECK(task_runner_->BelongsToCurrentThread()); |
| + |
| media::DemuxerAndroidClient* client = |
| demuxer_clients_.Lookup(demuxer_client_id); |
| + |
| if (client) |
| client->OnDemuxerDurationChanged(duration); |
| + else |
| + pending_configs_[demuxer_client_id].duration = duration; |
| } |
| } // namespace content |