Import PopupNotifications.jsm lazily into PeerConnection.js

RESOLVED FIXED in mozilla31

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: njn, Assigned: njn)

Tracking

(Blocks 1 bug)

unspecified
mozilla31
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

PopupNotification.jsm can be imported lazily, AFAICT.
Blocks: 986323
The identifier |PopupNotifications| doesn't actually appear anywhere in
PeerConnection.js, so it might instead be possible to just remove the import.
Attachment #8407281 - Flags: review?(jib)
BTW, I confirmed that this prevents the creation of a PopupNotifications.jsm compartment at start-up.
(In reply to Nicholas Nethercote [:njn] from comment #1)
> The identifier |PopupNotifications| doesn't actually appear anywhere in
> PeerConnection.js, so it might instead be possible to just remove the import.

Martin, this looks left over from the UI that got pulled in Bug 884573 comment 88 - OK to remove?
Flags: needinfo?(martin.thomson)
(In reply to Jan-Ivar Bruaroey [:jib] from comment #3)
> (In reply to Nicholas Nethercote [:njn] from comment #1)
> > The identifier |PopupNotifications| doesn't actually appear anywhere in
> > PeerConnection.js, so it might instead be possible to just remove the import.
> 
> Martin, this looks left over from the UI that got pulled in Bug 884573
> comment 88 - OK to remove?

Absolutely!  Nuke it.
Flags: needinfo?(martin.thomson)
Comment on attachment 8407281 [details] [diff] [review]
Import PopupNotifications.jsm lazily into PeerConnection.js

Nicholas do you want to do the honors?
Attachment #8407281 - Flags: review?(jib)
Attachment #8407281 - Attachment is obsolete: true
Attachment #8407405 - Flags: review?(jib) → review+
https://hg.mozilla.org/mozilla-central/rev/70bb7587900c
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
You need to log in before you can comment on or make changes to this bug.