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

Unified Diff: chrome/android/java/proguard.flags

Issue 2131353002: Moving all for test keeps into for test flags file. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@testapk
Patch Set: rebased Created 4 years, 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | testing/android/proguard_for_test.flags » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/proguard.flags
diff --git a/chrome/android/java/proguard.flags b/chrome/android/java/proguard.flags
index f3e6c117344447dea964b0ea1935ba90e6279a45..cf546d1ba7ebc350d2eac18eebae58716d6d46b8 100644
--- a/chrome/android/java/proguard.flags
+++ b/chrome/android/java/proguard.flags
@@ -8,11 +8,6 @@
-keep public class com.google.android.apps.chrome** extends android.app.Fragment
-keep public class org.chromium** extends android.app.Fragment
-# TODO(aurimas): figure out why we need to keep these classes.
--keep class org.chromium.base.test.** {
- *;
-}
-
# Keep protobuf code used via reflection
# TODO(tonyg): Removing these -keeps results in new notes, but nothing seems to
# break. This exclusion costs almost 100k of dex size so consider replacing it
@@ -54,85 +49,3 @@
-keepclassmembers,allowshrinking,allowobfuscation class android.support.v7.widget.AppCompatButton {
!static final <fields>;
}
-
-# Everything below this is kept because they are referenced by the test APK.
--keep class android.support.v7.mediarouter.R* {
- *;
-}
-
--keep class android.support.v7.media.MediaRouteProvider** {
- *;
-}
-
--keep class android.support.v4.app.FragmentManager** {
- *;
-}
-
--keep class android.support.v4.app.DialogFragment** {
- *;
-}
-
--keep class android.support.v7.app.NotificationCompat** {
- *;
-}
-
--keep class android.support.v7.app.AlertDialog** {
- *;
-}
-
--keep class com.google.android.gms.cast.CastMediaControlIntent* {
- *;
-}
-
--keepnames class com.google.android.gms.gcm.** {
- *;
-}
-
--keepclassmembers class com.google.android.gms.gcm.TaskParams {
- public <init>(java.lang.String);
-}
-
--keepnames class jp.tomorrowkey.android.gifplayer.** {
- public *;
-}
-
-# Keeping @VisibleForTesting and its annotated classes + methods
--keep @interface org.chromium.base.VisibleForTesting
--keep @org.chromium.base.VisibleForTesting class **
--keepclasseswithmembers class * {
- @org.chromium.base.VisibleForTesting <methods>;
-}
-
-# Used in tests.
--keep class android.support.v4.view.ViewCompat {
- public static int getLayoutDirection(android.view.View);
-}
-
-# flingViewport is used by Android WebView and a Chrome test.
--keepclassmembers class org.chromium.content.browser.ContentViewCore {
- public void flingViewport(long, int, int);
-}
-
-# Needed to compile ChromeTest.apk
--keep class android.support.customtabs.** {
- *;
-}
-
-# TODO(yfriedman): Remove when crbug.com/488192 is fixed.
--dontwarn org.apache.http.conn.scheme.LayeredSocketFactory
-
-# Needed to run ChromeTest.apk
--keepnames class com.google.android.gms.common.GoogleApiAvailability {
- *;
-}
-
-# Needed for chrome_sync_shell_test_apk. Note - these do no affect chrome_apk's
-# size.
--keep class org.chromium.sync.protocol.* { *; }
-
-# These resources are referenced in tests, but not in the real application.
--keepclassmembers class org.chromium.chrome.R$id {
- int webapp_splash_space;
- int mr_chooser_list;
- int find_toolbar;
-}
« no previous file with comments | « no previous file | testing/android/proguard_for_test.flags » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698