Add histogram WebRTC.UserMediaRequest.Result to track getUserMedia success and failure reasons.

BUG=399835

Review URL: https://codereview.chromium.org/427713004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287614 0039d316-1c4b-4281-b951-d872f2087c98

content/renderer/media/media_stream_impl.cc

@@ -601,12 +601,14 @@ void MediaStreamImpl::GetUserMediaRequestSucceeded(
     const blink::WebMediaStream& stream,
     blink::WebUserMediaRequest* request_info) {
   DVLOG(1) << "MediaStreamImpl::GetUserMediaRequestSucceeded";
+  LogUserMediaRequestResult(MEDIA_DEVICE_OK);
   request_info->requestSucceeded(stream);
 }
 
 void MediaStreamImpl::GetUserMediaRequestFailed(
     blink::WebUserMediaRequest* request_info,
     content::MediaStreamRequestResult result) {
+  LogUserMediaRequestResult(result);
   switch (result) {
     case MEDIA_DEVICE_OK:
       NOTREACHED();

content/renderer/media/webrtc_uma_histograms.cc

@@ -8,6 +8,11 @@
 
 namespace content {
 
+void LogUserMediaRequestResult(MediaStreamRequestResult result) {
+  UMA_HISTOGRAM_ENUMERATION(
+      "WebRTC.UserMediaRequest.Result", result, NUM_MEDIA_REQUEST_RESULTS);
+}
+
 void UpdateWebRTCMethodCount(JavaScriptAPIName api_name) {
   DVLOG(3) << "Incrementing WebRTC.webkitApiCount for " << api_name;
   UMA_HISTOGRAM_ENUMERATION("WebRTC.webkitApiCount", api_name, INVALID_NAME);

content/renderer/media/webrtc_uma_histograms.h

@@ -8,9 +8,12 @@
 #include "base/memory/singleton.h"
 #include "base/threading/non_thread_safe.h"
 #include "content/common/content_export.h"
+#include "content/public/common/media_stream_request.h"
 
 namespace content {
 
+void LogUserMediaRequestResult(MediaStreamRequestResult result);
+
 // Helper enum used for histogramming calls to WebRTC APIs from JavaScript.
 enum JavaScriptAPIName {
   WEBKIT_GET_USER_MEDIA,

tools/metrics/histograms/histograms.xml

@@ -35733,6 +35733,14 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   </summary>
 </histogram>
 
+<histogram name="WebRTC.UserMediaRequest.Result"
+    enum="MediaStreamRequestResult">
+  <owner>andresp@chromium.org</owner>
+  <summary>
+    Counters for UserMediaRequests results such as failure reasons.
+  </summary>
+</histogram>
+
 <histogram name="WebRTC.webkitApiCount" enum="JavaScriptAPIName">
   <owner>perkj@chromium.org</owner>
   <summary>Counts number of calls to WebRTC APIs from JavaScript.</summary>
@@ -43188,6 +43196,18 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   <int value="2" label="All external links protected"/>
 </enum>
 
+<enum name="MediaStreamRequestResult" type="int">
+  <int value="0" label="Ok"/>
+  <int value="1" label="Permission Denied"/>
+  <int value="2" label="Permission Dismissed"/>
+  <int value="3" label="Invalid State"/>
+  <int value="4" label="No Hardware"/>
+  <int value="5" label="Invalid Security Origin"/>
+  <int value="6" label="Tab Capture Failure"/>
+  <int value="7" label="Capture Failure"/>
+  <int value="8" label="Track Start Failure"/>
+</enum>
+
 <enum name="MetaTagTypeEnum" type="int">
   <int value="0" label="No viewport tag"/>
   <int value="1" label="Viewport meta with device width"/>