AuthContainerView: fix rare SystemUI crash

This can happen if the fingerprint is validated twice in a row before
the view has time to disappear. Make the method synchronized to avoid
race conditions of any kind and add an additional try-catch as a
safeguard.

Change-Id: Ic2b76be1cc0f5217351e39ec24159ca23043ab4f
Signed-off-by: Pranav Vashi <[email protected]>

Author: xdevs23

packages/SystemUI/src/com/android/systemui/biometrics/AuthContainerView.java

@@ -596,7 +596,7 @@ private void sendPendingCallbackIfNotNull() {
         }
     }
 
-    private void removeWindowIfAttached(boolean sendReason) {
+    private synchronized void removeWindowIfAttached(boolean sendReason) {
         if (sendReason) {
             sendPendingCallbackIfNotNull();
         }
@@ -607,7 +607,12 @@ private void removeWindowIfAttached(boolean sendReason) {
         }
         Log.d(TAG, "Removing container, mSysUiSessionId: " + mConfig.mSysUiSessionId);
         mContainerState = STATE_GONE;
-        mWindowManager.removeView(this);
+        try {
+            mWindowManager.removeView(this);
+        } catch (IllegalArgumentException e) {
+            // Looks like the view is already gone??
+            // Whatever, just ignore it then.
+        }
     }
 
     @VisibleForTesting