Closed
Bug 1130502
Opened 10 years ago
Closed 10 years ago
window.Notification() throws exception NS_ERROR_FACTORY_NOT_REGISTERED
Categories
(SeaMonkey :: General, defect)
SeaMonkey
General
Tracking
(seamonkey2.35 fixed)
RESOLVED
FIXED
seamonkey2.35
Tracking | Status | |
---|---|---|
seamonkey2.35 | --- | fixed |
People
(Reporter: philip.chee, Assigned: philip.chee)
References
()
Details
Attachments
(1 file)
2.94 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
> 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•10 years ago
|
||
> 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•10 years ago
|
||
Here is a handy webpage to test HTML5 notifications:
http://www.bennish.net/web-notifications.html
Updated•10 years ago
|
Attachment #8560875 -
Flags: review?(neil) → review+
Assignee | ||
Comment 3•10 years ago
|
||
Pushed to comm-central: http://hg.mozilla.org/comm-central/rev/ba1aee061155
Status: ASSIGNED → RESOLVED
Closed: 10 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.
Description
•