summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author David Padlipsky <dpad@google.com> 2024-10-17 01:19:56 +0000
committer David Padlipsky <dpad@google.com> 2024-10-17 01:19:56 +0000
commitf318ad521c8dfa4106219547995aeb773f7fdc83 (patch)
tree8ececdf93e49c4a3efb56ed51b1a6f15de0f0161
parentdd6edec34a3f6683d58473df55ac1580aa35774f (diff)
Update bug report shortcut to launch the bug handler app
Updates the Meta + Ctrl + Backspace shortcut to open the assigned bug handler. Bug: 369926469 Test: Manually on device Flag: EXEMPT due to minor update of debug behavior Change-Id: I41fbfb94372f3698d3bce15195790c118ad563fd
-rw-r--r--services/core/java/com/android/server/policy/PhoneWindowManager.java8
-rw-r--r--services/tests/wmtests/src/com/android/server/policy/TestPhoneWindowManager.java4
2 files changed, 8 insertions, 4 deletions
diff --git a/services/core/java/com/android/server/policy/PhoneWindowManager.java b/services/core/java/com/android/server/policy/PhoneWindowManager.java
index ad5c84026aa6..26651e586f0e 100644
--- a/services/core/java/com/android/server/policy/PhoneWindowManager.java
+++ b/services/core/java/com/android/server/policy/PhoneWindowManager.java
@@ -3608,7 +3608,9 @@ public class PhoneWindowManager implements WindowManagerPolicy {
if (mEnableBugReportKeyboardShortcut && firstDown
&& event.isMetaPressed() && event.isCtrlPressed()) {
try {
- mActivityManagerService.requestInteractiveBugReport();
+ if (!mActivityManagerService.launchBugReportHandlerApp()) {
+ mActivityManagerService.requestInteractiveBugReport();
+ }
} catch (RemoteException e) {
Slog.d(TAG, "Error taking bugreport", e);
}
@@ -4102,7 +4104,9 @@ public class PhoneWindowManager implements WindowManagerPolicy {
case KeyGestureEvent.KEY_GESTURE_TYPE_TRIGGER_BUG_REPORT:
if (complete && mEnableBugReportKeyboardShortcut) {
try {
- mActivityManagerService.requestInteractiveBugReport();
+ if (!mActivityManagerService.launchBugReportHandlerApp()) {
+ mActivityManagerService.requestInteractiveBugReport();
+ }
} catch (RemoteException e) {
Slog.d(TAG, "Error taking bugreport", e);
}
diff --git a/services/tests/wmtests/src/com/android/server/policy/TestPhoneWindowManager.java b/services/tests/wmtests/src/com/android/server/policy/TestPhoneWindowManager.java
index a85f8666d2e1..0f6a33823125 100644
--- a/services/tests/wmtests/src/com/android/server/policy/TestPhoneWindowManager.java
+++ b/services/tests/wmtests/src/com/android/server/policy/TestPhoneWindowManager.java
@@ -822,9 +822,9 @@ class TestPhoneWindowManager {
void assertTakeBugreport(boolean wasCalled) throws RemoteException {
mTestLooper.dispatchAll();
if (wasCalled) {
- verify(mActivityManagerService).requestInteractiveBugReport();
+ verify(mActivityManagerService).launchBugReportHandlerApp();
} else {
- verify(mActivityManagerService, never()).requestInteractiveBugReport();
+ verify(mActivityManagerService, never()).launchBugReportHandlerApp();
}
}