diff options
| author | 2017-09-20 16:13:21 -0700 | |
|---|---|---|
| committer | 2017-09-20 16:14:05 -0700 | |
| commit | 620a6ceddeee8154f9a279f2d045d08e9f55658b (patch) | |
| tree | 8c67c17551a8142ac93c198b7d70900cdd9e05a2 | |
| parent | b1b99b02993c0532fe910ee5bb38b01b383b9d43 (diff) | |
Only call negative listener when SaveUI is dismissed by tapping button.
Test: cts-tradefed run commandAndExit cts-dev -m CtsAutoFillServiceTestCases
Fixes: 66235266
Change-Id: Ieb3085b64e96ccbd3dd97fca58fcb9d4930c82fa
| -rw-r--r-- | services/autofill/java/com/android/server/autofill/ui/SaveUi.java | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/services/autofill/java/com/android/server/autofill/ui/SaveUi.java b/services/autofill/java/com/android/server/autofill/ui/SaveUi.java index 32f4d69fc3e3..63f1483b237f 100644 --- a/services/autofill/java/com/android/server/autofill/ui/SaveUi.java +++ b/services/autofill/java/com/android/server/autofill/ui/SaveUi.java @@ -263,9 +263,7 @@ final class SaveUi { } else { noButton.setText(R.string.autofill_save_no); } - final View.OnClickListener cancelListener = - (v) -> mListener.onCancel(info.getNegativeActionListener()); - noButton.setOnClickListener(cancelListener); + noButton.setOnClickListener((v) -> mListener.onCancel(info.getNegativeActionListener())); final View yesButton = view.findViewById(R.id.autofill_save_yes); yesButton.setOnClickListener((v) -> mListener.onSave()); @@ -273,9 +271,6 @@ final class SaveUi { mDialog = new Dialog(context, R.style.Theme_DeviceDefault_Light_Panel); mDialog.setContentView(view); - // Dialog can be dismissed when touched outside. - mDialog.setOnDismissListener((d) -> mListener.onCancel(info.getNegativeActionListener())); - final Window window = mDialog.getWindow(); window.setType(WindowManager.LayoutParams.TYPE_APPLICATION_OVERLAY); window.addFlags(WindowManager.LayoutParams.FLAG_NOT_FOCUSABLE |