Closed
Bug 288770
Opened 20 years ago
Closed 20 years ago
browser.xml removeObserver call still sometimes throws exception
Categories
(Firefox :: Tabbed Browser, defect, P1)
Firefox
Tabbed Browser
Tracking
()
RESOLVED
FIXED
Firefox1.5
People
(Reporter: dbaron, Assigned: dbaron)
References
Details
(Keywords: memory-leak, Whiteboard: [patch])
Attachments
(1 file)
|
1.40 KB,
patch
|
mconnor
:
review+
|
Details | Diff | Splinter Review |
Even after bug 283088 and bug 283116 the removeObserver call in browser.xml's destroy method still throws an exception sometimes. With a try-catch around it, nothing after it throws an exception, so I think the solution is to put a try-catch around just that one call so that the code after it still runs.
| Assignee | ||
Comment 1•20 years ago
|
||
Attachment #179398 -
Flags: review?(mconnor)
| Assignee | ||
Updated•20 years ago
|
Status: NEW → ASSIGNED
Keywords: mlk
Priority: -- → P1
Whiteboard: [patch]
Target Milestone: --- → Firefox1.1
| Assignee | ||
Comment 2•20 years ago
|
||
Note that I haven't figured out under what conditions it throws an exception, but I log all of the stdout of my browser sessions, and it's not too common, but it does happen occasionally.
Updated•20 years ago
|
Attachment #179398 -
Flags: review?(mconnor) → review+
| Assignee | ||
Comment 3•20 years ago
|
||
Fix checked in to trunk, 2005-04-04 21:41 -0700.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Comment 4•20 years ago
|
||
*** Bug 291054 has been marked as a duplicate of this bug. ***
Comment 5•20 years ago
|
||
*** Bug 278864 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•