Bug 1335555 - Removes a call to telemetry that is no longer needed r?cpearce draft
authorJay Harris <jharris@mozilla.com>
Wed, 01 Feb 2017 14:15:15 +1300
changeset 469182 176295d50c4b79e450a6b8406bbcfdad5c8b5120
parent 468903 8fe75ecf1777f02f366a6445a33030d71c7ff765
child 544120 530a8ffb02496f8faecf0af57f9b9eb0347bf9b7
push id43642
push userbmo:jharris@mozilla.com
push dateWed, 01 Feb 2017 20:19:34 +0000
reviewerscpearce
bugs1335555
milestone54.0a1
Bug 1335555 - Removes a call to telemetry that is no longer needed r?cpearce MozReview-Commit-ID: Fmi29mNFBL4
dom/media/eme/MediaKeySession.cpp
toolkit/components/telemetry/Histograms.json
--- a/dom/media/eme/MediaKeySession.cpp
+++ b/dom/media/eme/MediaKeySession.cpp
@@ -305,19 +305,16 @@ MediaKeySession::GenerateRequest(const n
   }
 
   // Let sanitized init data be a validated and sanitized version of init data.
 
   // If sanitized init data is empty, reject promise with a NotSupportedError.
 
   // Note: Remaining steps of generateRequest method continue in CDM.
 
-  Telemetry::Accumulate(Telemetry::VIDEO_CDM_GENERATE_REQUEST_CALLED,
-                        ToCDMTypeTelemetryEnum(mKeySystem));
-
   // Convert initData to hex for easier logging.
   // Note: CreateSession() Move()s the data out of the array, so we have
   // to copy it here.
   nsAutoCString hexInitData(ToHexString(data));
   PromiseId pid = mKeys->StorePromise(promise);
   mKeys->ConnectPendingPromiseIdWithToken(pid, Token());
   mKeys->GetCDMProxy()->CreateSession(Token(),
                                       mSessionType,
--- a/toolkit/components/telemetry/Histograms.json
+++ b/toolkit/components/telemetry/Histograms.json
@@ -9238,25 +9238,16 @@
     "alert_emails": ["cpearce@mozilla.com"],
     "expires_in_version": "58",
     "bug_numbers": [1304207],
     "kind": "enumerated",
     "n_values": 6,
     "description": "Note the type of CDM (0=ClearKey, 1=Primetime, 2=Widevine, 3=unknown) every time we successfully instantiate an EME MediaKeys object.",
     "releaseChannelCollection": "opt-out"
   },
-  "VIDEO_CDM_GENERATE_REQUEST_CALLED": {
-    "alert_emails": ["cpearce@mozilla.com"],
-    "expires_in_version": "58",
-    "bug_numbers": [1305552],
-    "kind": "enumerated",
-    "n_values": 6,
-    "description": "Note the type of CDM (0=ClearKey, 1=Primetime, 2=Widevine, 3=unknown) every time we call MediaKeySession.generateRequest().",
-    "releaseChannelCollection": "opt-out"
-  },
   "MEDIA_CODEC_USED": {
     "alert_emails": ["cpearce@mozilla.com"],
     "expires_in_version": "never",
     "keyed": true,
     "kind": "count",
     "description": "Count of use of audio/video codecs in HTMLMediaElements and WebAudio. Those with 'resource' prefix are approximate; report based on HTTP ContentType or sniffing. Those with 'webaudio' prefix are for WebAudio."
   },
   "FX_SANITIZE_TOTAL": {