ext-geolocation.js needs to use constructor() rather than onStartup()

RESOLVED INVALID

Status

()

Toolkit
WebExtensions: General
P1
normal
RESOLVED INVALID
8 months ago
8 months ago

People

(Reporter: bsilverberg, Assigned: bsilverberg)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: triaged)

(Assignee)

Description

8 months ago
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() [1] so that should be fixed.

[1] 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.
(Assignee)

Comment 2

8 months ago
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.