745857 - Need a way to set the optionsURL to a chrome window (or anything else)

Status: RESOLVED INCOMPLETE

(Add-on SDK Graveyard :: General, enhancement, P4)

Description

[Erik Vold]

This is something that any traditional add-on could do easily, and it should be easy to do with the add-on sdk as well.

Comment 1

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

What would be the use case for this?

Comment 2

[Erik Vold [:erikvold] (please needinfo? me)]

bug 559306 would resolve this.

Comment 3

[Mitar]

This ticket was marked as a duplicate of 559306, but that is not true. optionsURL is specified in install.rdf and not chrome.manifest. So there is still no way to specify optionsURL (and optionsType).

Comment 4

[Erik Vold [:erikvold] (please needinfo? me)]

If we want to do this, then I have basically done it while working on bug 915376 already, but I think we might want to use a preference instead here in order to avoid package.json bloat.

Irkali thoughts?

Comment 5

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

Erik we talked about this during weekly, so yes it looks like we should provide a way to expose `optionsURL`.

That being said I'm not sure what do you mean by using preference instead, can you elaborate please ? Some example may help as well.

Comment 6

[Erik Vold [:erikvold] (please needinfo? me)]

(In reply to Irakli Gozalishvili [:irakli] [:gozala] [@gozala] from comment #5)
> Erik we talked about this during weekly, so yes it looks like we should
> provide a way to expose `optionsURL`.
> 
> That being said I'm not sure what do you mean by using preference instead,
> can you elaborate please ? Some example may help as well.

At the moment the AOM determines the optionsURL from a value in the install.rdf only.  I'm suggesting that there could be a fallback check for a optionsURL in say `'extensions.' + id + '.optionsURL'` for example, which would be provided by the `package.json` like so:

{
  ...
  "preferences": [{
    "name": "optionsURL",
    "type": "hidden"
    "value": "chrome://.."
  }]
  ...
}

Comment 7

[Erik Vold [:erikvold] (please needinfo? me)]

I'm not working on this at the moment, and it probably doesn't make sense to work on this until bug 915376 is complete.

Comment 8

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1399562