Bug 1360448 - Only ignore OPTIONS_TYPE_INLINE_BROWSER instead of ignoring all types which aren't OPTIONS_TYPE_INLINE r?kmag draft
authorMatthew Wein <mwein@mozilla.com>
Wed, 10 May 2017 22:54:40 -0400
changeset 575964 100adc700d015a21a8dfe5b4bcc8078072c6273f
parent 575624 ce2218406119c36a551e3faea4e192186ee46cc5
child 628054 7ee47410049b5d6486c6f263875665ee0661cf89
push id58208
push usermwein@mozilla.com
push dateThu, 11 May 2017 03:09:07 +0000
reviewerskmag
bugs1360448
milestone55.0a1
Bug 1360448 - Only ignore OPTIONS_TYPE_INLINE_BROWSER instead of ignoring all types which aren't OPTIONS_TYPE_INLINE r?kmag MozReview-Commit-ID: Ks3Yryy22lL
mobile/android/chrome/content/aboutAddons.js
--- a/mobile/android/chrome/content/aboutAddons.js
+++ b/mobile/android/chrome/content/aboutAddons.js
@@ -220,25 +220,21 @@ var Addons = {
   },
 
   _createItemForAddon: function _createItemForAddon(aAddon) {
     let appManaged = (aAddon.scope == AddonManager.SCOPE_APPLICATION);
     let opType = this._getOpTypeForOperations(aAddon.pendingOperations);
     let updateable = (aAddon.permissions & AddonManager.PERM_CAN_UPGRADE) > 0;
     let uninstallable = (aAddon.permissions & AddonManager.PERM_CAN_UNINSTALL) > 0;
 
-    let optionsURL;
-    switch (parseInt(aAddon.optionsType)) {
-      case AddonManager.OPTIONS_TYPE_INLINE:
-        optionsURL = aAddon.optionsURL || "";
-        break;
-      default:
-        // Bug 1302504 - Ignore WebExtension options (which are type OPTIONS_TYPE_INLINE_BROWSER)
-        // until options_ui support is added - Bug 1302504.
-        optionsURL = "";
+    let optionsURL = aAddon.optionsURL || "";
+
+    if (aAddon.optionsType == AddonManager.OPTIONS_TYPE_INLINE_BROWSER) {
+      // Ignore OPTIONS_TYPE_INLINE_BROWSER until support is added in bug 1302504.
+      optionsURL = "";
     }
 
     let blocked = "";
     switch(aAddon.blocklistState) {
       case Ci.nsIBlocklistService.STATE_BLOCKED:
         blocked = "blocked";
         break;
       case Ci.nsIBlocklistService.STATE_SOFTBLOCKED: