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

Unified Diff: components/cronet/android/sample/javatests/proguard.cfg

Issue 512953002: Use proguard.cfg in cronet sample apk. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address review comments, move UsedByReflection to org.chromium.base. Created 6 years, 4 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 | « components/cronet/android/proguard.cfg ('k') | components/cronet/tools/cr_cronet.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cronet/android/sample/javatests/proguard.cfg
diff --git a/components/cronet/android/sample/javatests/proguard.cfg b/components/cronet/android/sample/javatests/proguard.cfg
new file mode 100644
index 0000000000000000000000000000000000000000..7c740aa458ba1c350f12fd65a398e3466ab23f86
--- /dev/null
+++ b/components/cronet/android/sample/javatests/proguard.cfg
@@ -0,0 +1,46 @@
+# Proguard settings for CronetSampleTestApk.
+
+# Keep classes directly accessed from test apk
+-keep class org.chromium.base.library_loader.LibraryLoaderHelper {
xunjieli 2014/09/02 16:53:16 It looks like the sample is only using https://cod
mef 2014/09/12 20:49:24 base_javatests include org.chromium.base.LibraryLo
+ void deleteWorkaroundLibrariesSynchronously(...);
+ java.io.File getWorkaroundLibDir(...);
+ boolean loadNativeLibrariesUsingWorkaroundForTesting(...);
+}
+-keep class org.chromium.base.CommandLine {
Charles 2014/09/02 16:52:19 If you're keeping all these classes here, adding p
mef 2014/09/12 20:49:24 Good point. I think it makes sense to build Cronet
+ *;
+}
+-keep class org.chromium.base.ObserverList* {
+ *;
+}
+-keep class org.chromium.cronet_sample_apk.CronetSampleActivity {
+ *;
+}
+-keep class org.chromium.net.ChromiumUrlRequestContext {
+ void startNetLogToFile(...);
+ void stopNetLog();
+}
+-keep class org.chromium.net.ChromiumUrlRequestFactory {
+ org.chromium.net.ChromiumUrlRequest createRequest(...);
+ org.chromium.net.ChromiumUrlRequestContext getRequestContext();
+}
+-keep class org.chromium.net.HttpUrlRequest {
+ *;
+}
+-keep class org.chromium.net.HttpUrlRequestFactory {
+ org.chromium.net.HttpUrlRequestFactory createFactory(...);
+ java.lang.String getName();
+}
+-keep class org.chromium.net.HttpUrlRequestFactoryConfig {
+ org.chromium.net.HttpUrlRequestFactoryConfig enableLegacyMode(...);
+}
+-keep class org.chromium.net.HttpUrlRequestListener {
+ *;
+}
+-keep class org.chromium.net.ChunkedWritableByteChannel {
+ *;
+}
+
+# Keep Test classes
+-keep class * extends junit.framework.TestCase {
+ *;
+}
« no previous file with comments | « components/cronet/android/proguard.cfg ('k') | components/cronet/tools/cr_cronet.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698