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

Side by Side Diff: content/renderer/media/webmediaplayer_ms.cc

Issue 2969093002: Make rendering of MediaStreams reflect changes to its set of tracks. (Closed)
Patch Set: Fix unit tests Created 3 years, 5 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/renderer/media/webmediaplayer_ms.h" 5 #include "content/renderer/media/webmediaplayer_ms.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <limits> 8 #include <limits>
9 #include <string> 9 #include <string>
10 #include <utility> 10 #include <utility>
11 11
12 #include "base/bind.h" 12 #include "base/bind.h"
13 #include "base/callback.h" 13 #include "base/callback.h"
14 #include "base/threading/thread_task_runner_handle.h" 14 #include "base/threading/thread_task_runner_handle.h"
15 #include "build/build_config.h" 15 #include "build/build_config.h"
16 #include "cc/blink/web_layer_impl.h" 16 #include "cc/blink/web_layer_impl.h"
17 #include "cc/layers/video_frame_provider_client_impl.h" 17 #include "cc/layers/video_frame_provider_client_impl.h"
18 #include "cc/layers/video_layer.h" 18 #include "cc/layers/video_layer.h"
19 #include "content/child/child_process.h" 19 #include "content/child/child_process.h"
20 #include "content/public/renderer/media_stream_audio_renderer.h" 20 #include "content/public/renderer/media_stream_audio_renderer.h"
21 #include "content/public/renderer/media_stream_renderer_factory.h" 21 #include "content/public/renderer/media_stream_renderer_factory.h"
22 #include "content/public/renderer/media_stream_video_renderer.h" 22 #include "content/public/renderer/media_stream_video_renderer.h"
23 #include "content/renderer/media/media_stream_audio_track.h"
24 #include "content/renderer/media/media_stream_video_track.h"
23 #include "content/renderer/media/web_media_element_source_utils.h" 25 #include "content/renderer/media/web_media_element_source_utils.h"
24 #include "content/renderer/media/webmediaplayer_ms_compositor.h" 26 #include "content/renderer/media/webmediaplayer_ms_compositor.h"
25 #include "content/renderer/render_frame_impl.h" 27 #include "content/renderer/render_frame_impl.h"
26 #include "content/renderer/render_thread_impl.h" 28 #include "content/renderer/render_thread_impl.h"
27 #include "media/base/bind_to_current_loop.h" 29 #include "media/base/bind_to_current_loop.h"
28 #include "media/base/media_content_type.h" 30 #include "media/base/media_content_type.h"
29 #include "media/base/media_log.h" 31 #include "media/base/media_log.h"
30 #include "media/base/video_frame.h" 32 #include "media/base/video_frame.h"
31 #include "media/base/video_rotation.h" 33 #include "media/base/video_rotation.h"
32 #include "media/base/video_types.h" 34 #include "media/base/video_types.h"
33 #include "media/blink/webmediaplayer_util.h" 35 #include "media/blink/webmediaplayer_util.h"
34 #include "services/ui/public/cpp/gpu/context_provider_command_buffer.h" 36 #include "services/ui/public/cpp/gpu/context_provider_command_buffer.h"
35 #include "third_party/WebKit/public/platform/WebMediaPlayerClient.h" 37 #include "third_party/WebKit/public/platform/WebMediaPlayerClient.h"
36 #include "third_party/WebKit/public/platform/WebMediaPlayerSource.h" 38 #include "third_party/WebKit/public/platform/WebMediaPlayerSource.h"
37 #include "third_party/WebKit/public/platform/WebRect.h" 39 #include "third_party/WebKit/public/platform/WebRect.h"
38 #include "third_party/WebKit/public/platform/WebSize.h" 40 #include "third_party/WebKit/public/platform/WebSize.h"
39 #include "third_party/WebKit/public/web/WebLocalFrame.h" 41 #include "third_party/WebKit/public/web/WebLocalFrame.h"
40 42
43 namespace {
44 enum class RendererReloadAction {
45 KEEP_RENDERER,
46 REMOVE_RENDERER,
47 NEW_RENDERER
48 };
49 } // namespace
50
41 namespace content { 51 namespace content {
42 52
43 // FrameDeliverer is responsible for delivering frames received on 53 // FrameDeliverer is responsible for delivering frames received on
44 // the IO thread by calling of EnqueueFrame() method of |compositor_|. 54 // the IO thread by calling of EnqueueFrame() method of |compositor_|.
45 // 55 //
46 // It is created on the main thread, but methods should be called and class 56 // It is created on the main thread, but methods should be called and class
47 // should be destructed on the IO thread. 57 // should be destructed on the IO thread.
48 class WebMediaPlayerMS::FrameDeliverer { 58 class WebMediaPlayerMS::FrameDeliverer {
49 public: 59 public:
50 FrameDeliverer(const base::WeakPtr<WebMediaPlayerMS>& player, 60 FrameDeliverer(const base::WeakPtr<WebMediaPlayerMS>& player,
(...skipping 132 matching lines...) Expand 10 before | Expand all | Expand 10 after
183 delegate_id_ = delegate_->AddObserver(this); 193 delegate_id_ = delegate_->AddObserver(this);
184 194
185 media_log_->AddEvent( 195 media_log_->AddEvent(
186 media_log_->CreateEvent(media::MediaLogEvent::WEBMEDIAPLAYER_CREATED)); 196 media_log_->CreateEvent(media::MediaLogEvent::WEBMEDIAPLAYER_CREATED));
187 } 197 }
188 198
189 WebMediaPlayerMS::~WebMediaPlayerMS() { 199 WebMediaPlayerMS::~WebMediaPlayerMS() {
190 DVLOG(1) << __func__; 200 DVLOG(1) << __func__;
191 DCHECK(thread_checker_.CalledOnValidThread()); 201 DCHECK(thread_checker_.CalledOnValidThread());
192 202
203 if (!web_stream_.IsNull()) {
204 MediaStream* native_stream = MediaStream::GetMediaStream(web_stream_);
205 if (native_stream)
206 native_stream->RemoveObserver(this);
207 }
208
193 // Destruct compositor resources in the proper order. 209 // Destruct compositor resources in the proper order.
194 get_client()->SetWebLayer(nullptr); 210 get_client()->SetWebLayer(nullptr);
195 if (video_weblayer_) 211 if (video_weblayer_)
196 static_cast<cc::VideoLayer*>(video_weblayer_->layer())->StopUsingProvider(); 212 static_cast<cc::VideoLayer*>(video_weblayer_->layer())->StopUsingProvider();
197 213
198 if (frame_deliverer_) 214 if (frame_deliverer_)
199 io_task_runner_->DeleteSoon(FROM_HERE, frame_deliverer_.release()); 215 io_task_runner_->DeleteSoon(FROM_HERE, frame_deliverer_.release());
200 216
201 if (compositor_) 217 if (compositor_)
202 compositor_->StopUsingProvider(); 218 compositor_->StopUsingProvider();
(...skipping 13 matching lines...) Expand all
216 232
217 void WebMediaPlayerMS::Load(LoadType load_type, 233 void WebMediaPlayerMS::Load(LoadType load_type,
218 const blink::WebMediaPlayerSource& source, 234 const blink::WebMediaPlayerSource& source,
219 CORSMode /*cors_mode*/) { 235 CORSMode /*cors_mode*/) {
220 DVLOG(1) << __func__; 236 DVLOG(1) << __func__;
221 DCHECK(thread_checker_.CalledOnValidThread()); 237 DCHECK(thread_checker_.CalledOnValidThread());
222 238
223 // TODO(acolwell): Change this to DCHECK_EQ(load_type, LoadTypeMediaStream) 239 // TODO(acolwell): Change this to DCHECK_EQ(load_type, LoadTypeMediaStream)
224 // once Blink-side changes land. 240 // once Blink-side changes land.
225 DCHECK_NE(load_type, kLoadTypeMediaSource); 241 DCHECK_NE(load_type, kLoadTypeMediaSource);
226 blink::WebMediaStream web_stream = 242 web_stream_ = GetWebMediaStreamFromWebMediaPlayerSource(source);
DaleCurtis 2017/07/31 16:22:40 How can this be null now? I don't see a change for
Guido Urdaneta 2017/07/31 16:30:23 Null check in line 235 from the original file sugg
DaleCurtis 2017/07/31 16:31:19 Ah missed that, thanks!
227 GetWebMediaStreamFromWebMediaPlayerSource(source); 243 if (!web_stream_.IsNull()) {
244 MediaStream* native_stream = MediaStream::GetMediaStream(web_stream_);
245 if (native_stream)
246 native_stream->AddObserver(this);
247 }
228 248
229 compositor_ = new WebMediaPlayerMSCompositor( 249 compositor_ = new WebMediaPlayerMSCompositor(
230 compositor_task_runner_, io_task_runner_, web_stream, AsWeakPtr()); 250 compositor_task_runner_, io_task_runner_, web_stream_, AsWeakPtr());
231 251
232 SetNetworkState(WebMediaPlayer::kNetworkStateLoading); 252 SetNetworkState(WebMediaPlayer::kNetworkStateLoading);
233 SetReadyState(WebMediaPlayer::kReadyStateHaveNothing); 253 SetReadyState(WebMediaPlayer::kReadyStateHaveNothing);
234 std::string stream_id = 254 std::string stream_id =
235 web_stream.IsNull() ? std::string() : web_stream.Id().Utf8(); 255 web_stream_.IsNull() ? std::string() : web_stream_.Id().Utf8();
236 media_log_->AddEvent(media_log_->CreateLoadEvent(stream_id)); 256 media_log_->AddEvent(media_log_->CreateLoadEvent(stream_id));
237 257
238 frame_deliverer_.reset(new WebMediaPlayerMS::FrameDeliverer( 258 frame_deliverer_.reset(new WebMediaPlayerMS::FrameDeliverer(
239 AsWeakPtr(), 259 AsWeakPtr(),
240 base::Bind(&WebMediaPlayerMSCompositor::EnqueueFrame, compositor_))); 260 base::Bind(&WebMediaPlayerMSCompositor::EnqueueFrame, compositor_)));
241 video_frame_provider_ = renderer_factory_->GetVideoRenderer( 261 video_frame_provider_ = renderer_factory_->GetVideoRenderer(
242 web_stream, media::BindToCurrentLoop(base::Bind( 262 web_stream_,
243 &WebMediaPlayerMS::OnSourceError, AsWeakPtr())), 263 media::BindToCurrentLoop(
264 base::Bind(&WebMediaPlayerMS::OnSourceError, AsWeakPtr())),
244 frame_deliverer_->GetRepaintCallback(), io_task_runner_, 265 frame_deliverer_->GetRepaintCallback(), io_task_runner_,
245 media_task_runner_, worker_task_runner_, gpu_factories_); 266 media_task_runner_, worker_task_runner_, gpu_factories_);
246 267
247 RenderFrame* const frame = RenderFrame::FromWebFrame(frame_); 268 RenderFrame* const frame = RenderFrame::FromWebFrame(frame_);
248 269
249 if (frame) { 270 if (frame) {
250 // Report UMA and RAPPOR metrics. 271 // Report UMA and RAPPOR metrics.
251 GURL url = source.IsURL() ? GURL(source.GetAsURL()) : GURL(); 272 GURL url = source.IsURL() ? GURL(source.GetAsURL()) : GURL();
252 media::ReportMetrics(load_type, url, frame_->GetSecurityOrigin(), 273 media::ReportMetrics(load_type, url, frame_->GetSecurityOrigin(),
253 media_log_.get()); 274 media_log_.get());
254 275
255 audio_renderer_ = renderer_factory_->GetAudioRenderer( 276 audio_renderer_ = renderer_factory_->GetAudioRenderer(
256 web_stream, frame->GetRoutingID(), initial_audio_output_device_id_, 277 web_stream_, frame->GetRoutingID(), initial_audio_output_device_id_,
257 initial_security_origin_); 278 initial_security_origin_);
258 } 279 }
259 280
260 if (!video_frame_provider_ && !audio_renderer_) { 281 if (!video_frame_provider_ && !audio_renderer_) {
261 SetNetworkState(WebMediaPlayer::kNetworkStateNetworkError); 282 SetNetworkState(WebMediaPlayer::kNetworkStateNetworkError);
262 return; 283 return;
263 } 284 }
264 285
265 if (audio_renderer_) { 286 if (audio_renderer_) {
266 audio_renderer_->SetVolume(volume_); 287 audio_renderer_->SetVolume(volume_);
267 audio_renderer_->Start(); 288 audio_renderer_->Start();
289
290 // Store the ID of audio track being played in |current_video_track_id_|
291
292 blink::WebVector<blink::WebMediaStreamTrack> audio_tracks;
293 if (!web_stream_.IsNull()) {
294 web_stream_.AudioTracks(audio_tracks);
295 DCHECK_GT(audio_tracks.size(), 0U);
296 current_audio_track_id_ = audio_tracks[0].Id();
297 }
268 } 298 }
269 if (video_frame_provider_) 299 if (video_frame_provider_) {
270 video_frame_provider_->Start(); 300 video_frame_provider_->Start();
301
302 // Store the ID of video track being played in |current_video_track_id_|
303 if (!web_stream_.IsNull()) {
304 blink::WebVector<blink::WebMediaStreamTrack> video_tracks;
305 web_stream_.VideoTracks(video_tracks);
306 DCHECK_GT(video_tracks.size(), 0U);
307 current_video_track_id_ = video_tracks[0].Id();
308 }
309 }
271 if (audio_renderer_ && !video_frame_provider_) { 310 if (audio_renderer_ && !video_frame_provider_) {
272 // This is audio-only mode. 311 // This is audio-only mode.
273 SetReadyState(WebMediaPlayer::kReadyStateHaveMetadata); 312 SetReadyState(WebMediaPlayer::kReadyStateHaveMetadata);
274 SetReadyState(WebMediaPlayer::kReadyStateHaveEnoughData); 313 SetReadyState(WebMediaPlayer::kReadyStateHaveEnoughData);
275 } 314 }
276 } 315 }
277 316
317 void WebMediaPlayerMS::TrackAdded(const blink::WebMediaStreamTrack& track) {
318 Reload();
319 }
320
321 void WebMediaPlayerMS::TrackRemoved(const blink::WebMediaStreamTrack& track) {
322 Reload();
323 }
324
325 void WebMediaPlayerMS::Reload() {
326 DCHECK(thread_checker_.CalledOnValidThread());
327 if (web_stream_.IsNull())
328 return;
329
330 ReloadVideo();
331 ReloadAudio();
332 }
333
334 void WebMediaPlayerMS::ReloadVideo() {
335 DCHECK(thread_checker_.CalledOnValidThread());
336 DCHECK(!web_stream_.IsNull());
337 blink::WebVector<blink::WebMediaStreamTrack> video_tracks;
338 // VideoTracks() is a getter.
339 web_stream_.VideoTracks(video_tracks);
340
341 RendererReloadAction renderer_action = RendererReloadAction::KEEP_RENDERER;
342 if (video_tracks.IsEmpty()) {
343 if (video_frame_provider_)
344 renderer_action = RendererReloadAction::REMOVE_RENDERER;
345 current_video_track_id_ = blink::WebString();
346 } else if (video_tracks[0].Id() != current_video_track_id_) {
347 renderer_action = RendererReloadAction::NEW_RENDERER;
348 current_video_track_id_ = video_tracks[0].Id();
349 }
350
351 switch (renderer_action) {
352 case RendererReloadAction::NEW_RENDERER:
353 if (video_frame_provider_)
354 video_frame_provider_->Stop();
355
356 video_frame_provider_ = renderer_factory_->GetVideoRenderer(
357 web_stream_,
358 media::BindToCurrentLoop(
359 base::Bind(&WebMediaPlayerMS::OnSourceError, AsWeakPtr())),
360 frame_deliverer_->GetRepaintCallback(), io_task_runner_,
361 media_task_runner_, worker_task_runner_, gpu_factories_);
362 DCHECK(video_frame_provider_);
363 video_frame_provider_->Start();
364 break;
365
366 case RendererReloadAction::REMOVE_RENDERER:
367 video_frame_provider_->Stop();
368 video_frame_provider_ = nullptr;
369 break;
370
371 default:
372 return;
373 }
374
375 DCHECK_NE(renderer_action, RendererReloadAction::KEEP_RENDERER);
376 if (!paused_)
377 delegate_->DidPlayerSizeChange(delegate_id_, NaturalSize());
378 }
379
380 void WebMediaPlayerMS::ReloadAudio() {
381 DCHECK(thread_checker_.CalledOnValidThread());
382 DCHECK(!web_stream_.IsNull());
383 RenderFrame* const frame = RenderFrame::FromWebFrame(frame_);
384 if (!frame)
385 return;
386
387 blink::WebVector<blink::WebMediaStreamTrack> audio_tracks;
388 // AudioTracks() is a getter.
389 web_stream_.AudioTracks(audio_tracks);
390
391 RendererReloadAction renderer_action = RendererReloadAction::KEEP_RENDERER;
392 if (audio_tracks.IsEmpty()) {
393 if (audio_renderer_)
394 renderer_action = RendererReloadAction::REMOVE_RENDERER;
395 current_audio_track_id_ = blink::WebString();
396 } else if (audio_tracks[0].Id() != current_video_track_id_) {
397 renderer_action = RendererReloadAction::NEW_RENDERER;
398 current_audio_track_id_ = audio_tracks[0].Id();
399 }
400
401 switch (renderer_action) {
402 case RendererReloadAction::NEW_RENDERER:
403 if (audio_renderer_)
404 audio_renderer_->Stop();
405
406 audio_renderer_ = renderer_factory_->GetAudioRenderer(
407 web_stream_, frame->GetRoutingID(), initial_audio_output_device_id_,
408 initial_security_origin_);
409 audio_renderer_->SetVolume(volume_);
410 audio_renderer_->Start();
411 audio_renderer_->Play();
412 break;
413
414 case RendererReloadAction::REMOVE_RENDERER:
415 audio_renderer_->Stop();
416 audio_renderer_ = nullptr;
417 break;
418
419 default:
420 break;
421 }
422 }
423
278 void WebMediaPlayerMS::Play() { 424 void WebMediaPlayerMS::Play() {
279 DVLOG(1) << __func__; 425 DVLOG(1) << __func__;
280 DCHECK(thread_checker_.CalledOnValidThread()); 426 DCHECK(thread_checker_.CalledOnValidThread());
281 427
282 media_log_->AddEvent(media_log_->CreateEvent(media::MediaLogEvent::PLAY)); 428 media_log_->AddEvent(media_log_->CreateEvent(media::MediaLogEvent::PLAY));
283 if (!paused_) 429 if (!paused_)
284 return; 430 return;
285 431
286 if (video_frame_provider_) 432 if (video_frame_provider_)
287 video_frame_provider_->Resume(); 433 video_frame_provider_->Resume();
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
374 return (video_frame_provider_.get() != nullptr); 520 return (video_frame_provider_.get() != nullptr);
375 } 521 }
376 522
377 bool WebMediaPlayerMS::HasAudio() const { 523 bool WebMediaPlayerMS::HasAudio() const {
378 DCHECK(thread_checker_.CalledOnValidThread()); 524 DCHECK(thread_checker_.CalledOnValidThread());
379 return (audio_renderer_.get() != nullptr); 525 return (audio_renderer_.get() != nullptr);
380 } 526 }
381 527
382 blink::WebSize WebMediaPlayerMS::NaturalSize() const { 528 blink::WebSize WebMediaPlayerMS::NaturalSize() const {
383 DCHECK(thread_checker_.CalledOnValidThread()); 529 DCHECK(thread_checker_.CalledOnValidThread());
530 if (!video_frame_provider_)
531 return blink::WebSize();
532
384 if (video_rotation_ == media::VIDEO_ROTATION_90 || 533 if (video_rotation_ == media::VIDEO_ROTATION_90 ||
385 video_rotation_ == media::VideoRotation::VIDEO_ROTATION_270) { 534 video_rotation_ == media::VideoRotation::VIDEO_ROTATION_270) {
386 const gfx::Size& current_size = compositor_->GetCurrentSize(); 535 const gfx::Size& current_size = compositor_->GetCurrentSize();
387 return blink::WebSize(current_size.height(), current_size.width()); 536 return blink::WebSize(current_size.height(), current_size.width());
388 } 537 }
389 return blink::WebSize(compositor_->GetCurrentSize()); 538 return blink::WebSize(compositor_->GetCurrentSize());
390 } 539 }
391 540
392 bool WebMediaPlayerMS::Paused() const { 541 bool WebMediaPlayerMS::Paused() const {
393 DCHECK(thread_checker_.CalledOnValidThread()); 542 DCHECK(thread_checker_.CalledOnValidThread());
(...skipping 327 matching lines...) Expand 10 before | Expand all | Expand 10 after
721 video_renderer_.ResetCache(); 870 video_renderer_.ResetCache();
722 } 871 }
723 872
724 void WebMediaPlayerMS::TriggerResize() { 873 void WebMediaPlayerMS::TriggerResize() {
725 get_client()->SizeChanged(); 874 get_client()->SizeChanged();
726 875
727 delegate_->DidPlayerSizeChange(delegate_id_, NaturalSize()); 876 delegate_->DidPlayerSizeChange(delegate_id_, NaturalSize());
728 } 877 }
729 878
730 } // namespace content 879 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/media/webmediaplayer_ms.h ('k') | content/shell/test_runner/mock_web_user_media_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698