Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(383)

Side by Side Diff: base/android/release_proguard_config.flags

Issue 2329573002: Refactor base's proguard flags to be more usable by cronet (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « base/android/proguard/chromium_code.flags ('k') | chrome/android/chrome_public_apk_tmpl.gni » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 # Allows Proguard freedom in removing these log related calls. We ask for debug
2 # and verbose logs to be stripped out in base.Log, so we are just ensuring we
3 # get rid of all other debug/verbose logs.
4 -assumenosideeffects class android.util.Log {
5 static *** d(...);
6 static *** v(...);
7 static *** isLoggable(...);
8 }
9
10 # The following chart was created on July 20, 2016, to decide on 3 optimization
11 # passes for Chrome.
12 # optimization passes | time | .dex size | dirty memory per process
13 # -----------------------------------------------------------------
14 # 1 | 0:48 | 5805676 | 488972
15 # 2 | 1:07 | 5777376 | 487092
16 # 3 | 1:24 | 5772192 | 486596
17 # 4 | 1:42 | 5771124 | 486484
18 # 5 | 1:56 | 5770504 | 486432
19 -optimizationpasses 3
20
21 # Class merging messes up stacktraces beyond the point of them being
22 # deobfuscatable. If turned on, it would give us a 2% reduction in .dex size.
23 -optimizations !class/merging/*
24
25 # Allowing Proguard to change modifiers. This change shrinks the .dex size by
26 # ~1%, and reduces the method count by ~4%.
27 -allowaccessmodification
28
29 # The support library contains references to newer platform versions.
30 # Don't warn about those in case this app is linking against an older
31 # platform version. We know about them, and they are safe.
32 -dontwarn android.support.**
OLDNEW
« no previous file with comments | « base/android/proguard/chromium_code.flags ('k') | chrome/android/chrome_public_apk_tmpl.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698