Closed Bug 1117206 Opened 9 years ago Closed 6 years ago

[e10s] Down Them All addon throws exception

Categories

(Firefox :: Extension Compatibility, defect)

x86_64
Windows 8
defect
Not set
normal

Tracking

()

RESOLVED WONTFIX
Tracking Status
e10s + ---

People

(Reporter: ally, Assigned: nmaier)

References

Details

Telemetry reports that this addon regularly throws an exception in multiprocess firefox on line 90 in preferences.jsm

84 /**
85  * Gets a preference branch 
86  * @param branch (string) Branch to get
87  * @return (nsIPrefBranch) Requested branch 
88  */
89 function getBranch(branch) {
90 	return prefs.getBranch(branch);
tracking-e10s: --- → ?
Summary: [e10s] Down the Mall addon throws exception → [e10s] Down Them All addon throws exception
This is bug 1083561, caused by compartment-per-add-on. The solution here is to make the add-on QI the right interface.
Blocks: 1030420
Should we get in touch with the author through AMO?
Flags: needinfo?(jorge)
Added developers to CC.
Flags: needinfo?(jorge)
Good thing we use that legacy piece of ... function only once and in the shutdown observer code of all places. And it will only affect people who still use legacy "clear history" stuff. So the impact is very minor, expect for spamming telemetry apparently.

Anyway, I'll fix it when I get some time.
Assignee: nobody → maierman
This looks like the proper component for this bug. Please change if not.
Component: Untriaged → Extension Compatibility
Mass-closing old Extension Compatibility bugs that relate to legacy add-ons or NPAPI plug-ins. If you think this bug is still valid, please reopen or comment.

Sorry for the bug spam, and happy Friday!
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.