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

Side by Side Diff: chrome/android/java/proguard.flags

Issue 2067303002: Proguard: Logging stripped and allowaccessmodification turned on (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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 | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Keep line number information, useful for stack traces. 1 # Keep line number information, useful for stack traces.
2 -keepattributes SourceFile,LineNumberTable 2 -keepattributes SourceFile,LineNumberTable
3 3
4 # Keep the annotations. 4 # Keep the annotations.
5 -keep @interface *** 5 -keep @interface ***
6 6
7 # Disable obfuscation for the following two packages. 7 # Disable obfuscation for the following two packages.
8 -keepnames class com.google.android.apps.chrome.**,org.chromium.** { 8 -keepnames class com.google.android.apps.chrome.**,org.chromium.** {
9 *; 9 *;
10 } 10 }
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
86 # platform version. We know about them, and they are safe. 86 # platform version. We know about them, and they are safe.
87 -dontwarn android.support.** 87 -dontwarn android.support.**
88 88
89 # Proguard mistakenly optimizes away some null checks on final fields 89 # Proguard mistakenly optimizes away some null checks on final fields
90 # in the support library. 90 # in the support library.
91 # See https://sourceforge.net/p/proguard/bugs/531/#fdc9/41e2 91 # See https://sourceforge.net/p/proguard/bugs/531/#fdc9/41e2
92 -keepclassmembers,allowshrinking,allowobfuscation class android.support.v7.widge t.AppCompatButton { 92 -keepclassmembers,allowshrinking,allowobfuscation class android.support.v7.widge t.AppCompatButton {
93 !static final <fields>; 93 !static final <fields>;
94 } 94 }
95 95
96 # Allowing Proguard to change modifiers. This change shrinks the .dex size by
97 # ~1.1%, and reduces the method count by ~4.3%.
98 -allowaccessmodification
99
100 # Allows Proguard freedom in removing these log related calls. We ask for debug
101 # and verbose logs to be stripped out in base.Log, so we are just ensuring we
102 # get rid of all other debug/verbose logs.
103 -assumenosideeffects class android.util.Log {
104 static *** d(...);
105 static *** v(...);
106 static *** isLoggable(...);
107 }
108
96 # Everything below this is kept because they are referenced by the test APK. 109 # Everything below this is kept because they are referenced by the test APK.
97 -dontwarn javax.annotation.Nullable 110 -dontwarn javax.annotation.Nullable
98 111
99 -keep class android.support.v7.mediarouter.R* { 112 -keep class android.support.v7.mediarouter.R* {
100 *; 113 *;
101 } 114 }
102 115
103 -keep class android.support.v7.media.MediaRouteProvider** { 116 -keep class android.support.v7.media.MediaRouteProvider** {
104 *; 117 *;
105 } 118 }
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 # Needed to run ChromeTest.apk 174 # Needed to run ChromeTest.apk
162 -keepnames class com.google.android.gms.common.GoogleApiAvailability { 175 -keepnames class com.google.android.gms.common.GoogleApiAvailability {
163 *; 176 *;
164 } 177 }
165 178
166 #-keep public class android.net.http.SslCertificate 179 #-keep public class android.net.http.SslCertificate
167 #-keep public class android.webkit.WebVie 180 #-keep public class android.webkit.WebVie
168 181
169 #-dontwarn android.webkit.WebView 182 #-dontwarn android.webkit.WebView
170 -dontwarn android.net.http.SslCertificate 183 -dontwarn android.net.http.SslCertificate
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698