Bug 1314445 - Remove unused EME notification handlers from GMPProvider.jsm. r=spohl draft
authorChris Pearce <cpearce@mozilla.com>
Wed, 02 Nov 2016 13:08:08 +1300
changeset 432843 88b11bbbe0fffb434d7699f9c720c12ab7797b5c
parent 432842 50eb62c487aa88e687fa6a6ff30cdfd762aa51a7
child 432844 09a5b25939bf0f29ee23669c887fd24eff9cd810
push id34444
push usercpearce@mozilla.com
push dateWed, 02 Nov 2016 22:51:27 +0000
reviewersspohl
bugs1314445
milestone52.0a1
Bug 1314445 - Remove unused EME notification handlers from GMPProvider.jsm. r=spohl Gecko no longer emits this notification, so there's no point handling it in GMPProvider. MozReview-Commit-ID: BIvQCXN7LhF
toolkit/mozapps/extensions/internal/GMPProvider.jsm
--- a/toolkit/mozapps/extensions/internal/GMPProvider.jsm
+++ b/toolkit/mozapps/extensions/internal/GMPProvider.jsm
@@ -407,17 +407,17 @@ GMPWrapper.prototype = {
     let parsedData;
     try {
       parsedData = JSON.parse(data);
     } catch (ex) {
       this._log.error("Malformed EME video message with data: " + data);
       return;
     }
     let {status: status, keySystem: keySystem} = parsedData;
-    if (status == "cdm-not-installed" || status == "cdm-insufficient-version") {
+    if (status == "cdm-not-installed") {
       this.checkForUpdates(0);
     }
   },
 
   onPrefEnabledChanged: function() {
     if (!this._plugin.isEME || !this.appDisabled) {
       this._handleEnabledChanged();
     }