Closed Bug 1112788 Opened 11 years ago Closed 9 years ago

Add "e10s" as an artificial Firefox version in order to use it for compatibility overrides

Categories

(addons.mozilla.org Graveyard :: Compatibility Tools, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: jorgev, Assigned: jorgev)

References

Details

We want to flag add-ons as incompatible with e10s, and it seems like the easiest solution is to introduce "e10s" as a valid Firefox version so it can be used in the compatibility override tool. This will also require some work on the Firefox side in order to support this version. I'll start by staging this change to make sure there isn't unexpected breakage.
Okay, so I added "e10s" as a valid version number on staging (addons-dev.allizom.org). Then I modified a compatibility override so the Firefox version range is e10s - e10s. The add-on is gTranslator: https://addons-dev.allizom.org/addon/gtranslator/. I noticed a couple of things: * The compatibility override tool takes free-form version numbers, so we can probably do this without adding e10s as a valid version number. The advantage of not adding it is that it won't show up on version number dropdowns across the site, which could confuse some devs. * Our version comparison algorithm places e10s below any numerical versions (so e10s < 0). This is why I set the Firefox version range to e10s - e10s for the gTranslator test. If set to e10s - *, then it would effectively flag the add-on as incompatible with all Firefox versions. So, on the Firefox side we would need to change the Add-on Manager code that reads the overrides, in order to support the special e10s - e10s range and apply it when e10s is enabled in Firefox.
Adding this AMO bug to the list of e10s bugs we're tracking.
tracking-e10s: --- → +
Product: addons.mozilla.org → addons.mozilla.org Graveyard
As per Chris Peterson [:cpeterson] feedback, setting status as Resolved:Won't Fix.
Status: NEW → RESOLVED
Closed: 9 years ago
tracking-e10s: + → ---
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.