It is more standard to shut things down at xpcom-shutdown. I looked at the landing of this patch and it didn't seem like there was any particular reason to use quit-application instead.
Also, remove trailing spaces from network_geolocation.sjs. try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=ede1cf265db5
Attachment #8709642 - Flags: review?(dougt)
what about all of the other places where we call quit-application?
(In reply to Doug Turner (:dougt) from comment #2) > what about all of the other places where we call quit-application? Yeah, I'm not really sure. I think it is just a good idea to shut down anything that uses CC in xpcom-shutdown, so we can sure it will get cleaned up. I'm not sure when quit-application actually happens.
You need to log in before you can comment on or make changes to this bug.