Use a blacklist to avoid compressing specific file types which already compressed

RESOLVED FIXED

Status

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: yurenju, Assigned: yurenju)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

we need more cpu resource to handle specific file types in zip which has been compressed such as png & jpg, we should use a blacklist to avoid compression those files.
Fabrice, Do you think we should fix it on 1.3T? I know it eats CPU resource but have no idea how much does it eat.
blocking-b2g: --- → 1.3T?
Flags: needinfo?(fabrice)
sorry wrong falg.
blocking-b2g: 1.3T? → ---
Created attachment 8364240 [details] [diff] [review]
no-compress.patch
Attachment #8364240 - Flags: review?(yurenju.mozilla)
Flags: needinfo?(fabrice)
Comment on attachment 8364240 [details] [diff] [review]
no-compress.patch

awesome! thank you :D
Attachment #8364240 - Flags: review?(yurenju.mozilla) → review+
https://github.com/mozilla-b2g/gaia/commit/297d10886f7174485c902bcaf7881135a486ac42
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Revert commit since we got some problem for ringtones on bug 964805, I'll reland this after bug fixed, sorry for that.

https://github.com/mozilla-b2g/gaia/commit/8ed91982a0a85b57b9f7c02941aaea9bc5ad5fb8
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
landed again.

https://github.com/mozilla-b2g/gaia/commit/29b78b6b619452d4bdeac7ac111731a016ec6981
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.