From a3120f1082927e5d130e1db93b70ae5d27ca52a6 Mon Sep 17 00:00:00 2001 From: "mscott%netscape.com" Date: Tue, 30 Nov 1999 22:24:44 +0000 Subject: [PATCH] comment out the msgWindow variable that we were creating every time this script was loaded. This was happening every time we loaded a message. It had a side effect of causing a uri dispatching crash and it had severe performance implications. We shouldn't be creating a new msg window every time we display a message. this may have a bad side effect when we support multiple 3-pane windows because we can no longer call SetWindow. --- mailnews/mime/emitters/resources/content/attach.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/mailnews/mime/emitters/resources/content/attach.js b/mailnews/mime/emitters/resources/content/attach.js index afd9e3630281..2fbbdcfea54d 100644 --- a/mailnews/mime/emitters/resources/content/attach.js +++ b/mailnews/mime/emitters/resources/content/attach.js @@ -26,15 +26,15 @@ var msgWindowProgID = "component://netscape/messenger/msgwindow"; var messenger = Components.classes['component://netscape/messenger'].createInstance(); messenger = messenger.QueryInterface(Components.interfaces.nsIMessenger); //Create message window object -var msgWindow = Components.classes[msgWindowProgID].createInstance(); -msgWindow = msgWindow.QueryInterface(Components.interfaces.nsIMsgWindow); +//var msgWindow = Components.classes[msgWindowProgID].createInstance(); +//msgWindow = msgWindow.QueryInterface(Components.interfaces.nsIMsgWindow); function OpenAttachURL(url, displayName, messageUri) { dump("\nOpenAttachURL from XUL\n"); dump(url); dump("\n"); - messenger.SetWindow(window, msgWindow); + // messenger.SetWindow(window, msgWindow); messenger.openAttachment(url, displayName, messageUri); }