Closed
Bug 499226
Opened 16 years ago
Closed 16 years ago
Remove unneeded branding icons for Linux
Categories
(Firefox :: General, defect)
Tracking
()
RESOLVED
FIXED
Firefox 3.6a1
People
(Reporter: faaborg, Assigned: Dolske)
Details
Attachments
(1 file)
|
98.81 KB,
patch
|
mconnor
:
review+
|
Details | Diff | Splinter Review |
Follow up bug from bug 496472 (removing unneeded branding images) and bug 495250 (landing the new branding on Linux). We can now remove these files since they are old and no longer used:
mozicon16.xpm
mozicon50.xpm
document.png
|
Assignee | |
Comment 1•16 years ago
|
||
Removes the mozicon[50|16].xpm reference from the tree, as well as a couple of xpm files from xulrunner.
I didn't remove document.png, because it looks like it's actually still used. :-(
See: browser/components/shell/src/nsGNOMEShellService.cpp#288, where it copies document.png to ~/.icons/firefox-document.png when you trigger the "set default browser" code. This seems kinda crappy, since it means icon updates should repeat the copy or the user has the old version floating around.
Assignee: nobody → dolske
Attachment #386777 -
Flags: review?(mconnor)
|
||
Updated•16 years ago
|
Attachment #386777 -
Flags: review?(mconnor) → review+
|
|
||
Comment 2•16 years ago
|
||
Comment on attachment 386777 [details] [diff] [review]
Patch v.1
r=mconnor
|
|
Assignee | |
Comment 3•16 years ago
|
||
XPM begone!
Pushed http://hg.mozilla.org/mozilla-central/rev/ee1640509c32
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.6a1
|
|
Assignee | |
Comment 4•16 years ago
|
||
Filed bug 504645 for removing document.png.
|
||
Updated•15 years ago
|
Flags: in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•