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

Side by Side Diff: base/android/proguard/chromium_apk.flags

Issue 2493543002: Disable proguard optimization that removes write-only fields. (Closed)
Patch Set: Created 4 years, 1 month 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 # Copyright 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # Contains flags that we'd like all Chromium .apks to use. 5 # Contains flags that we'd like all Chromium .apks to use.
6 6
7 # Keep line number information, useful for stack traces. 7 # Keep line number information, useful for stack traces.
8 -keepattributes SourceFile,LineNumberTable 8 -keepattributes SourceFile,LineNumberTable
9 9
10 # Keep all CREATOR fields within Parcelable that are kept. 10 # Keep all CREATOR fields within Parcelable that are kept.
(...skipping 29 matching lines...) Expand all
40 # ----------------------------------------------------------------- 40 # -----------------------------------------------------------------
41 # 1 | 0:48 | 5805676 | 488972 41 # 1 | 0:48 | 5805676 | 488972
42 # 2 | 1:07 | 5777376 | 487092 42 # 2 | 1:07 | 5777376 | 487092
43 # 3 | 1:24 | 5772192 | 486596 43 # 3 | 1:24 | 5772192 | 486596
44 # 4 | 1:42 | 5771124 | 486484 44 # 4 | 1:42 | 5771124 | 486484
45 # 5 | 1:56 | 5770504 | 486432 45 # 5 | 1:56 | 5770504 | 486432
46 -optimizationpasses 3 46 -optimizationpasses 3
47 47
48 # Class merging messes up stacktraces beyond the point of them being 48 # Class merging messes up stacktraces beyond the point of them being
49 # deobfuscatable. If turned on, it would give us a 2% reduction in .dex size. 49 # deobfuscatable. If turned on, it would give us a 2% reduction in .dex size.
50 -optimizations !class/merging/* 50 # Write-only field removal removes some static fields that are intended to stay
51 # around until the end of execution.
52 -optimizations !class/merging/*,!field/removal/writeonly
51 53
52 # Allowing Proguard to change modifiers. This change shrinks the .dex size by 54 # Allowing Proguard to change modifiers. This change shrinks the .dex size by
53 # ~1%, and reduces the method count by ~4%. 55 # ~1%, and reduces the method count by ~4%.
54 -allowaccessmodification 56 -allowaccessmodification
55 57
56 # The support library contains references to newer platform versions. 58 # The support library contains references to newer platform versions.
57 # Don't warn about those in case this app is linking against an older 59 # Don't warn about those in case this app is linking against an older
58 # platform version. We know about them, and they are safe. 60 # platform version. We know about them, and they are safe.
59 -dontwarn android.support.** 61 -dontwarn android.support.**
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