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

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

Issue 2712983004: Simplify/Cleanup MediaClient (Closed)
Patch Set: Fix test leak Created 3 years, 8 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>
(...skipping 230 matching lines...) Expand 10 before | Expand all | Expand 10 after
241 web_stream, media::BindToCurrentLoop(base::Bind( 241 web_stream, media::BindToCurrentLoop(base::Bind(
242 &WebMediaPlayerMS::OnSourceError, AsWeakPtr())), 242 &WebMediaPlayerMS::OnSourceError, AsWeakPtr())),
243 frame_deliverer_->GetRepaintCallback(), io_task_runner_, 243 frame_deliverer_->GetRepaintCallback(), io_task_runner_,
244 media_task_runner_, worker_task_runner_, gpu_factories_); 244 media_task_runner_, worker_task_runner_, gpu_factories_);
245 245
246 RenderFrame* const frame = RenderFrame::FromWebFrame(frame_); 246 RenderFrame* const frame = RenderFrame::FromWebFrame(frame_);
247 247
248 if (frame) { 248 if (frame) {
249 // Report UMA and RAPPOR metrics. 249 // Report UMA and RAPPOR metrics.
250 GURL url = source.isURL() ? GURL(source.getAsURL()) : GURL(); 250 GURL url = source.isURL() ? GURL(source.getAsURL()) : GURL();
251 media::ReportMetrics(load_type, url, frame_->getSecurityOrigin()); 251 media::ReportMetrics(load_type, url, frame_->getSecurityOrigin(),
252 media_log_);
252 253
253 audio_renderer_ = renderer_factory_->GetAudioRenderer( 254 audio_renderer_ = renderer_factory_->GetAudioRenderer(
254 web_stream, frame->GetRoutingID(), initial_audio_output_device_id_, 255 web_stream, frame->GetRoutingID(), initial_audio_output_device_id_,
255 initial_security_origin_); 256 initial_security_origin_);
256 } 257 }
257 258
258 if (!video_frame_provider_ && !audio_renderer_) { 259 if (!video_frame_provider_ && !audio_renderer_) {
259 SetNetworkState(WebMediaPlayer::NetworkStateNetworkError); 260 SetNetworkState(WebMediaPlayer::NetworkStateNetworkError);
260 return; 261 return;
261 } 262 }
(...skipping 440 matching lines...) Expand 10 before | Expand all | Expand 10 after
702 void WebMediaPlayerMS::ResetCanvasCache() { 703 void WebMediaPlayerMS::ResetCanvasCache() {
703 DCHECK(thread_checker_.CalledOnValidThread()); 704 DCHECK(thread_checker_.CalledOnValidThread());
704 video_renderer_.ResetCache(); 705 video_renderer_.ResetCache();
705 } 706 }
706 707
707 void WebMediaPlayerMS::TriggerResize() { 708 void WebMediaPlayerMS::TriggerResize() {
708 get_client()->sizeChanged(); 709 get_client()->sizeChanged();
709 } 710 }
710 711
711 } // namespace content 712 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/media/render_media_client_unittest.cc ('k') | content/renderer/render_frame_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698