Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/permissions/PermissionDialogController.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/permissions/PermissionDialogController.java b/chrome/android/java/src/org/chromium/chrome/browser/permissions/PermissionDialogController.java |
| index d2f5e5a01a55c916d717327bdad0371d874c7b1c..194e8e06a18854d9f80b2c9507b9f46a1abb2320 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/permissions/PermissionDialogController.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/permissions/PermissionDialogController.java |
| @@ -191,6 +191,11 @@ public class PermissionDialogController implements AndroidPermissionRequester.Re |
| mDialog.setOnDismissListener(new DialogInterface.OnDismissListener() { |
| @Override |
| public void onDismiss(DialogInterface dialog) { |
| + // For some reason this is ocassionally null. See crbug.com/708562. |
| + if (mDialogDelegate == null) { |
| + scheduleDisplay(); |
|
Timothy Loh
2017/05/10 07:38:37
Was there supposed to be a return here?
dominickn
2017/05/10 08:42:15
Hmmmmmmm.... yes. I think so. Oops.
|
| + } |
| + |
| mDialog = null; |
| if (mDecision == ACCEPTED) { |
| // Request Android permissions if necessary. This will call back into either |