Canceling a scanning download should make it done

VERIFIED WONTFIX

Status

()

Toolkit
Downloads API
VERIFIED WONTFIX
10 years ago
10 years ago

People

(Reporter: Mardak, Assigned: Mardak)

Tracking

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Instead of making a scanning download canceled when canceling, we should just let it finish as if there was no scan.

Bug 407655 introduces this into the UI, but there's potentially other calls to CancelDownload.
(Assignee)

Updated

10 years ago
Blocks: 407657
(Assignee)

Updated

10 years ago
Depends on: 407660
(Assignee)

Comment 1

10 years ago
Created attachment 292471 [details] [diff] [review]
v1

I actually haven't tested this.. I'll try getting a windows build.

Note for the UI, I changed it to delete only if the download is actually canceled by the back-end. Hopefully the SetState FINISHED notification gets to the UI to update "state" so that checking it works..
Assignee: nobody → edilee
Status: NEW → ASSIGNED
Attachment #292471 - Flags: review?(comrade693+bmo)
<beltzner> well, I think we've pretty much decided that cancel sucks
<beltzner> I think we need to make sure to look for bad hanging cases, fix the restart case, and then see if we need to pref this off

WONTFIX

We'll need a bug to fix the case where we crash so we can set it right, and perhaps a bug to timeout if we don't get feedback from the virus scanner (since we can't find in the api where it says it times out.  Perhaps I was seeing things when I read that?).

Please file follow-up at least for the first issue.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → WONTFIX

Updated

10 years ago
Attachment #292471 - Flags: review?(comrade693+bmo)
Status: RESOLVED → VERIFIED
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.