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

Merge "Catch WindowManager exceptions so they don't crash the system." into oc-dev am: 301fc823f3

Browse Source 
am: 9a1b6b0eb2

Change-Id: Ifbda0a75a79e644763e83baf86a7823310a56f5b
This commit is contained in:
Felipe Leme
2017-06-06 23:53:16 +00:00
committed by android-build-merger
parent 38e49013e0 9a1b6b0eb2
commit 04e794c12b
1 changed files with 18 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
services/autofill/java/com/android/server/autofill/ui/FillUi.java
View File

@@ -398,6 +398,12 @@ final class FillUi {
} catch (WindowManager.BadTokenException e) {
if (sDebug) Slog.d(TAG, "Filed with with token " + params.token + " gone.");
mCallback.onDestroy();
} catch (IllegalStateException e) {
// WM throws an ISE if mContentView was added twice; this should never happen -
// since show() and hide() are always called in the UIThread - but when it does,
// it should not crash the system.
Slog.e(TAG, "Exception showing window " + params, e);
mCallback.onDestroy();
}
}
@@ -405,10 +411,18 @@ final class FillUi {
* Hides the window.
*/
void hide() {
if (mShowing) {
mContentView.setOnTouchListener(null);
mWm.removeView(mContentView);
mShowing = false;
try {
if (mShowing) {
mContentView.setOnTouchListener(null);
mWm.removeView(mContentView);
mShowing = false;
}
} catch (IllegalStateException e) {
// WM might thrown an ISE when removing the mContentView; this should never
// happen - since show() and hide() are always called in the UIThread - but if it
// does, it should not crash the system.
Slog.e(TAG, "Exception hiding window ", e);
mCallback.onDestroy();
}
}