1281152 - [linux] Washed out download protection icons

Status: NEW

(Firefox :: Downloads Panel, defect, P3)

Description

[Bogdan Maris, Desktop QA]

Attachment: Screenshot showing the issue

[Affected versions]:
- Firefox 48 beta 2
- latest Developer Edition 49.0a2

[Unaffected versions]:
- latest Nightly 50.0a1

[Affected platforms]:
- Ubuntu 16.04 32-bit

[Steps to reproduce]:
1. Start Firefox with clean profile
2. Visit http://testsafebrowsing.appspot.com/
3. Click on item #2 and #5 and save/open files
4. Click Download icon

[Expected result]:
- Red and blue icons are properly displayed

[Actual result]:
- Red and blue icons are washed out

[Regression range]:
- Don't quite know if this is a regression but first Aurora build affected is 48.0a2. Will investigate further.

[Additional notes]:
- Screencast showing the issue attached.

Comment 1

[Bogdan Maris, Desktop QA]

I was able to track down a regression here and it lead to this:

Last good revision: fc15477ce628599519cb0055f52cc195d640dc94
First bad revision: 37f04460ddb76d6ef4e7c32a8a6b2fbc44cb8776
Pushlog:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=fc15477ce628599519cb0055f52cc195d640dc94&tochange=37f04460ddb76d6ef4e7c32a8a6b2fbc44cb8776

a36e52a4543d	Paolo Amadini — Bug 1265358 - Enable blocking of uncommon downloads. r=gcp
f36212ee8d4a	Paolo Amadini — Bug 1265359 - Enable blocking of potentially unwanted downloads. r=gcp

Comment 2

[Bogdan Maris, Desktop QA]

So this is not a regression, it was introduced while enabling blocking unwanted and uncommon downloads.

Comment 3

[Panos Astithas (he/him) [:past] (please ni?)]

Bug 1265341 made the icons look nice. Paolo, do you think we should uplift it or live with the lo-dpi icons for 48 & 49?

Comment 4

[:Paolo Amadini]

This affects a very small number of users, those on Linux encountering a blocked download. Given that the styling changes from bug 1265341 also have some risk of visual regressions on less common platforms, I'm tempted to say that we shouldn't uplift to Beta, although we could consider Developer Edition. Jonathan, what do you think?

Comment 5

[Jonathan Kingston [:jkt] he/him]

I'm not entirely sure on the risks to other platforms, but certainly there is a possibility. This is how the icons looked to me when I tested in Ubuntu and my patch changes this. 
I suspect dev edition uplift is potentially a safer bet though.
Maybe checking all platforms with mozscreenshots is worth doing first though?

Comment 6

[Panos Astithas (he/him) [:past] (please ni?)]

Let's decide in the team meeting.

Comment 7

[Erin Lancaster [:elan]]

Not for beta. For Aurora DevEdition. Thank you.

Comment 8

[Sylvestre Ledru [:Sylvestre]]

Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information