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 06cd819a2c45fd4b759f3ba4b05fc28cdea772fb..e1bb4435a485c6a230ed70e1dbabcc38a5d44c75 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()));

[xhwang, 2016/04/29 00:04:17]

I guess we are more interested in who's failing than how it failed. So I am not
even using the |error| here. Or do we actually want to distinguish at least the
major error cases? e.g. decode error, not supported, etc.

[xhwang, 2016/04/29 00:13:14]

Just FWIW, the top errors are

PIPELINE_ERROR_ABORT
PIPELINE_ERROR_DECODE
PIPELINE_ERROR_NETWORK
PIPELINE_ERROR_READ

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