From 1a83dede85a0ad6e973174dc011e460db846ffc1 Mon Sep 17 00:00:00 2001 From: Felipe Leme Date: Wed, 21 Sep 2016 10:16:21 -0700 Subject: Use a system property to call different bugreport types. Currently, we define 4 hardcoded init services to launch dumpstate with different command-line options (since dumpstate must be launched by root): - bugreport - bugreportplus - bugreportwear - bugreportremote This approach does not scale well; a better option is to have just one service, and let the framework pass the extra arguments through a system property. BUG: 31649719 Test: manual Change-Id: I1d627a73bfbdd9930c605fa4468c15282a23bb65 --- packages/Shell/src/com/android/shell/BugreportProgressService.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'packages/Shell/src') diff --git a/packages/Shell/src/com/android/shell/BugreportProgressService.java b/packages/Shell/src/com/android/shell/BugreportProgressService.java index 18c7dbec9ca3..772c344fbfdb 100644 --- a/packages/Shell/src/com/android/shell/BugreportProgressService.java +++ b/packages/Shell/src/com/android/shell/BugreportProgressService.java @@ -180,7 +180,7 @@ public class BugreportProgressService extends Service { /** System property (and value) used to stop dumpstate. */ // TODO: should call ActiveManager API instead private static final String CTL_STOP = "ctl.stop"; - private static final String BUGREPORT_SERVICE = "bugreportplus"; + private static final String BUGREPORT_SERVICE = "bugreport"; /** * Directory on Shell's data storage where screenshots will be stored. -- cgit v1.2.3-59-g8ed1b