Bug 1040270 - If an add-on has a ./package.json file, then call it a jetpack#44
Bug 1040270 - If an add-on has a ./package.json file, then call it a jetpack#44erikvold wants to merge 1 commit intomozilla:masterfrom erikvold:1040270
Conversation
There was a problem hiding this comment.
We know that we have a zip reader here so you can just call aZipReader.hasEntry
|
I don't think the XPIUtils.jsm module is necessary here. Nor is breaking out the Preferences.jsm module, XPIProvider.jsm already has a Prefs helper object, Prefs.getBoolPref should do all you need. |
|
This code also looks incomplete, it switches to reading package.json, but then still attempts to parse with loadManifestFromRDF. |
|
(Mass-PR-close) |
@Mossop, the XPIProvider.jsm file is already more than 7500 lines long. Wouldn't it be easier to manage if it were in separate files and more modular? It feels that way to me, it also makes the function easier to unit test since I do not have to export it from |
|
Replies to this PR spams people watching this repo (I'm watching this repo to point new contributors who mistakenly open PRs not realising they are not accepted) - please can review requests like this take place against personal repos? :-) |
Adding Connection lines
…emulator Bug 1104012 - add cppunit for b2g emulator r=lightsofapollo
No description provided.