1546445 - Add appName to WebExtension.MessageDelegate.onMessage

Status: RESOLVED FIXED

(GeckoView :: Extensions, enhancement, P1)

Description

[[ex-Mozilla] Agi Sferro | :agi]

A delegate that handles messages from multiple native apps can tell which native app identifier received the message by looking at port.name.

The same cannot be done with one-time messages as onMessage does not expose this, we should close this little API gap.

Comment 1

[Chris Peterson [:cpeterson]]

P1

Comment 2

[[ex-Mozilla] Agi Sferro | :agi]

Looks like this should've been a P1? I'm just going to do this real quick.

Comment 3

[[ex-Mozilla] Agi Sferro | :agi]

Attachment: Bug 1546445 - Add nativeApp to MessageDelegate.onMessage.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ba266451040e
Add nativeApp to MessageDelegate.onMessage. r=snorp

Comment 5

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/ba266451040e

Comment 6

[Chris Peterson [:cpeterson]]

firefox70=wontfix because we don't need to uplift this fix to GV 70 Beta.

Comment 7

[Chris Peterson [:cpeterson]]

Moving some WebExtension bugs to the GeckoView::Extensions component.