Some code was changed in ExtensionAPI.jsm which removed onStartup() and requires constructor() to be used in its place, but ext-geolocation.js still uses onStartup()  so that should be fixed.  http://searchfox.org/mozilla-central/source/toolkit/components/extensions/ext-geolocation.js#12
No, constructor() should not be used in its place. The onStartup() method is still supposed to work, so if it doesn't in this case, that needs to be fixed.
It turns out this isn't a problem at all. We thought it was because I was working on ext-url-overrides.js and onStartup() was not being fired for that API, but when I tested ext-geolocation.js it works fine.
Status: NEW → RESOLVED
Last Resolved: 8 months ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.