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

Side by Side Diff: content/browser/media/android/browser_media_player_manager.cc

Issue 1640123004: Propagate media session id into MediaPlayerAndroid (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@pass-media-session-id-over-ipc
Patch Set: Fix stray kDefaultMediaSessionID 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "content/browser/media/android/browser_media_player_manager.h" 5 #include "content/browser/media/android/browser_media_player_manager.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/android/scoped_java_ref.h" 9 #include "base/android/scoped_java_ref.h"
10 #include "content/browser/frame_host/render_frame_host_impl.h" 10 #include "content/browser/frame_host/render_frame_host_impl.h"
(...skipping 126 matching lines...) Expand 10 before | Expand all | Expand 10 after
137 137
138 MediaPlayerAndroid* BrowserMediaPlayerManager::CreateMediaPlayer( 138 MediaPlayerAndroid* BrowserMediaPlayerManager::CreateMediaPlayer(
139 const MediaPlayerHostMsg_Initialize_Params& media_player_params, 139 const MediaPlayerHostMsg_Initialize_Params& media_player_params,
140 bool hide_url_log, 140 bool hide_url_log,
141 BrowserDemuxerAndroid* demuxer) { 141 BrowserDemuxerAndroid* demuxer) {
142 switch (media_player_params.type) { 142 switch (media_player_params.type) {
143 case MEDIA_PLAYER_TYPE_REMOTE_ONLY: 143 case MEDIA_PLAYER_TYPE_REMOTE_ONLY:
144 case MEDIA_PLAYER_TYPE_URL: { 144 case MEDIA_PLAYER_TYPE_URL: {
145 const std::string user_agent = GetContentClient()->GetUserAgent(); 145 const std::string user_agent = GetContentClient()->GetUserAgent();
146 MediaPlayerBridge* media_player_bridge = new MediaPlayerBridge( 146 MediaPlayerBridge* media_player_bridge = new MediaPlayerBridge(
147 media_player_params.player_id, 147 media_player_params.player_id, media_player_params.url,
148 media_player_params.url, 148 media_player_params.first_party_for_cookies, user_agent, hide_url_log,
149 media_player_params.first_party_for_cookies,
150 user_agent,
151 hide_url_log,
152 this, 149 this,
153 base::Bind(&BrowserMediaPlayerManager::OnDecoderResourcesReleased, 150 base::Bind(&BrowserMediaPlayerManager::OnDecoderResourcesReleased,
154 weak_ptr_factory_.GetWeakPtr()), 151 weak_ptr_factory_.GetWeakPtr()),
155 media_player_params.frame_url, 152 media_player_params.frame_url, media_player_params.allow_credentials,
156 media_player_params.allow_credentials); 153 media_player_params.media_session_id);
157 154
158 if (media_player_params.type == MEDIA_PLAYER_TYPE_REMOTE_ONLY) 155 if (media_player_params.type == MEDIA_PLAYER_TYPE_REMOTE_ONLY)
159 return media_player_bridge; 156 return media_player_bridge;
160 157
161 bool should_block = false; 158 bool should_block = false;
162 bool extract_metadata = 159 bool extract_metadata =
163 // Initialize the player will cause MediaMetadataExtractor to decode 160 // Initialize the player will cause MediaMetadataExtractor to decode
164 // small chunks of data. 161 // small chunks of data.
165 RequestDecoderResources(media_player_params.player_id, true); 162 RequestDecoderResources(media_player_params.player_id, true);
166 #if !defined(USE_AURA) 163 #if !defined(USE_AURA)
(...skipping 16 matching lines...) Expand all
183 0, 0, false); 180 0, 0, false);
184 } else if (!should_block) { 181 } else if (!should_block) {
185 media_player_bridge->Initialize(); 182 media_player_bridge->Initialize();
186 } 183 }
187 return media_player_bridge; 184 return media_player_bridge;
188 } 185 }
189 186
190 case MEDIA_PLAYER_TYPE_MEDIA_SOURCE: { 187 case MEDIA_PLAYER_TYPE_MEDIA_SOURCE: {
191 if (media::UseMediaThreadForMediaPlayback()) { 188 if (media::UseMediaThreadForMediaPlayback()) {
192 return new MediaCodecPlayer( 189 return new MediaCodecPlayer(
193 media_player_params.player_id, 190 media_player_params.player_id, weak_ptr_factory_.GetWeakPtr(),
194 weak_ptr_factory_.GetWeakPtr(),
195 base::Bind(&BrowserMediaPlayerManager::OnDecoderResourcesReleased, 191 base::Bind(&BrowserMediaPlayerManager::OnDecoderResourcesReleased,
196 weak_ptr_factory_.GetWeakPtr()), 192 weak_ptr_factory_.GetWeakPtr()),
197 demuxer->CreateDemuxer(media_player_params.demuxer_client_id), 193 demuxer->CreateDemuxer(media_player_params.demuxer_client_id),
198 media_player_params.frame_url); 194 media_player_params.frame_url,
195 media_player_params.media_session_id);
199 } else { 196 } else {
200 return new MediaSourcePlayer( 197 return new MediaSourcePlayer(
201 media_player_params.player_id, 198 media_player_params.player_id, this,
202 this,
203 base::Bind(&BrowserMediaPlayerManager::OnDecoderResourcesReleased, 199 base::Bind(&BrowserMediaPlayerManager::OnDecoderResourcesReleased,
204 weak_ptr_factory_.GetWeakPtr()), 200 weak_ptr_factory_.GetWeakPtr()),
205 demuxer->CreateDemuxer(media_player_params.demuxer_client_id), 201 demuxer->CreateDemuxer(media_player_params.demuxer_client_id),
206 media_player_params.frame_url); 202 media_player_params.frame_url,
203 media_player_params.media_session_id);
207 } 204 }
208 } 205 }
209 } 206 }
210 207
211 NOTREACHED(); 208 NOTREACHED();
212 return NULL; 209 return NULL;
213 } 210 }
214 211
215 BrowserMediaPlayerManager::BrowserMediaPlayerManager( 212 BrowserMediaPlayerManager::BrowserMediaPlayerManager(
216 RenderFrameHost* render_frame_host) 213 RenderFrameHost* render_frame_host)
(...skipping 523 matching lines...) Expand 10 before | Expand all | Expand 10 after
740 if (!player) 737 if (!player)
741 return; 738 return;
742 player->Start(); 739 player->Start();
743 if (fullscreen_player_id_ == player_id && fullscreen_player_is_released_) { 740 if (fullscreen_player_id_ == player_id && fullscreen_player_is_released_) {
744 video_view_->OpenVideo(); 741 video_view_->OpenVideo();
745 fullscreen_player_is_released_ = false; 742 fullscreen_player_is_released_ = false;
746 } 743 }
747 } 744 }
748 745
749 } // namespace content 746 } // namespace content
OLDNEW
« no previous file with comments | « chrome/browser/media/android/remote/remote_media_player_bridge.cc ('k') | media/base/android/media_codec_player.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698