Blocked downloads don't keep data on Unices other than Linux and Darwin

RESOLVED FIXED in Firefox 38

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jbeich, Assigned: jbeich)

Tracking

39 Branch
Firefox 40
All
FreeBSD
Points:
---

Firefox Tracking Flags

(firefox38 fixed, firefox39 fixed, firefox40 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

4 years ago
User Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:39.0) Gecko/20100101 Firefox/39.0
Build ID: 20150325181557

Steps to reproduce:

1. Go to http://www.eicar.org/85-0-Download.html
2. Download eicar.com 68 Bytes


Actual results:

File is not saved. Download Panel says "Blocked: May contain a virus or spyware".


Expected results:

Still blocked but context menu has Unblock item. Same behavior as custom/non-official Linux builds or attachment 8572227 [details] [diff] [review].
(Assignee)

Updated

4 years ago
Blocks: 1137909

Updated

4 years ago
Duplicate of this bug: 1147285
(Assignee)

Updated

4 years ago
Attachment #8583890 - Flags: review?(jaws)
(Assignee)

Updated

4 years ago
Component: Untriaged → Downloads Panel
Hardware: x86_64 → All
(Assignee)

Updated

4 years ago
status-firefox38: --- → affected
status-firefox39: --- → affected
Comment on attachment 8583890 [details] [diff] [review]
Drop redundant check

Review of attachment 8583890 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/components/jsdownloads/src/DownloadIntegration.jsm
@@ +186,5 @@
>        return this.shouldKeepBlockedDataInTest;
>      }
>  
>      const FIREFOX_ID = "{ec8030f7-c20a-464f-9b0e-13a3a9e97384}";
>      let os = Services.appinfo.OS;

`os` is unused and can be removed.
Attachment #8583890 - Flags: review?(jaws) → review+

Comment 4

4 years ago
(In reply to Jan Beich from comment #2)
> I'm not sure how bug 1137909 comment 13 led to adding OS checks but IDs are
> different between apps or addons would break.
> 
> https://dxr.mozilla.org/comm-central/search?q=MOZ_APP_ID%3D&case=true

Thanks for the link, I missed that in the review as I thought the IDs were the same.
(Assignee)

Comment 5

4 years ago
Removed |let os ...| and carrying over r+ per comment 3.
Attachment #8583890 - Attachment is obsolete: true
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/fx-team/rev/be975e204f2a
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/be975e204f2a
Status: NEW → RESOLVED
Last Resolved: 4 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 40
(Assignee)

Comment 8

4 years ago
Comment on attachment 8586821 [details] [diff] [review]
Drop redundant check

Approval Request Comment
[Feature/regressing bug #]: regression between patch revisions of the new feature in bug 1137909: attachment 8572227 [details] [diff] [review] vs. attachment 8572886 [details] [diff] [review]
[User impact if declined]: less feature parity with Linux <- blocked downloads cannot be unblocked on Tier3 platforms (Solaris and various BSDs)
[Describe test coverage new/current, TreeHerder]: m-c, m-i, Try in comment 2
[Risks and why]: Low, at worst (typos?) either errors in console or the feature enabled on non-Desktop platforms (Android, B2G, etc)
[String/UUID change made/needed]: None
Attachment #8586821 - Flags: approval-mozilla-beta?
Attachment #8586821 - Flags: approval-mozilla-aurora?
Comment on attachment 8586821 [details] [diff] [review]
Drop redundant check

We are about other platforms too, taking it. Should be in 38 beta 3.
Attachment #8586821 - Flags: approval-mozilla-beta?
Attachment #8586821 - Flags: approval-mozilla-beta+
Attachment #8586821 - Flags: approval-mozilla-aurora?
Attachment #8586821 - Flags: approval-mozilla-aurora+
(Assignee)

Updated

4 years ago
Assignee: nobody → jbeich
You need to log in before you can comment on or make changes to this bug.