summaryrefslogtreecommitdiff
path: root/packages/Shell/src
AgeCommit message (Collapse)Author
2017-03-23Add API to request Bug Report with a certain title and descriptionNaveen Kalla
ActivityManagerService will send the details to dumpstate and BugreportProgressService will get the details from dumpstate and use that for the notification. Bug: 33561517 Test: Modify TelephonyMonitor to use this API and ensure that the notification uses that the title and description sent as arguments to the API. File a bug using betterBug and ensure that the information gets populated automatically. Change-Id: I455f1df2278e5d569dd19c74245a0704dcf5a321
2017-03-14Enabling bugreport notifications on TVDmitri Plotnikov
Bug: 36175844 Test: by generating a bugreport (long-press DPAD_CENTER + BACK) Test: verified that BugreportProgressServiceTest still passes Change-Id: If5bf53512c60a359838aa76dc61b90d4e67a45df
2017-02-24Fix sharing bugreports from lockscreenJorim Jaggi
When launching ChooserActivity from lockscreen, we will start it in stopped state because lockscreen is still showing, meaning that the activity goes through start -> resume -> pause -> stop immediately after it was launched, and will be later resumed once Keyguard actually goes away. However, ResolverActivity finished itself in onStop. We add a private extra to change this behavior for sharing bugreports. Test: Take bugreport, double tap on it on lockscreen Test: com.android.shell.BugreportReceiverTest$1 Bug: 33009364 Change-Id: I973b2c71587950499b7c88b16af9cf1387795e17
2017-02-15Moved bugreport intents to the com.android.internal namespace.Felipe Leme
Change-Id: I5ad53668b9393188cee4d8b0382f9c303ff95777 Fixes: 35275802 Test: manual verification
2017-02-14Merge "Added more logging on BugreportProgressService."TreeHugger Robot
2017-02-10Added more logging on BugreportProgressService.Felipe Leme
BUG: 35010191 BUG: 35077822 Test: manual verification Test: BugreportReceiverTest passes Change-Id: Iaa37f633c6415127a3efa07c166f5a771a13ff5a
2017-02-10Dismiss keyguard before displaying bugrepot details dialog.Felipe Leme
Fixes: 27494228 Test: manual verification Test: BugreportReceiverTest passes Change-Id: I7755fc2d3d4f1f9b55301c65f7a58c9374e93664
2017-01-05Bugreport sharing is now work profile awareMakoto Onuki
Now we look at work profile too, if available, when looking for a preferred email address. TODO: The chooser activity should default to work profile too, but it seems to require a lot more work, so not done yet. Test: manual tests Test: adb shell am instrument -w -e class com.android.shell.BugreportProgressServiceTest com.android.shell.tests Bug 30865666 Change-Id: I9b4776d53455a23bfdc4960da25e98bd1dc1b2a1
2016-11-15Fixed obsolete comment.Felipe Leme
Bug:32906084 Test: not needed Change-Id: I0c79e4ffb66405a30d3c83d2e7654cbef413f6e1
2016-11-11Refactored Shell to use IDumpstateListener.Felipe Leme
BugreportProgressService used to poll system properties to get the progress of the underlying dumpstate process, but now dumpstate provides a IDumpstateListener and IDumpstateToken binder objects for the same purpose. Test: BugreportReceiverTest passes Test: manual verification BUG: 31636879 Change-Id: I81d551781a511cb1cc218a27452875b0bb267f7a
2016-11-08Update package names to work with the proto3 compilerTamas Berghammer
Bug: b/28974522 Change-Id: I5f3adf4946ee4ba1e09e4f40afe83c151405972a
2016-09-28Remove support for archives from External and Bugreport providers.Tomasz Mikolajewski
Change-Id: Ia281fe05a0a6a5dd6ff1b66d7197312cf4391793 Test: Compiles. Bug: 31783726
2016-09-21Use a system property to call different bugreport types.Felipe Leme
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
2016-08-31Push delete notification when Bug reports are deleted. am: 86805aa3af am: ↵Ben Lin
64146c5d24 am: b01be62d58 Change-Id: I8e1b6dc42e44307feef1ba912a3465c2d25594e1
2016-08-29Push delete notification when Bug reports are deleted.Ben Lin
Bug: 28768706 Change-Id: I6d3e91fe7d12be44918f97cef12c99d936401270
2016-08-08Cancel dialog before stopping service.Felipe Leme
Fixes: 30746086 Change-Id: Iea8a197a2785259dd80aa8130d79ded484562826
2016-08-04Merge "Enable name text field when re-using the details dialog."TreeHugger Robot
2016-08-03Enable name text field when re-using the details dialog.Felipe Leme
Change-Id: I10671fc00ea918da901f36f122641e646057888c Fixes: 30642226
2016-08-04Merge "Fixed BugreportReceiverTest failures:"TreeHugger Robot
2016-08-03Fixed BugreportReceiverTest failures:Felipe Leme
- testProgress_changeJustDetailsTouchingNotification was failing because the notification mechanism changed and now provides a way to expand and collapse the actions bar, and the test was collapsing it instead of opening the details dialog. It was fixed by tapping the notification content instead of the notification title. - Similarly, openProgressNotification() was refactored to use the bugreport name instead of id. - Uses getObject() (instead of getVisibleObject()) to get the activity from the intent chooser. - Removed the redundant call to back to dismiss the keyboard, which was causing some tests to fail due to an accessibility bug. - Retry a few times in case the bugreport name system property assertion fails, since the property is set by a background thread. - Improved how notifications are canceled. - Created a sleep() helper. Besides the test case improvements, it also removed a redundant call to setCancelable() in the main code. BUG: 30641229 BUG: 30639703 Change-Id: Icd79fada22d0b8e4be034068c3e9143ef0134eed
2016-08-03Fixed year on copyright notice.Felipe Leme
BUG: 30429392 Change-Id: If6477ccde5820dd118ad92bb5d4f74d938cf0d83
2016-08-03Removed unused imports and fixed copyright notice.Felipe Leme
BUG: 30429392 Change-Id: I3c849e507437ef733b04954af68bd47d00dc5165
2016-08-02Use SurfaceControl.screenshot() instead of screencap for screenshots.Felipe Leme
*** cherrypick of 8cfc23f *** Change-Id: I5a9e7bbc5f3ae176ac5ae7209a133526992e92d4 Fixes: 30429392
2016-07-28Use SurfaceControl.screenshot() instead of screencap for screenshots.Felipe Leme
Change-Id: I5a9e7bbc5f3ae176ac5ae7209a133526992e92d4 Fixes: 30429392
2016-07-28Always use chooser for Share Bug Report intent.Felipe Leme
When using startActivity(intent), the platform allows the user to pick the same activity for the intent, which can be a problem for sharing bug reports when the device has personal and work profiles. Change-Id: I1fd66905feb5d894307bbe5656c2aec705a59ab7 Fixes: 22115530
2016-07-19Merge \\"Use the Shell app to show the bugreport notification on Wear.\\" ↵Wei Liu
into nyc-mr1-dev am: 1dfbf794b5 am: bc3c068189 Change-Id: I4555c232c5fc9d10a9b9d9cd0d64ac80720e30e9
2016-07-18Use the Shell app to show the bugreport notification on Wear.Wei Liu
b/30041885 Change-Id: I998aa1606eb12f74a407fbafd9abc11c9454fb66
2016-07-18Close details dialog when bugreport is canceled by user.Felipe Leme
BUG: 30158896 Change-Id: I0eab22586f6b431f2abe837088d48a655e03d213
2016-05-23Register change Uri and notify changes in bugreportServices.Ben Lin
Bug: 28767380 Change-Id: Ia4d1f5a2e44881d300f2869c628d4990406caf40 (cherry picked from commit 6c9ff513c6d7d3de7ab8920d88111c2b1dc1bb1f) (cherry picked from commit 93fafd67a6127a35ba9b7a0f7de3a0b54309d4c2)
2016-05-18Register change Uri and notify changes in bugreportServices.Ben Lin
Bug: 28767380 Change-Id: Ia4d1f5a2e44881d300f2869c628d4990406caf40 (cherry picked from commit 6c9ff513c6d7d3de7ab8920d88111c2b1dc1bb1f)
2016-04-27Add archive support to Bug report docs provider.Steve McKay
Bug: 28386918 Change-Id: I2270324be3f020d73a68151002f885f70eb1449c
2016-04-25Don't opt-out of warning dialog by default on user builds.Felipe Leme
BUG: 28341946 Change-Id: I9ce6bf52a02a14055d4ded41aac50c79abdb4ee9
2016-04-22Show a toast when details cannot be added to zip file.Felipe Leme
BUG: 28291423 Change-Id: I7febf66ef23f38b41fb1909324ee2260444518f6
2016-04-20Set BugreportProgressService to run on foreground.Felipe Leme
BugreportProgressService do not persist the user-provided information (like details and screenshot paths), so if it's killed by the framework, that info is lost. Running it as foreground mitigates the changes of it being killed. BUG: 27431998 BUG: 28291423 Change-Id: I2f58507beb38309628f2f19d3f7f950d07eca16f
2016-04-20Merge "Only try to add details to zip once." into nyc-devTreeHugger Robot
2016-04-20Only try to add details to zip once.Felipe Leme
When the user enters details (title or description) to the bugreport, Shell tries to add a title.txt (and/or description.txt) to the zip and uses 2 instance variables (addedDetailsToZip and addingDetailsToZip) to control its state. The problem with the current approach is that if there is a failure adding the entries (for example, if the entries already exist), these variables are not updated and hence when the user taps Share, it will try to add the entries again, which most likely would fail. BUG: 28291423 Change-Id: I56a71256be4f8de15f8126b815334277319e8e8a
2016-04-19Show "Android System" instead of "Shell" on notifications.Felipe Leme
BUG: 26517701 Change-Id: If2543b8db0fdd913365df6f69cf26146d32d4b15
2016-04-18Remove initial screenshot on interactive bugreport.Felipe Leme
One of the changes in the 'interactive bugreport' bugreport workflow introduced on N is that the initial screenshot was taken right away (by Shell, not dumpstate). Unfortunately, such initial screenshot is often delayed when the system is overload. Also, if the user is not interested in a screenshot, it would be adding more load on the system unnecessarily. Given these issues, and the fact that the user can still easily take an initial screenhsot by selecting the proper notification action, the initial screenshot is being removed. BUG: 28167977 Change-Id: I2cf6616ce3124102b62ec9a36dc5a0ce6455a909
2016-04-13Merge "Revert warning logic so it has a "don't show again" message." into ↵TreeHugger Robot
nyc-dev
2016-04-13Revert warning logic so it has a "don't show again" message.Felipe Leme
BUG: 28140003 Change-Id: I93e7b1494a0f4c5ca080fbe9dd94dc2168092ffa
2016-04-12Removed redundant notification title.Felipe Leme
When a bugreport is finished with a pending notification, it already display a subtext explaining the situation - not only the extra title is redundant, but it's too large. BUG: 27583025 Change-Id: I8d8171faf7b8b86b34f6d860555839918be10550
2016-04-12Merge "Minor improvements useful for debugging." into nyc-devFelipe Leme
2016-04-06Fixed the contentinfo api usage in the bugreportSelim Cinek
Removed the percentage shown in the header and migrated the name to the subtext as contentinfo was deprecated. Change-Id: Ifd79a67cad8958049bd29b8eb4c9bcbb4822688b
2016-04-04Minor improvements useful for debugging.Felipe Leme
- Better dump of received intents by displayed the relevant extras. - Gracefully handles the case where the bugreport file URI is invalid during development. BUG: 27996121 Change-Id: I97a48d1e9641142a43c66c1dded2f7f322dc66aa
2016-03-30Add menu option to show/hide internal storage.Aga Wronska
This reverts commit 7c4395804e450533afb553ab992c47f737da8a9b. Bug: 27683276 Change-Id: Id1a310b57ab7fa3c8c14b464786473f211eaf41f
2016-03-28Removed FLAG_ADVANCED.Felipe Leme
On M, internal storage such as bugreport files were only shown when user selected the "Show internal storage", but such UI has changed on N. BUG: 27862860 Change-Id: I1edf086a9f9345303595ee952e4646764709d36d
2016-03-23Proper handle of duplicated BUGREPORT_STARTED.Felipe Leme
When Shell receives a BUGREPORT_STARTED intent for a process it's already monitoring, it should completely ignore it, but current it's taking an extra screenshot. BUG: 27804637 Change-Id: I733cacfee5e9c82646a3295b50c3856b6e0352c3
2016-03-22Merge "Changed logic when dumpstate's max progress increases." into nyc-devFelipe Leme
2016-03-22Changed logic when dumpstate's max progress increases.Felipe Leme
When dumpstate starts, it estimates its maximum duration and sends it through an extra on BUGREPORT_STARTED; as it progress, it sets a system property with its current progress and if the progress value overflows the estimated max, it increases the max as well. Shell uses the max/progress to display the progress % in the system notification, and need to handle the scenario where the max changes. The initial implementation would recalculate the progress, with makes it swing back and forth as dumpstate increases the max. This CL changes the Shell logic so the progress never go back, just forward. The drawback of this approach is that if dumpstate underestimate the maximum, the progress might get stuck in a high value (99%) early on, but such issue will be addressed in the long term by tuning the estimated max value. BUG: 26354314 Change-Id: I3a5416acaffaaa43fd28d2f1f8ec8ea12aa0d91e
2016-03-21Add config flag to show/hide internal storage.Aga Wronska
Hide internal storage by default. Revert "Always show intern storage." This reverts commit 7c4395804e450533afb553ab992c47f737da8a9b. Bug: 27683276 Change-Id: I04ea8b9a307babcea003f9bec200d41265a42c7f
generated by cgit v1.2.3 (git 2.25.1) at 2025-10-03 15:09:38 +0000