Make callers use wrapper functions in ActivityManagerService.java.
The code to send broadcast to trigger API workflow (after checking the API
flag) is added in requestBugreport() in ActivityManagerService.java.
Remove that logic from this file.
Bug: 137825297
Bug: 141355059
Test: builds and full/interactive bugreports work as expected
Change-Id: Ibc305e4aca7d38fd4878b60f3f7f8c3274719c13
Bug: 136548098
Test: build, flash, take bugreport from Settings uses API workflow by
default. Turning off the feature flag from Settings uses old bugreport
workflow
Change-Id: Icbfa82e2dc218769fdcc03d15ce3cc7bb50cc4ef
To avoid flag duplication, this flag is moved to FeatureFlagUtils.java
where other callers can also use this flag. Using this flag's value from
FeatureFlagUtils instead of the local FeatureFlag file in Settings.
Bug: 136548292
Test: Settings can still access this feature flag as before
Test: Builds and API workflow gets triggered by turning on the flag
Change-Id: I60e4d96033219c934a43e728705fee93e338d60e
Send broadcast to shell, so that it makes a call to bugreport API
directly.
Permission protected call is made so that not any app can send this
broadcast intent to shell.
Use Settings feature flag to turn on this workflow and leave the old workflow
untouched.
Bug: 123617758
Test: build and manual test by triggering bugreports from the device
Change-Id: Iec43fa1e5e379e2f5520a48f7fe036d59e2a8d28
Having consistent import order will reduce chance of merge
conflict between internal and external master
Test: rebuild
Change-Id: I0b1a170967ddcce7f388603fd521f6ed1eeba30b
This patch focused on fixing compile errors and some runtime errors.
Test: We can't test it now. But we will have an integration test later.
Bug: 110259478
Change-Id: I16c471ddcd0fa1460c665b7f74d86fcace5ee67b
The delay was useful when the interactive bugreport took an initial
screenshot, which is not the case anymore.
BUG: 30429546
Test: manual
Change-Id: I648db4e1acdea3aa89df9205148cac43880763be
Previously the call was made on takeBugreport(), which caused a NPE on
interative report calls because the context was null since it was
handled by a handler.
BUG: 27354167
Change-Id: I19e91cb4b43a7aa0bfcdab440c4cd4225110fdfb
* Normal Report: the new, enhanced option is more user-friendly (it
shows a system notification with the progress, allow user to cancel,
etc...), at the cost of consuming more resources. When this option is
selected, the bugreport will be delayed 2 seconds so user can dismiss
the Settings app before a screenshot is taken.
* Emergency Report: a lighter approach that is less intrusive and
should be use when the phone is unresponsive or too slow.
BUG: 26034608
BUG: 11665244
Change-Id: If9c0242a163c8c33adecfc8a65af561e839bab8d
'bugreportplus' is more user-friendly (it shows a system notification
with the progress, allow user to cancel, etc...), at the cost of
consuming more resources. As such, the "Take Bug Report" UI will be
changed to offer the user a combo with these 2 options, but for now it's
always going to be 'bugreportplus'
BUG: 26034608
Change-Id: Idbb325fe7cd18e487197b2fb79cfc264e9859570
