From 1477565f496ce42595981ace90e564b5013fa6cc Mon Sep 17 00:00:00 2001 From: "martijn.martijn%gmail.com" Date: Wed, 6 Sep 2006 19:33:28 +0000 Subject: [PATCH] Bug 344872 - Mail window is leaking folderObserver on shutdown, patch by Adam Guthrie, r=bienvenu --- mail/base/content/msgMail3PaneWindow.js | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/mail/base/content/msgMail3PaneWindow.js b/mail/base/content/msgMail3PaneWindow.js index 55e1c4c1d097..76a1ccc70ca9 100644 --- a/mail/base/content/msgMail3PaneWindow.js +++ b/mail/base/content/msgMail3PaneWindow.js @@ -899,6 +899,8 @@ function OnUnloadMessenger() // FIX ME - later we will be able to use onload from the overlay OnUnloadMsgHeaderPane(); + OnUnloadFolderPane(); + OnMailWindowUnload(); } @@ -1178,6 +1180,12 @@ function OnLoadFolderPane() folderTree.addEventListener("mousedown",TreeOnMouseDown,true); } +function OnUnloadFolderPane() +{ + var folderTreeBuilder = GetFolderTree().builder.QueryInterface(Components.interfaces.nsIXULTreeBuilder); + folderTreeBuilder.removeObserver(folderObserver); +} + // builds prior to 12-08-2001 did not have the labels column // in the thread pane. so if a user ran an old build, and then // upgraded, they get the new column, and this causes problems.