552865 - Jetpack SDK should enforce packaging of all possible files into JAR

Status: RESOLVED WONTFIX

(Add-on SDK Graveyard :: General, defect)

Description

[Dietrich Ayala (:dietrich)]

Every file outside of a JAR in an extension hurts Firefox performance, by increasing the amount of filesystem IO. Jetpack should enforce performance best-practices by JARing everything by default.

More discussion in bug 533038. See also omnijar: bug 552121.

And the add-on performance project is tracking issues like this here: https://wiki.mozilla.org/Performance/Addons#Activity

Comment 1

[Atul Varma [:atul]]

This shouldn't be hard to integrate into the 'cfx xpi' command, which currently bundles everything into a non-JAR XPI.

However, some of the low-level Jetpack code uses APIs that expect some resources to be available as actual files on the filesystem, so modifying those aspects of the code will take some work.

Comment 2

[Myk Melez [:myk] [@mykmelez]]

Yup, we should definitely do this; I'm just not sure yet.  I'm setting the milestone to future and will update that once we figure out which dev cycle it should live in.

Comment 3

[Myk Melez [:myk] [@mykmelez]]

s/not sure yet/not sure when yet/

Comment 4

[Myk Melez [:myk] [@mykmelez]]

The Add-on SDK is no longer a Mozilla Labs experiment and has become a big enough project to warrant its own Bugzilla product, so the "Add-on SDK" product has been created for it, and I am moving its bugs to that product.

To filter bugmail related to this change, filter on the word "looptid".

Comment 5

[Myk Melez [:myk] [@mykmelez]]

This was pre-omnijar; doing this now would make things worse.