Bug 1312416 - use tabbrowser message manager group to listen for decoder doctor messages, r?jaws
For browsers in e.g. panels, we can't really make decoder doctor work well, so we simply elect to only listen for such messages within tabbrowser browsers.
MozReview-Commit-ID: Frrhf6AOgAy
--- a/browser/base/content/browser-media.js
+++ b/browser/base/content/browser-media.js
@@ -351,14 +351,14 @@ let gDecoderDoctorHandler = {
// first time we get this resolution -> Clear prefs and report telemetry.
Services.prefs.clearUserPref(formatsPref);
Services.prefs.clearUserPref(buttonClickedPref);
histogram.add(decoderDoctorReportId, TELEMETRY_DDSTAT_SOLVED);
}
},
}
-window.messageManager.addMessageListener("DecoderDoctor:Notification", gDecoderDoctorHandler);
-window.messageManager.addMessageListener("EMEVideo:ContentMediaKeysRequest", gEMEHandler);
+window.getGroupMessageManager("browsers").addMessageListener("DecoderDoctor:Notification", gDecoderDoctorHandler);
+window.getGroupMessageManager("browsers").addMessageListener("EMEVideo:ContentMediaKeysRequest", gEMEHandler);
window.addEventListener("unload", function() {
- window.messageManager.removeMessageListener("EMEVideo:ContentMediaKeysRequest", gEMEHandler);
- window.messageManager.removeMessageListener("DecoderDoctor:Notification", gDecoderDoctorHandler);
+ window.getGroupMessageManager("browsers").removeMessageListener("EMEVideo:ContentMediaKeysRequest", gEMEHandler);
+ window.getGroupMessageManager("browsers").removeMessageListener("DecoderDoctor:Notification", gDecoderDoctorHandler);
}, false);