media: Report RAPPOR for media pipeline errors

media/blink/webmediaplayer_util.cc

Index: media/blink/webmediaplayer_util.cc
diff --git a/media/blink/webmediaplayer_util.cc b/media/blink/webmediaplayer_util.cc
index 981cba22c7ace321729a2d748eb161842e324a04..297ff6827f975a310d22c6943822841d9a632e86 100644
--- a/media/blink/webmediaplayer_util.cc
+++ b/media/blink/webmediaplayer_util.cc
@@ -139,6 +139,20 @@ void ReportMetrics(blink::WebMediaPlayer::LoadType load_type,
   }
 }
 
+void ReportPipelineError(blink::WebMediaPlayer::LoadType load_type,
+                         const blink::WebSecurityOrigin& security_origin,
+                         PipelineStatus error) {
+  DCHECK_NE(PIPELINE_OK, error);
+
+  // Report the origin from where the media player is created.
+  if (!GetMediaClient())
+    return;
+
+  GetMediaClient()->RecordRapporURL(
+      "Media.OriginUrl." + LoadTypeToString(load_type) + "PipelineError",
+      blink::WebStringToGURL(security_origin.toString()));

[Ilya Sherman, 2016/04/29 20:25:30]

Hmm, would it make sense to just have a single metric, and send the load type as
an additional param for the metric?  Steve recently added this functionality,
and it seems like a good match for what you're trying to do here.

(OTOH, I see that you already have other metrics broken down along similar axes,
so maybe it makes more sense to remain parallel with those.)

+}
+
 void RecordOriginOfHLSPlayback(const GURL& origin_url) {
   if (media::GetMediaClient())
     GetMediaClient()->RecordRapporURL("Media.OriginUrl.HLS", origin_url);