User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:56.0) Gecko/20100101 Firefox/56.0 Build ID: 20170628030206 Steps to reproduce: It says , notifications.create can fail on some systems, so i think it should return a Promise.reject here (or something else) instead of just returning 'Promise.resolve(notificationId);' .  http://searchfox.org/mozilla-central/source/toolkit/components/extensions/ext-notifications.js#37  http://searchfox.org/mozilla-central/source/toolkit/components/extensions/ext-notifications.js#116 Because, otherwise how can we detect if notifications.create succeed or not? If notifications.create failed for some reason we then can create our own notification and display it to the user. Because my addon relies on the notification.
This seems like a valid suggestion. Kris, what do you think?
I suppose one problem with doing this would be if Chrome does not do this, in which case we might break some extensions by throwing errors when they are not expected. Redirecting my needinfo to zombie for his ideas.
Flags: needinfo?(kmaglione+bmo) → needinfo?(tomica)
I don't think this is actually a real issue, that comment probably refers to non-supported platforms like Thunderbird or something. I guess rejecting in that case wouldn't hurt, but probably P5.
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.