Bug 1485045 wants to produce a "fat" AAR, which includes all supported architectures in one package. We would then rely on downstream tools to split appropriately. Those tools don't understand our placing of libraries in assets/, so we need to stop that.

I have a patch for this. https://treeherder.mozilla.org/#/jobs?repo=try&revision=3850f337ca74ae6fdb650345929c469f180c43ad&selectedJob=196173917 Fennec APK size appears to go from 33.5MB to 41.6MB. Is this an acceptable change?

Is this all or nothing? Does this mean we can't have the multiarch-GeckoView-AAR without disabling this for Fennec too?

It's hard to make a call without knowing the results of "Android App Bundles" and "Dynamic Delivery". However I tend to not blocking ourselves on Fennec given that Fennec's fate is sealed.

(In reply to Sebastian Kaspari (:sebastian) from comment #2) > Is this all or nothing? Does this mean we can't have the > multiarch-GeckoView-AAR without disabling this for Fennec too? I think it'll be a PITA, but could be doable. I think it depends on how much we think that size savings has helped Fennec.

It turns out we've probably been getting lucky loading libraries in GeckoView this whole time. The XZ libs are compressed with deflate in the AAR, and that has been getting carried over into the app APK. Apparently it works most of the time because we're unable to actually deflate xz-compressed data. According to the docs, we should be able to specify that these files are not compressed[0], but it doesn't actually work. Further, even if it did work, it's not clear to me that the final APK would carry over the decisions made in the AAR. Apps can work around this by including the following block in the `android` section: aaptOptions { noCompress 'so' } This is not an acceptable long-term solution. [0] https://google.github.io/android-gradle-dsl/current/com.android.build.gradle.internal.dsl.AaptOptions.html#com.android.build.gradle.internal.dsl.AaptOptions:noCompress

Moving to GeckoView so we prioritize it appropriately.

This also uses idiomatic packaging of the native libraries, which will allow easier downstream consumption of GeckoView.

This allows us to use the same Java code for any native platform, enabling a "fat" AAR. Depends on D10775

Pushed by jwillcox@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2bc6db0a2574 Don't use XZ compression for libs on Android r=glandium,nalexander

Backout by apavel@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4e1b2b7e0c37 Backed out changeset 2bc6db0a2574 android raptor failures ona CLOSED TREE

Backed out for failing raptor android Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&group_state=expanded&selectedJob=211775539&revision=7174015fbbd584f9afa8783c1ac119ab3ac8bbf4 Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=211770973&repo=autoland&lineNumber=1017 Backout: https://hg.mozilla.org/integration/autoland/rev/4e1b2b7e0c37fb3fcdb6795537af53510b9308ef

Pushed by jwillcox@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1e076b0054cc Don't use XZ compression for libs on Android r=glandium,nalexander

Backed out for toolchains gradle bustages. Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&fromchange=bae43dfd3700460a55f3559eabbc888975f3a3c0&tochange=a38af3c7c0e4743645888fdaf63c50101d813297&selectedJob=213155692 Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=213155692&repo=autoland Backout link: https://hg.mozilla.org/integration/autoland/rev/a38af3c7c0e4743645888fdaf63c50101d813297

64=wontfix because we won't want to uplift this compression change to Beta.

Pushed by nalexander@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9823c01c5580 Don't use XZ compression for libs on Android r=glandium,nalexander

status-geckoview64=wontfix