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

Unified Diff: media/blink/webmediaplayer_impl.cc

Issue 2712983004: Simplify/Cleanup MediaClient (Closed)
Patch Set: Little fixes Created 3 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 side-by-side diff with in-line comments
Download patch
Index: media/blink/webmediaplayer_impl.cc
diff --git a/media/blink/webmediaplayer_impl.cc b/media/blink/webmediaplayer_impl.cc
index 97c5b7f87324732622838bd82182c5c15acdcffe..ad77d954bdd40e6e4e0b0aa4d8c06ebf2d6679a3 100644
--- a/media/blink/webmediaplayer_impl.cc
+++ b/media/blink/webmediaplayer_impl.cc
@@ -394,7 +394,7 @@ void WebMediaPlayerImpl::DoLoad(LoadType load_type,
DCHECK(main_task_runner_->BelongsToCurrentThread());
GURL gurl(url);
- ReportMetrics(load_type, gurl, frame_->getSecurityOrigin());
+ ReportMetrics(load_type, gurl, frame_->getSecurityOrigin(), media_log_);
// Set subresource URL for crash reporting.
base::debug::SetCrashKeyValue("subresource_url", gurl.spec());
@@ -1207,7 +1207,8 @@ void WebMediaPlayerImpl::OnError(PipelineStatus status) {
if (suppress_destruction_errors_)
return;
- ReportPipelineError(load_type_, frame_->getSecurityOrigin(), status);
+ ReportPipelineError(load_type_, frame_->getSecurityOrigin(), status,
+ media_log_);
media_log_->AddEvent(media_log_->CreatePipelineErrorEvent(status));
if (ready_state_ == WebMediaPlayer::ReadyStateHaveNothing) {

Powered by Google App Engine
This is Rietveld 408576698