Created attachment 539742 [details] [diff] [review] patch In order to release the browser when something is erroneously holding a reference to a removed tab, we should cut off the link between the two. mochitest-browser-chrome passed on try with this patch.
Attachment #539742 - Flags: review?(gavin.sharp)
Comment on attachment 539742 [details] [diff] [review] patch Why null out linkedPanel? It's just an ID. It's hard to know whether anyone would be depending on tabs keeping their browsers alive, but I guess there isn't much to do aside from trying it and seeing what breaks.
Attachment #539742 - Flags: review?(gavin.sharp) → review+
Created attachment 540227 [details] [diff] [review] patch v2 removed the linkedPanel part, added a test
Attachment #539742 - Attachment is obsolete: true
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 7
Setting resolution to Verified Fixed on Mozilla/5.0 (Windows NT 6.1; rv:7.0) Gecko/20100101 Firefox/7.0
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.