1339420 - Remove unused chrome://branding/content/icon128.png file

Status: RESOLVED WONTFIX

(Firefox :: General, defect)

Description

[Florian Quèze [:florian]]

chrome://branding/content/icon128.png is flagged as unused by my test in bug 1316187. This file was added in bug 750106, but doesn't seem to be used anymore.
At some point it was used by the control center, but that was removed at https://hg.mozilla.org/mozilla-central/rev/fde6d2f7d6bc#l12.117

Note: it's listed as a duplicate of browser/icons/mozicon128.png in browser/installer/allowed-dupes.mn

Comment 1

[Dan Banner]

Attachment: Bug 1339420 - Remove references to unused chrome://branding/content/icon128.png

Review commit: https://reviewboard.mozilla.org/r/172160/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/172160/

Comment 2

[[e-mail]]

Comment on attachment 8900715 [details]
Bug 1339420 - Remove references to unused chrome://branding/content/icon128.png

https://reviewboard.mozilla.org/r/172160/#review178944

we should remove icon files in browser/branding/xxx/mozicon128.png as well

Comment 3

[[e-mail]]

We'll need icon128/mozicon128 in bug 1392468 to provid 2x size of the overlay notification icon.

Since there's a new dependency, should we still remove these icons?

Comment 4

[Mark Banner (:standard8)]

(In reply to Fred Lin [:gasolin] from comment #2)
> Comment on attachment 8900715 [details]
> Bug 1339420 - Remove references to unused
> chrome://branding/content/icon128.png
> 
> https://reviewboard.mozilla.org/r/172160/#review178944
> 
> we should remove icon files in browser/branding/xxx/mozicon128.png as well

I wasn't convinced when looking at this. The icon is packaged separately here:

http://searchfox.org/mozilla-central/rev/cd82cacec2cf734768827ff85ba2dba90a534c5e/browser/branding/branding-common.mozbuild#49

iirc, I've seen comments in the past that at least some Linux distros use it, or that there have been some references pointing people to use it as a desktop icon.

Hence we went for the safer option of removing the clearly unused chrome references to it.

Comment 5

[Florian Quèze [:florian]]

Comment on attachment 8900715 [details]
Bug 1339420 - Remove references to unused chrome://branding/content/icon128.png

https://reviewboard.mozilla.org/r/172160/#review179000

r+ because the patch looks good, but as pointed out in comment 3, this seems to be in direct conflict with the patch from bug 1392468, so we may end up wontfixing this bug.

Comment 6

[[e-mail]]

since icon128 has new dependency now, set this issue as wontfix