Bug 1329027 - Fix regression caused by bug 1300808 r=kmag a=gchang
authorMatthew Wein <mwein@mozilla.com>
Wed, 15 Mar 2017 15:57:38 -0400
changeset 379257 1b0b9e1915d147b4e904ac07a5f16677035600ce
parent 379256 6188b6cab49e2bfa036b24d8112cda5f5f9416f0
child 379258 404583ced67a4c7c1c9c8ed538de5f9d2fc18b3b
push id1419
push userjlund@mozilla.com
push dateMon, 10 Apr 2017 20:44:07 +0000
treeherdermozilla-release@5e6801b73ef6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerskmag, gchang
bugs1329027, 1300808
milestone53.0
Bug 1329027 - Fix regression caused by bug 1300808 r=kmag a=gchang MozReview-Commit-ID: CM5cyueJcNk
mobile/android/chrome/content/aboutAddons.js
--- a/mobile/android/chrome/content/aboutAddons.js
+++ b/mobile/android/chrome/content/aboutAddons.js
@@ -220,23 +220,24 @@ 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;
 
-    // TODO(matt): Add support for OPTIONS_TYPE_INLINE_BROWSER once bug 1302504 lands.
     let optionsURL;
-    switch (aAddon.optionsType) {
+    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 blocked = "";
     switch(aAddon.blocklistState) {
       case Ci.nsIBlocklistService.STATE_BLOCKED:
         blocked = "blocked";
         break;