Index: content/browser/media/android/browser_media_player_manager.cc |
diff --git a/content/browser/media/android/browser_media_player_manager.cc b/content/browser/media/android/browser_media_player_manager.cc |
index 68268f7406d215d98953787c36061d5b870c0f8f..20fd34e89de28e72e243f44ec986366dcabc57af 100644 |
--- a/content/browser/media/android/browser_media_player_manager.cc |
+++ b/content/browser/media/android/browser_media_player_manager.cc |
@@ -25,13 +25,11 @@ |
#include "content/public/browser/web_contents_delegate.h" |
#include "content/public/common/content_client.h" |
#include "content/public/common/content_switches.h" |
-#include "media/base/android/media_codec_player.h" |
#include "media/base/android/media_player_bridge.h" |
#include "media/base/android/media_source_player.h" |
#include "media/base/android/media_url_interceptor.h" |
#include "media/base/media_switches.h" |
-using media::MediaCodecPlayer; |
using media::MediaPlayerAndroid; |
using media::MediaPlayerBridge; |
using media::MediaPlayerManager; |
@@ -115,24 +113,13 @@ |
} |
case MEDIA_PLAYER_TYPE_MEDIA_SOURCE: { |
- if (base::CommandLine::ForCurrentProcess()-> |
- HasSwitch(switches::kEnableMediaThreadForMediaPlayback)) { |
- return new MediaCodecPlayer( |
- media_player_params.player_id, |
- manager, |
- base::Bind(&BrowserMediaPlayerManager::OnMediaResourcesRequested, |
- weak_ptr_factory_.GetWeakPtr()), |
- demuxer->CreateDemuxer(media_player_params.demuxer_client_id), |
- media_player_params.frame_url); |
- } else { |
- return new MediaSourcePlayer( |
- media_player_params.player_id, |
- manager, |
- base::Bind(&BrowserMediaPlayerManager::OnMediaResourcesRequested, |
- weak_ptr_factory_.GetWeakPtr()), |
- demuxer->CreateDemuxer(media_player_params.demuxer_client_id), |
- media_player_params.frame_url); |
- } |
+ return new MediaSourcePlayer( |
+ media_player_params.player_id, |
+ manager, |
+ base::Bind(&BrowserMediaPlayerManager::OnMediaResourcesRequested, |
+ weak_ptr_factory_.GetWeakPtr()), |
+ demuxer->CreateDemuxer(media_player_params.demuxer_client_id), |
+ media_player_params.frame_url); |
} |
} |
@@ -156,11 +143,6 @@ |
// (e.g. the WebContents may be destroyed before the render process). So |
// we cannot DCHECK(players_.empty()) here. Instead, all media players in |
// |players_| will be destroyed here because |player_| is a ScopedVector. |
- |
- for (MediaPlayerAndroid* player : players_) |
- player->DeleteOnCorrectThread(); |
- |
- players_.weak_clear(); |
} |
void BrowserMediaPlayerManager::ExitFullscreen(bool release_media_player) { |
@@ -548,8 +530,7 @@ |
it != players_.end(); ++it) { |
if ((*it)->player_id() == player_id) { |
ReleaseMediaResources(player_id); |
- (*it)->DeleteOnCorrectThread(); |
- players_.weak_erase(it); |
+ players_.erase(it); |
audio_monitor_->RemovePlayer(render_frame_host_, player_id); |
break; |
} |