Bug 2071819 - turn obfuscation on in Fenix. #438 (firefox-autoland@autoland)
|
|
|
| Warnings |
|
| Blockers |
|
| Branches | actually-really-switch-on-obfuscation -> autoland |
| Repo | firefox-autoland@autoland (git) (git://github.com/pollym/firefox.git) |
| Author | pollym |
| State | closed |
| Commit Title |
Bug 2071819 - turn obfuscation on in Fenix. |
| Commit Body |
by removing -dontobfuscate. NB Fenix now has the broadest possible keep rule to allow us to retain its full class and function names in stack traces for now. We can work on this to narrow the rule to what is really required and further improve performance. [here is a green try, updated after rebase and rule tweak to retain other r8 optimisation in fenix](https://treeherder.mozilla.org/jobs?repo=try&revision=37f18896f408a5885b30b518889390c58a7f877e) (i rebased to try and get rid of the noise in the pr bots, which has nothing to do with this change, but it didn't work 🥲 ) [perf compare running here](https://perf.compare/compare-lando-results?landoInstance=lando-prod-2025&baseLando=116720&newLando=116918&baseRepo=try&newRepo=try&framework=15&initialized=1) - expecting this to be neutral (due to current broad keep rule). this pr brings apk size down 0.8MB though, which is nice |
Landings
Landing requested on , by pmceldowney@mozilla.com.
Landing Job: 117394
Last update:
Result: 7aaa812fd8f3435c96ea278e393544a353f196e3 on firefox-autoland@autoland (git)
Links: Treeherder