All users were logged out of Bugzilla on October 13th, 2018

Tabview can leak if it is open when a tab is restored in e10s mode

RESOLVED FIXED in Firefox 36

Status

RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: mossop, Assigned: mossop)

Tracking

unspecified
Firefox 36
Bug Flags:
firefox-backlog +
qe-verify -

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

4 years ago
It uses a DOM event listener to wait for a load event that never comes in e10s mode so the listener leaks.
Flags: qe-verify-
Flags: firefox-backlog+
(Assignee)

Comment 1

4 years ago
Created attachment 8512935 [details] [diff] [review]
patch
Attachment #8512935 - Flags: review?(mconley)
(Assignee)

Comment 2

4 years ago
Created attachment 8512938 [details] [diff] [review]
patch
Attachment #8512935 - Attachment is obsolete: true
Attachment #8512935 - Flags: review?(mconley)
Attachment #8512938 - Flags: review?(mconley)
Comment on attachment 8512938 [details] [diff] [review]
patch

Review of attachment 8512938 [details] [diff] [review]:
-----------------------------------------------------------------

This looks fine - but I'm wondering if this particular pattern is more common than we think. A quick dxr for addEventListener("load", has quite a few non-mochitest results... we might want to audit those at some point to ensure e10s doesn't cause similar leaks elsewhere.
Attachment #8512938 - Flags: review?(mconley) → review+
tracking-e10s: ? → m3+
https://hg.mozilla.org/mozilla-central/rev/5a9f63201e5c
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.