Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Bug 674492 - Simplify jetpack module loader #220

Closed
wants to merge 44 commits into
from

Conversation

Projects
None yet
1 participant
Member

Gozala commented Jul 29, 2011

Ported special behavior of self module from securable-module. Now I'm able to load all the high level modules with this loader and execute all the samples from documentation (If I modify requires by appending 'addon-kit/' to them :)

Next step is make test runner compatible, but I'm afraid it will be bit complicated as cfx does non regular things on cfx test.

Member

Gozala commented Jul 29, 2011

Here is my personal task list for this work
https://workflowy.com/shared/a6adb4da-abdb-96d3-6592-b4b447c9f95e/

@Gozala Gozala closed this Aug 19, 2011

@Gozala Gozala reopened this Aug 19, 2011

Member

Gozala commented Aug 19, 2011

At this stage pull request contains stripped down version of module loader is able to load all modules as long as they are in manifest or specified with absolute or relative IDs. In addition there is a glue layer that starts remote / local browser depending weather --e10s option is passed to cfx. Glue layer allows e10s-ifying APIs so that they can used both with or without e10s, also non e10s-ified modules will work as expected if --e10s property is not specified.

Actual example of e10s-ified module can be found under different pull request that branched out from this one:
#226

Member

Gozala commented Oct 19, 2011

Closing in favor of #247

@Gozala Gozala closed this Oct 19, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment