lock out clicks while switching.

Bug: 7416992
Change-Id: Ic0fcf809c4d9bb3515dd1168d427f8b22d5f76b6
Proto-Id: I0ea871fb012feb2e17249e37a9253861f2ef5011

Author: cwren

policy/src/com/android/internal/policy/impl/keyguard/KeyguardMultiUserAvatar.java

@@ -207,9 +207,11 @@ public void onAnimationEnd(Animator animation) {
 
     @Override
     public void setPressed(boolean pressed) {
-        super.setPressed(pressed);
-        mFramed.setPressed(pressed);
-        mUserImage.invalidate();
+        if (!pressed || isClickable()) {
+            super.setPressed(pressed);
+            mFramed.setPressed(pressed);
+            mUserImage.invalidate();
+        }
     }
 
     public UserInfo getUserInfo() {

policy/src/com/android/internal/policy/impl/keyguard/KeyguardMultiUserSelectorView.java

@@ -110,36 +110,50 @@ public boolean onInterceptTouchEvent(MotionEvent event) {
         return false;
     }
 
+    private void setAllClickable(boolean clickable)
+    {
+        for(int i = 0; i < mUsersGrid.getChildCount(); i++) {
+            View v = mUsersGrid.getChildAt(i);
+            v.setClickable(clickable);
+            v.setPressed(false);
+        }
+    }
+
     @Override
     public void onClick(View v) {
         if (!(v instanceof KeyguardMultiUserAvatar)) return;
         final KeyguardMultiUserAvatar avatar = (KeyguardMultiUserAvatar) v;
-        if (mActiveUserAvatar == avatar) {
-            // If they click the currently active user, show the unlock hint
-            mCallback.showUnlockHint();
-            return;
-        } else {
-            // Reset the previously active user to appear inactive
-            mCallback.hideSecurityView(FADE_OUT_ANIMATION_DURATION);
-            mActiveUserAvatar.setActive(false, true, new Runnable() {
-                @Override
-                public void run() {
-                    mActiveUserAvatar = avatar;
-                    mActiveUserAvatar.setActive(true, true, new Runnable() {
-                        @Override
-                        public void run() {
-                            if (this.getClass().getName().contains("internal")) {
-                                try {
-                                    ActivityManagerNative.getDefault()
-                                            .switchUser(avatar.getUserInfo().id);
-                                } catch (RemoteException re) {
-                                    Log.e(TAG, "Couldn't switch user " + re);
+        if (avatar.isClickable()) { // catch race conditions
+            if (mActiveUserAvatar == avatar) {
+                // If they click the currently active user, show the unlock hint
+                mCallback.showUnlockHint();
+                return;
+            } else {
+                // Reset the previously active user to appear inactive
+                mCallback.hideSecurityView(FADE_OUT_ANIMATION_DURATION);
+                setAllClickable(false);
+                mActiveUserAvatar.setActive(false, true, new Runnable() {
+                    @Override
+                    public void run() {
+                        mActiveUserAvatar = avatar;
+                        mActiveUserAvatar.setActive(true, true, new Runnable() {
+                            @Override
+                            public void run() {
+                                if (this.getClass().getName().contains("internal")) {
+                                    try {
+                                        ActivityManagerNative.getDefault()
+                                                .switchUser(avatar.getUserInfo().id);
+                                    } catch (RemoteException re) {
+                                        Log.e(TAG, "Couldn't switch user " + re);
+                                    }
+                                } else {
+                                    setAllClickable(true);
                                 }
                             }
-                        }
-                    });
-                }
-            });
+                        });
+                    }
+                });
+            }
         }
     }
 }