1398657 - (photon) Remove unused or redundant drawables

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, enhancement)

Description

[Jing-wei Wu [:jwu]]

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.

Comment 1

[Jing-wei Wu [:jwu]]

Attachment: Bug 1398657 - Part 1: Remove mdpi drawables.

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

Comment 2

[Jing-wei Wu [:jwu]]

Attachment: Bug 1398657 - Part 2: Remove redundant drawables.

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

Comment 3

[Nevin Chen(Not active on Bugzilla)]

Comment on attachment 8906950 [details]
Bug 1398657 - Part 1: Remove mdpi drawables.

https://reviewboard.mozilla.org/r/178690/#review184114

Comment 4

[Nevin Chen(Not active on Bugzilla)]

Comment on attachment 8906951 [details]
Bug 1398657 - Part 2: Remove redundant drawables.

https://reviewboard.mozilla.org/r/178692/#review184116

Comment 5

[Pulsebot]

Pushed by [e-mail]:
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

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/588f15fdda38
https://hg.mozilla.org/mozilla-central/rev/9dcc9c6cb581