fuelApplication.js Window._shutdown() removeEventListener() call is wrong

VERIFIED FIXED in Firefox 16

Status

()

Firefox
General
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: sgautherie, Assigned: Justin Lebar (not reading bugmail))

Tracking

Trunk
Firefox 16
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(firefox13 wontfix, firefox14 wontfix, firefox15 wontfix)

Details

(Whiteboard: [fixed by bug 750454], URL)

(Reporter)

Description

5 years ago
(In reply to Serge Gautherie (:sgautherie) from bug 758102 comment #2)
> {
> 97     this._tabbrowser.tabContainer.addEventListener(aType,
> 123       this._tabbrowser.removeEventListener(type, this._cleanup[type],
> true);
> }
> 
> See
> http://hg.mozilla.org/mozilla-central/rev/8ca8630b0c88
> 
> Is FF missing a '.tabContainer'?


(In reply to neil@parkwaycc.co.uk from bug 758102 comment #6)
> Sadly
> by the time the shutdown method runs, the binding no longer exists. I think
> the workaround might be to store the tabContainer in a property too, but
> :dao might have other ideas, after all he's going to have to fix the same
> bug ;-)
bug 750454 will fix this
Depends on: 750454

Updated

5 years ago
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: fixed by bug 750454
Target Milestone: --- → Firefox 16
(Reporter)

Comment 2

5 years ago
(In reply to Dão Gottwald [:dao] from comment #1)
> bug 750454 will fix this

It did in its
http://hg.mozilla.org/mozilla-central/rev/0a53a1ca7be0

V.Fixed
Assignee: nobody → justin.lebar+bug
Status: RESOLVED → VERIFIED
Flags: in-testsuite-
Whiteboard: fixed by bug 750454 → [fixed by bug 750454]
(Reporter)

Comment 3

5 years ago
Branches would still like a fix for this very bug.
status-firefox13: --- → wontfix
status-firefox14: --- → affected
status-firefox15: --- → affected
(Assignee)

Comment 4

5 years ago
(In reply to Serge Gautherie (:sgautherie) from comment #3)
> Branches would still like a fix for this very bug.

The fix is unfortunately kind of large.  And since this code isn't well-tested, I wanted this to ride the trains.  Perhaps there's a way to write a smaller fix.

That said, I'm really surprised we haven't seen any regressions off the FUEL bug.  I'm worried that all of a sudden people will realize that that I broke their favorite add-on.
(Reporter)

Comment 5

5 years ago
(In reply to Justin Lebar [:jlebar] from comment #4)
> Perhaps there's a way to write a smaller fix.

Per comment 0, I would expect a "one-liner" fix:
something similar to bug 758102 patch Dv1-SM211.
(Assignee)

Comment 6

5 years ago
> Per comment 0, I would expect a "one-liner" fix

Oh, I'm sorry; I thought this was a different bug.

If this is shutdown-only code, why does it matter?

Updated

5 years ago
status-firefox14: affected → wontfix
status-firefox15: affected → wontfix
You need to log in before you can comment on or make changes to this bug.