Open Bug 1275558 Opened 9 years ago Updated 2 years ago

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

Categories

(Toolkit :: General, defect, P4)

defect

Tracking

()

Tracking Status
firefox49 --- affected

People

(Reporter: Paolo, Unassigned)

References

Details

(Keywords: addon-compat, Whiteboard: [fxprivacy])

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.
Flags: qe-verify?
Whiteboard: [fxprivacy][triage] → [fxprivacy]
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.
Severity: normal → S3

Changing qe-verify? to qe-verify+.

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