1155956 - Remove unused private browsing mask icons

Status: RESOLVED FIXED

(Firefox :: Theme, defect)

Description

[Dão Gottwald [:dao]]

These icons are unused and should be removed:

/browser/themes/linux/Privacy-32.png
/browser/themes/linux/Privacy-48.png
/browser/themes/linux/Privacy-64.png
/browser/themes/osx/Privacy-32.png
/browser/themes/osx/Privacy-48.png
/browser/themes/osx/Privacy-64.png
/browser/themes/windows/Privacy-32-XP.png
/browser/themes/windows/Privacy-32.png
/browser/themes/windows/Privacy-48-aero.png
/browser/themes/windows/Privacy-48.png
/browser/themes/windows/Privacy-64-aero.png
/browser/themes/windows/Privacy-64.png

Some of these images are referenced in /browser/themes/windows/jar.mn, those references need to be removed too.

Comment 1

[James Lai]

Hi Dao,

I am interested in and going to work on this bug.

Comment 2

[James Lai]

Attachment: Bug 1155956 patch 01

Comment 3

[James Lai]

Hi Dao,

Just attached the patch. Appreciate if you may help check?

Comment 4

[Dão Gottwald [:dao]]

Comment on attachment 8594440 [details] [diff] [review]
Bug 1155956 patch 01

Nice! Looks perfect.

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/284297151baa

Comment 6

[Dão Gottwald [:dao]]

If you're interested in fixing another very similar bug, there's bug 1150800. Seems like the person who started working on this got stuck and gave up.

Comment 7

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/284297151baa

Comment 8

[James Lai]

(In reply to Dão Gottwald [:dao] from comment #6)
> If you're interested in fixing another very similar bug, there's bug
> 1150800. Seems like the person who started working on this got stuck and
> gave up.

Thanks and yes I would love to work on other bug fixes.