Open Bug 1665760 Opened 4 years ago Updated 2 years ago

Audit ProGuard configuration

Categories

(GeckoView :: General, task, P3)

Product:
GeckoView
Component:
General
Platform:
Unspecified
All
Type:
task

Tracking

(Not tracked)

People

(Reporter: snorp, Unassigned)

Details

We undoubtedly have a bunch of garbage in there leftover from Fennec. Do we even need to ship a ProGuard config?

Do we even need to ship a ProGuard config?

I think we do, e.g. we need to at least keep JNI entry points:

https://searchfox.org/mozilla-central/rev/5efefd3ef214ed6d3234ba245c1da3004ead94e0/mobile/android/geckoview/proguard-rules.txt#140-156

Also, from what I can see this is where we tell consumers to include our proguard config:

https://searchfox.org/mozilla-central/rev/5efefd3ef214ed6d3234ba245c1da3004ead94e0/mobile/android/geckoview/build.gradle#77

Severity: -- → S3
Priority: -- → P3

Tasks should have severity N/A.

Severity: S3 → N/A
You need to log in before you can comment on or make changes to this bug.