101182 - Downloads continue in the background when d/l window is closed

Status: VERIFIED FIXED

(Core :: Networking, defect)

Description

[Dennis Kehrig]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:0.9.4) Gecko/20010913
BuildID:    2001091303

Start a download, let it load a while, then cancel it. Close all active network
applications but leave a Mozilla window open. Then there is still traffic. The
file gets downloaded to c:\windows\temp. When going offline, the data is stored
under the file name originally selected (though it's incomplete). Obviously
cancelling a download should stop the download.

Reproducible: Always
Steps to Reproduce:
See description

Actual Results:  See description

Expected Results:  The download should have stopped directly after cancelling
it. Perhaps the user should be asked if the (incomplete) data should be deleted
or left on the hard disk (or in cache only, for resuming the download later).

Comment 1

[Matthias Versen [:Matti]]

wfm with win2k build 20010922..

I tried it 3x und the download stops always after I hit "cancel".

Comment 2

[Dennis Kehrig]

Update: cancel != closing the window. Hit Alt+F4 or click the "X" button to
close the download window instead of clicking "cancel". Shame on me. Cancel
works for me, too.

Comment 3

[Boris Zbarsky [:bzbarsky]]

setting status to new.  See discussion in bug 96891 on similar issues with a
different dialog.

Comment 4

[Bradley Baetz (:bbaetz)]

download dialog -> law

Comment 5

[Bill Law]

*** Bug 85603 has been marked as a duplicate of this bug. ***

Comment 6

[Bill Law]

->0.9.7

Need to make onunload sort of like oncancel, but oncancel has risk of deleting
the temporary file.  So, moving out till I can ensure there's no bad side-effects.

Comment 7

[R.K.Aa.]

*** Bug 111894 has been marked as a duplicate of this bug. ***

Comment 8

[Bill Law]

Attachment: Patch to do onCancel logic from onclose handler

Comment 9

[Brian Ryner (not reading)]

Comment on attachment 60702 [details] [diff] [review]
Patch to do onCancel logic from onclose handler

r=bryner

Comment 10

[David Hyatt]

sr=hyatt

Comment 11

[Bill Law]

fixed

Comment 12

[Tom Everingham]

qa -> me since I'm familiar with this one

Comment 13

[Tom Everingham]

verified:

Win98 2001-12-07-10-tr
Linux 2001-12-07-08-tr