Closed Bug 1398657 Opened 7 years ago Closed 7 years ago

(photon) Remove unused or redundant drawables

Categories

(Firefox for Android Graveyard :: General, enhancement)

Unspecified
Android
enhancement
Not set
normal

Tracking

(firefox57 fixed)

RESOLVED FIXED
Firefox 57
Tracking Status
firefox57 --- fixed

People

(Reporter: jwu, Assigned: jwu)

References

Details

(Whiteboard: [FNC][SPT57.3][INT])

Attachments

(2 files)

We have added some mdpi drawables added when implementing Photon, however, these dpi resources has been dropped since in Australis, we should remove all Photon related mdpi resources as well.

Also, there are some drawables both exist in `[h/xh/xxh/xxxh]dpi/` and `[h/xh/xxh/xxxh]dpi-v11/`, but since our minimum SDK is 16, resources in [h/xh/xxh/xxxh]dpi/ are redundant and should be removed.
Blocks: fatfennec
Comment on attachment 8906950 [details]
Bug 1398657 - Part 1: Remove mdpi drawables.

https://reviewboard.mozilla.org/r/178690/#review184114
Attachment #8906950 - Flags: review?(cnevinchen) → review+
Comment on attachment 8906951 [details]
Bug 1398657 - Part 2: Remove redundant drawables.

https://reviewboard.mozilla.org/r/178692/#review184116
Attachment #8906951 - Flags: review?(cnevinchen) → review+
Pushed by nechen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/588f15fdda38
Part 1: Remove mdpi drawables. r=nechen
https://hg.mozilla.org/integration/autoland/rev/9dcc9c6cb581
Part 2: Remove redundant drawables. r=nechen
Whiteboard: [FNC][SPT57.3][INT]
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.