Closed
Bug 1348522
Opened 7 years ago
Closed 7 years ago
Remove unused files from chrome://global/skin/media/
Categories
(Toolkit :: Themes, enhancement)
Toolkit
Themes
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: florian, Assigned: dbugs)
References
Details
Attachments
(1 file)
The 2 following files are shipped on all platforms but don't seem to be used anywhere: chrome://global/skin/media/clicktoplay-bgtexture.png chrome://global/skin/media/videoClickToPlayButton.svg
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → dbugs
Comment hidden (mozreview-request) |
Reporter | ||
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8869478 [details] Bug 1348522 - Remove unused files from chrome://global/skin/media/ https://reviewboard.mozilla.org/r/141098/#review144682 Thanks for taking this bug. The patch looks good, but I'm not r+'ing yet because there's one mention of clicktoplay-bgtexture.png left to remove: on Mac toolkit/skin/classic/global/media/clicktoplay-bgtexture.png was a duplicate of toolkit/skin/classic/global/icons/tabprompts-bgtexture.png, so we can now remove 2 lines from http://searchfox.org/mozilla-central/rev/24c443a440104dabd9447608bd41b8766e8fc2f5/browser/installer/allowed-dupes.mn#235 :-)
Attachment #8869478 -
Flags: review?(florian)
Comment hidden (mozreview-request) |
Reporter | ||
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8869478 [details] Bug 1348522 - Remove unused files from chrome://global/skin/media/ https://reviewboard.mozilla.org/r/141098/#review145212 Thanks!
Attachment #8869478 -
Flags: review?(florian) → review+
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8649b10a34c1 Remove unused files from chrome://global/skin/media/ r=florian
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8649b10a34c1
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•