Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "[DO NOT MERGE] Update onBackPressed() function signature" into tm-qpr-dev

Browse Source
This commit is contained in:
Rahul Banerjee
2022-10-18 17:19:16 +00:00
committed by Android (Google) Code Review
parent 80b4539ccb aaeebdaff9
commit 4b3a00129e
2 changed files with 5 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
packages/SystemUI/src/com/android/systemui/statusbar/phone/CentralSurfacesImpl.java
View File

@@ -3323,7 +3323,7 @@ public class CentralSurfacesImpl implements CoreStartable, CentralSurfaces {
@Override @Override
public boolean onBackPressed() { public boolean onBackPressed() {
if (mStatusBarKeyguardViewManager.canHandleBackPressed()) { if (mStatusBarKeyguardViewManager.canHandleBackPressed()) {
mStatusBarKeyguardViewManager.onBackPressed(false /* unused */); mStatusBarKeyguardViewManager.onBackPressed();
return true; return true;
} }
if (mNotificationPanelViewController.isQsCustomizing()) { if (mNotificationPanelViewController.isQsCustomizing()) {

15
packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarKeyguardViewManager.java
View File

@@ -203,7 +203,7 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb
if (DEBUG) { if (DEBUG) {
Log.d(TAG, "onBackInvokedCallback() called, invoking onBackPressed()"); Log.d(TAG, "onBackInvokedCallback() called, invoking onBackPressed()");
} }
onBackPressed(false /* unused */); onBackPressed();
}; };
private boolean mIsBackCallbackRegistered = false; private boolean mIsBackCallbackRegistered = false;
@@ -1088,16 +1088,9 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb
/** /**
* Notifies this manager that the back button has been pressed. * Notifies this manager that the back button has been pressed.
*/ */
// TODO(b/244635782): This "accept boolean and ignore it, and always return false" was done public void onBackPressed() {
// to make it possible to check this in *and* allow merging to master,
// where ArcStatusBarKeyguardViewManager inherits this class, and its
// build will break if we change this interface.
// So, overall, while this function refactors the behavior of onBackPressed,
// (it now handles the back press, and no longer returns *whether* it did so)
// its interface is not changing right now (but will, in a follow-up CL).
public boolean onBackPressed(boolean ignored) {
if (!canHandleBackPressed()) { if (!canHandleBackPressed()) {
return false; return;
} }
mCentralSurfaces.endAffordanceLaunch(); mCentralSurfaces.endAffordanceLaunch();
@@ -1118,7 +1111,7 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb
mNotificationPanelViewController.expandWithoutQs(); mNotificationPanelViewController.expandWithoutQs();
} }
} }
return false; return;
} }
@Override @Override