Add a doClose method to do some possible cleanup. Really just moving some old code out of the xfe close to where it should be used.

cmd/xfe/src/DownloadFrame.cpp

@@ -451,7 +451,25 @@ void
 XFE_DownloadFrame::configureLogo()
 {
 }
+//////////////////////////////////////////////////////////////////////////
+void
+XFE_DownloadFrame::doClose()
+{
+#ifdef notyet
+    /* We might be in an extend text selection on the text widgets.
+       If we destroy this now, we will dump core. So before destroying
+       we will disable extend of the selection. */
+    XtCallActionProc(m_url_value, "extend-end",
+                     NULL, NULL, 0);
+    /*XtCallActionProc(CONTEXT_DATA (context)->url_text, "extend-end",
+                     NULL, NULL, 0);*/
 
+    /* If we were downloading, cleanup the file we downloaded to. */
+    char *filename = fe_GetTextField(m_url_value);
+    XP_FileRemove(filename, xpTemporary);
+#endif
+    XFE_Frame::doClose();
+}
 //////////////////////////////////////////////////////////////////////////
 //
 // XFE_DownloadView
@@ -476,9 +494,6 @@ XFE_DownloadView::XFE_DownloadView(XFE_Component *toplevel, Widget parent,
 							XmNbottomAttachment,	XmATTACH_NONE,
 							NULL);
 
-	CONTEXT_DATA(m_contextData)->top_area = form;
-	
-
 	m_logo = new XFE_Logo((XFE_Frame *) toplevel,form,"logo");
 	
 	pane = XtVaCreateManagedWidget("pane",

cmd/xfe/src/DownloadFrame.h

@@ -51,6 +51,9 @@ public:
  	virtual XFE_Logo *	getLogo			();
  	virtual void		configureLogo	();
 
+    // Override XFE_Frame method
+    void doClose();
+
 private:
 	Widget m_stopButton;