Don't set the class corresponding to the notification ID on anchor icons

NEW
Unassigned

Status

()

P4
normal
2 years ago
2 years ago

People

(Reporter: Paolo, Unassigned)

Tracking

({addon-compat})

Trunk
addon-compat
Points:
---
Bug Flags:
qe-verify ?

Firefox Tracking Flags

(firefox49 affected)

Details

(Whiteboard: [fxprivacy])

(Reporter)

Description

2 years ago
The logic in PopupNotifications._updateAnchorIcons for setting the class corresponding to the notification ID on anchor icons might not be necessary anymore, in particular after bug 1147981.

Updated

2 years ago
Flags: qe-verify?
Whiteboard: [fxprivacy][triage] → [fxprivacy]
(Reporter)

Comment 1

2 years ago
It turns out that these notification classes are also used by the Hello chat window.

(In reply to Mike de Boer [:mikedeboer] from bug 1147981 comment #39)
> But it's more interesting and perhaps more feasible in the shorter term, to
> get rid of the combined notification anchor first. So separate audio/ video
> and screen sharing notification anchors and forget about all the other
> types, because Hello never uses them. In that case we can clean up
> PopupNotifications.jsm a bit.
You need to log in before you can comment on or make changes to this bug.