Closed Bug 1528186 Opened 9 months ago Closed 9 months ago

BuiltinLocation extensions dont have a scope

Categories

(WebExtensions :: General, enhancement)

enhancement
Not set

Tracking

(firefox67 fixed)

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: daleharvey, Assigned: aswan)

References

Details

Attachments

(1 file)

Andrew is this something you can fix or can you point me in the right direction? it doesnt look like the constructor gets called with a scope here like the other constructors

Cheers

Flags: needinfo?(aswan)

Yeah, we should pass AddonManager.SCOPE_APPLICATION there. This code was originally pasted from TemporaryInstallLocation which also doesn't include a scope...

Flags: needinfo?(aswan)

Does this need anything else to land?

Assignee: nobody → aswan
Status: NEW → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67

Does this fix require QA validation? if so please provide some info on how to correctly test it. Thanks!

Flags: needinfo?(aswan)

thanks, but no manual testing is needed.

Flags: needinfo?(aswan) → qe-verify-
You need to log in before you can comment on or make changes to this bug.