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()));

[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);
 }
 
 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