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 @erikvold<https://github.com/erikvold>is going to expose
> isJetpack for native jetpacks anyway, so is there point in delaying it ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mozilla/gecko-dev/pull/3/files#r9134079>
> .
>