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..9669387d98da85bc74edb6f666db5350b7a5d6f8 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_source_player.h" |
+#include "media/base/media_switches.h" |
namespace content { |
@@ -42,7 +45,7 @@ class BrowserDemuxerAndroid::Internal : public media::DemuxerAndroid { |
private: |
// Helper for DCHECKing that the ID is still registered. |
bool ClientIDExists() { |
- return demuxer_->demuxer_clients_.Lookup(demuxer_client_id_); |
+ return demuxer_->Lookup(demuxer_client_id_); |
} |
scoped_refptr<BrowserDemuxerAndroid> demuxer_; |
@@ -52,23 +55,32 @@ class BrowserDemuxerAndroid::Internal : public media::DemuxerAndroid { |
}; |
BrowserDemuxerAndroid::BrowserDemuxerAndroid() |
- : BrowserMessageFilter(MediaPlayerMsgStart) {} |
+ : BrowserMessageFilter(MediaPlayerMsgStart) { |
+ enable_media_thread_for_media_playback_ = |
+ base::CommandLine::ForCurrentProcess()-> |
+ HasSwitch(switches::kEnableMediaThreadForMediaPlayback); |
+ |
+} |
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 enable_media_thread_for_media_playback_ ? |
+ media::GetMediaTaskRunner().get() : |
+ BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI).get(); |
} |
+ |
+ return nullptr; |
} |
+ |
bool BrowserDemuxerAndroid::OnMessageReceived(const IPC::Message& message) { |
bool handled = true; |
IPC_BEGIN_MESSAGE_MAP(BrowserDemuxerAndroid, message) |
@@ -95,20 +107,29 @@ void BrowserDemuxerAndroid::AddDemuxerClient( |
media::DemuxerAndroidClient* client) { |
DVLOG(1) << __FUNCTION__ << " peer_pid=" << peer_pid() |
<< " demuxer_client_id=" << demuxer_client_id; |
+ |
+ base::AutoLock lock(lock_); |
demuxer_clients_.AddWithID(client, demuxer_client_id); |
} |
void BrowserDemuxerAndroid::RemoveDemuxerClient(int demuxer_client_id) { |
DVLOG(1) << __FUNCTION__ << " peer_pid=" << peer_pid() |
<< " demuxer_client_id=" << demuxer_client_id; |
+ |
+ base::AutoLock lock(lock_); |
demuxer_clients_.Remove(demuxer_client_id); |
} |
+media::DemuxerAndroidClient* |
+BrowserDemuxerAndroid::Lookup(int demuxer_client_id) { |
+ base::AutoLock lock(lock_); |
+ return demuxer_clients_.Lookup(demuxer_client_id); |
+} |
+ |
void BrowserDemuxerAndroid::OnDemuxerReady( |
int demuxer_client_id, |
const media::DemuxerConfigs& configs) { |
- media::DemuxerAndroidClient* client = |
- demuxer_clients_.Lookup(demuxer_client_id); |
+ media::DemuxerAndroidClient* client = Lookup(demuxer_client_id); |
if (client) |
client->OnDemuxerConfigsAvailable(configs); |
} |
@@ -116,8 +137,7 @@ void BrowserDemuxerAndroid::OnDemuxerReady( |
void BrowserDemuxerAndroid::OnReadFromDemuxerAck( |
int demuxer_client_id, |
const media::DemuxerData& data) { |
- media::DemuxerAndroidClient* client = |
- demuxer_clients_.Lookup(demuxer_client_id); |
+ media::DemuxerAndroidClient* client = Lookup(demuxer_client_id); |
if (client) |
client->OnDemuxerDataAvailable(data); |
} |
@@ -125,16 +145,14 @@ void BrowserDemuxerAndroid::OnReadFromDemuxerAck( |
void BrowserDemuxerAndroid::OnDemuxerSeekDone( |
int demuxer_client_id, |
const base::TimeDelta& actual_browser_seek_time) { |
- media::DemuxerAndroidClient* client = |
- demuxer_clients_.Lookup(demuxer_client_id); |
+ media::DemuxerAndroidClient* client = Lookup(demuxer_client_id); |
if (client) |
client->OnDemuxerSeekDone(actual_browser_seek_time); |
} |
void BrowserDemuxerAndroid::OnDurationChanged(int demuxer_client_id, |
const base::TimeDelta& duration) { |
- media::DemuxerAndroidClient* client = |
- demuxer_clients_.Lookup(demuxer_client_id); |
+ media::DemuxerAndroidClient* client = Lookup(demuxer_client_id); |
if (client) |
client->OnDemuxerDurationChanged(duration); |
} |