Bug 1335555 - Removes a call to telemetry that is no longer needed r?cpearce
MozReview-Commit-ID: Fmi29mNFBL4
--- 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": {