When a downloaded file marked as malware is unblocked, the red background is still shown in the Downloads Panel. This bug fundamentally existed before bug 950058 was implemented, but the latter made it visible because the new styling just checks the "verdict" attribute to show the background color without checking other attributes. At the time, to keep things simple we implemented an API where "succeeded" and "error" can be true at the same time, but it turns out it may have been worth spending a bit more time to design a better solution.
This can be tested manually at <https://testsafebrowsing.appspot.com/>.
[Tracking Requested - why for this release]: This slipped through when I reviewed bug 950058, which is targeted at Firefox 52. It's a rare case because malware downloads are a very small percentage, and the ones that are unblocked should be even smaller, but it should still be fixed for correctness.
tracking-firefox52: --- → ?
Comment on attachment 8829433 [details] Bug 1333064 - Unblocked malware downloads are styled incorrectly in the Downloads Panel. https://reviewboard.mozilla.org/r/106544/#review108302 the code makes sense, and it works (tested locally)!
Attachment #8829433 - Flags: review?(mak77) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/31f2b6fc8524 Unblocked malware downloads are styled incorrectly in the Downloads Panel. r=mak
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox54: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
2 years ago
Assignee: nobody → paolo.mozmail
status-firefox51: --- → unaffected
status-firefox52: --- → affected
status-firefox53: --- → affected
tracking-firefox53: --- → ?
Version: unspecified → 52 Branch
Tracking for 52 as caused by a recent change there.
tracking-firefox52: ? → +
Comment on attachment 8829433 [details] Bug 1333064 - Unblocked malware downloads are styled incorrectly in the Downloads Panel. Approval Request Comment [Feature/Bug causing the regression]: Bug 950058 [User impact if declined]: Incorrect styling after unblocking a download [Is this code covered by automated tests?]: No [Has the fix been verified in Nightly?]: Yes [Needs manual test from QE? If yes, steps to reproduce]: Yes, see comment 0. [List of other uplifts needed for the feature/fix]: Bug 991965 [Is the change risky?]: Low risk [Why is the change risky/not risky?]: This is a simple change that only influences a CSS attribute used for styling, after a blocked download has been unblocked. [String changes made/needed]: None
I reproduced this bug using Fx 54.0a1, build ID: 20170123080351, on Windows 10 x64. I can confirm this issue is fixed, after allowing a malware download the color changes back, I verified using Fx 54.0a1, build ID:20170130030205 on Windows 10 x64, Mac OS X 10.11 and Ubuntu 14.04 LTS. Cheers!
status-firefox54: fixed → verified
tracking-firefox52: + → ?
I think the tracking flag from comment 8 was reset accidentally.
(In reply to :Paolo Amadini from comment #11) > I think the tracking flag from comment 8 was reset accidentally. Yes, confirmed with Cristian on IRC this was an accident; resetting the flag, and tracking for 53 while I'm at it.
tracking-firefox52: ? → +
tracking-firefox53: ? → +
Comment on attachment 8829433 [details] Bug 1333064 - Unblocked malware downloads are styled incorrectly in the Downloads Panel. download panel styling fix, for aurora53 and beta52, should be in 52.0b3.
status-firefox53: affected → fixed
status-firefox52: affected → fixed
I've reproduced the initial issue described in comment 0 using old Nightly from 2017-01-23 and verified that the issues is not reproducible anymore using Firefox 52 beta 8 and latest Developer Edition 53.0a2 across platforms (macOS 10.12.3, Ubuntu 16.04 32it, Windows 10 64bit).
Status: RESOLVED → VERIFIED
status-firefox52: fixed → verified
status-firefox53: fixed → verified
You need to log in before you can comment on or make changes to this bug.