window.Notification() throws exception NS_ERROR_FACTORY_NOT_REGISTERED

RESOLVED FIXED in seamonkey2.35

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: philip.chee, Assigned: philip.chee)

Tracking

Trunk
seamonkey2.35

SeaMonkey Tracking Flags

(seamonkey2.35 fixed)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
> binki: window.Notification() throws exception NS_ERROR_FACTORY_NOT_REGISTERED
> binki: The desktop notification displays, but my JavaScript code’s execution gets halted by that exception
> NeilAway:	binki: ok, we seem to be missing NotificationStorage.js

We'll need the packaging bits from Bug 899574 (Notification API follow-up: provide a way to get current Notification objects)
(Assignee)

Comment 1

4 years ago
Created attachment 8560875 [details] [diff] [review]
Patch v1.0 Package fix.

>        case "final-ui-startup":
......
>          LoginManagerParent.init();
>          Components.utils.import("resource://gre/modules/Webapps.jsm");
> +        Components.utils.import("resource://gre/modules/NotificationDB.jsm");
"final-ui-startup" seems a convenient place to put this but I could be wrong.
Attachment #8560875 - Flags: review?(neil)
(Assignee)

Comment 2

4 years ago
Here is a handy webpage to test HTML5 notifications:
http://www.bennish.net/web-notifications.html
Attachment #8560875 - Flags: review?(neil) → review+
(Assignee)

Comment 3

4 years ago
Pushed to comm-central: http://hg.mozilla.org/comm-central/rev/ba1aee061155
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-seamonkey2.35: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.35
You need to log in before you can comment on or make changes to this bug.