1350526 - Support imageUrl in browser.notifications.create

Status: UNCONFIRMED

(WebExtensions :: Compatibility, defect, P5)

Description

[sankar88.aj]

Attachment: bug FF.zip

User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:52.0) Gecko/20100101 Firefox/52.0
Build ID: 20170316213829

Steps to reproduce:

1. Load the attached zip file as temporary addon.
2. debug the temporary addon.
3. Notification occurs every 5 seconds.




Actual results:

Notification is displayed without image.


Expected results:

Notification should be displayed with image content specified.

Comment 1

[YF (Yang)]

No "imageUrl" support on https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/notifications/create.

https://developer.chrome.com/apps/notifications#type-NotificationOptions


FYI:
"iconUrl": browser.extension.getURL("mjolnir-Spoof.jpg") is working.

Comment 2

[sankar88.aj]

I don't see any explicit word in the link says "No Support to imageURL". 
But i can see "imageURL" is listed as part of Notification option: https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/notifications/NotificationOptions

iconUrl is working fine, even without browser.extension.getURL(). I'm using it already pointing to dynamic web images. 

But the issue is imageUrl, which is not working as in chrome or not working at all.

Comment 3

[YF (Yang)]

http://stackoverflow.com/questions/42069629/include-image-in-notification

https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/notifications/TemplateType#Type

Comment 4

[:shell escalante]

looking for dupe

Comment 5

[Andy McKay]

I thought I knew of a dupe, but I can't find it. Next question would be if this is supported by the underlying API (which I think is nsIAlertsService), if not we'd need to file bugs for that.

Comment 6

[Bob Silverberg [:bsilverberg]]

Yeah, there's no dupe for this, per se. There is bug 1190681, which depends on bug 1225110, but that's more about supporting buttons than supporting a notification of type "image" which is essentially what this bug is asking for. Implementing that will also require work on nsIAlertsService, so we could open a bug for that, although I'm not sure that's even something we ever plan on supporting. The spec does say it's optional [1], so I don't suppose there's a reason to specifically not implement it, but I don't imagine the priority is very high for doing so.

[1] https://notifications.spec.whatwg.org/#image-url