Bug 911098 - Implement addon dubugger UI

[Gozala]

Note: This is just for a review, not for actually merging in.

[Gozala]

@Mossop I believe @erikvold is going to expose isJetpack for native jetpacks anyway, so is there point in delaying it ?

[Mossop]

I'd like Blair to review any API addition here.

On Thu, Jan 23, 2014 at 3:01 PM, Irakli Gozalishvili <
notifications@github.com> wrote:

In toolkit/mozapps/extensions/XPIProvider.jsm:

@@ -6594,6 +6594,22 @@ function AddonWrapper(aAddon) {
return ops;
});

• this.defineGetter("isBootstrapped", function AddonWrapper_isBootstrapped() {
• return this.operationsRequiringRestart === AddonManager.OP_NEEDS_RESTART_NONE;
• });

• this.defineGetter("isJetpack", function AddonWrapper_isJetpack() {
• // Assume that if there's a harness-options.json file in restartless add-on
• // it's a jetpack.
• return this.isBootstrapped && this.hasResource("harness-options.json");
• });

@Mossop https://github.com/Mossop I believe @erikvoldhttps://github.com/erikvoldis going to expose
isJetpack for native jetpacks anyway, so is there point in delaying it ?

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/3/files#r9134079
.

[erikvold]

I believe @erikvold is going to expose isJetpack for native jetpacks anyway, so is there point in delaying it ?

@Mossop @Gozala that is part of the plan

[edmorley]

(Mass-PR-close)
This PR is against gecko-dev, which is read only. Did you mean to submit it against another repo?
Changes to repos covered by gecko-dev must be made to the hg.mozilla.org repo, which is then mirrored here. Please see:
https://developer.mozilla.org/en-US/docs/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F