am 14adfe4: Address comments from merged "Fix remaining jank in sec..."

* commit '14adfe425e67c9ec12b0841a62c5c03ce00838ea':
  Address comments from merged "Fix remaining jank in sec..."

Author: John Spurlock

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

@@ -35,7 +35,7 @@
 import android.util.Log;
 import android.view.WindowManager;
 
-import com.android.internal.policy.impl.keyguard.KeyguardSecurityCallback.OnDismissAction;
+import com.android.internal.policy.impl.keyguard.KeyguardHostView.OnDismissAction;
 import com.android.internal.widget.LockPatternUtils;
 
 import java.util.List;

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

@@ -53,7 +53,6 @@
 import android.widget.RemoteViews.OnClickHandler;
 
 import com.android.internal.R;
-import com.android.internal.policy.impl.keyguard.KeyguardSecurityCallback.OnDismissAction;
 import com.android.internal.policy.impl.keyguard.KeyguardSecurityModel.SecurityMode;
 import com.android.internal.widget.LockPatternUtils;
 
@@ -115,6 +114,11 @@ public class KeyguardHostView extends KeyguardViewBase {
         void userActivity();
     }
 
+    /*package*/ interface OnDismissAction {
+        /* returns true if the dismiss should be deferred */
+        boolean onDismiss();
+    }
+
     public KeyguardHostView(Context context) {
         this(context, null);
     }

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

@@ -15,13 +15,9 @@
  */
 package com.android.internal.policy.impl.keyguard;
 
-public interface KeyguardSecurityCallback {
-
-    /*package*/ interface OnDismissAction {
+import com.android.internal.policy.impl.keyguard.KeyguardHostView.OnDismissAction;
 
-        /* returns true if the dismiss should be deferred */
-        boolean onDismiss();
-    }
+public interface KeyguardSecurityCallback {
 
     /**
      * Dismiss the given security screen.

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

@@ -567,6 +567,8 @@ public void onScreenTurnedOff(int why) {
             mScreenOn = false;
             if (DEBUG) Log.d(TAG, "onScreenTurnedOff(" + why + ")");
 
+            mKeyguardDonePending = false;
+
             // Lock immediately based on setting if secure (user has a pin/pattern/password).
             // This also "locks" the device when not secure to provide easy access to the
             // camera while preventing unwanted input.
@@ -1226,6 +1228,7 @@ private void handleShow(Bundle options) {
 
             mKeyguardViewManager.show(options);
             mShowing = true;
+            mKeyguardDonePending = false;
             updateActivityLockScreenState();
             adjustStatusBarLocked();
             userActivity();
@@ -1261,6 +1264,7 @@ private void handleHide() {
 
             mKeyguardViewManager.hide();
             mShowing = false;
+            mKeyguardDonePending = false;
             updateActivityLockScreenState();
             adjustStatusBarLocked();
         }