media: Report RAPPOR for media pipeline errors

media/blink/webmediaplayer_util.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/blink/webmediaplayer_util.h"
 
 #include <math.h>
 #include <stddef.h>
 #include <string>
 #include <utility>
@@ skipping 121 matching lines @@
         GetMediaClient()->RecordRapporURL("Media.OriginUrl.MSE.Secure",
                                           security_origin_url);
       } else {
         GetMediaClient()->RecordRapporURL("Media.OriginUrl.MSE.Insecure",
                                           security_origin_url);
       }
     }
   }
 }
 
+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);
 }
 
 EmeInitDataType ConvertToEmeInitDataType(
     blink::WebEncryptedMediaInitDataType init_data_type) {
   switch (init_data_type) {
     case blink::WebEncryptedMediaInitDataType::Webm:
       return EmeInitDataType::WEBM;
@@ skipping 83 matching lines @@
 
 }  // namespace
 
 OutputDeviceStatusCB ConvertToOutputDeviceStatusCB(
     blink::WebSetSinkIdCallbacks* web_callbacks) {
   return media::BindToCurrentLoop(
       base::Bind(RunSetSinkIdCallback, SetSinkIdCallback(web_callbacks)));
 }
 
 }  // namespace media