Closed Bug 1117206 Opened 10 years ago Closed 7 years ago

[e10s] Down Them All addon throws exception

Categories

(Firefox :: Extension Compatibility, defect)

Product:
Firefox
Component:
Extension Compatibility
Platform:
x86_64
Windows 8
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX
Tracking Flags:
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
tracking-e10s: ? → +
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: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.