Fixing issue where you can reorder/delete a non-widget page. (Bug 7493984)

- Laying out the drop target without padding from the multiuser picker

Change-Id: Icd3528a13f84a789341a841645bfde4e51fd1747

Author: Winson Chung

core/res/res/layout-land/keyguard_host_view.xml

@@ -37,7 +37,8 @@
             android:id="@+id/keyguard_widget_pager_delete_target"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:layout_gravity="top|center_horizontal" />
+            android:layout_gravity="top|center_horizontal"
+            androidprv:layout_childType="pageDeleteDropTarget" />
 
         <include layout="@layout/keyguard_widget_pager"
             android:id="@+id/app_widget_container"

core/res/res/layout-port/keyguard_host_view.xml

@@ -35,7 +35,8 @@
 
         <FrameLayout
             android:layout_width="match_parent"
-            android:layout_height="wrap_content">
+            android:layout_height="wrap_content"
+            androidprv:layout_childType="pageDeleteDropTarget">
             <include layout="@layout/keyguard_widget_remove_drop_target"
                 android:id="@+id/keyguard_widget_pager_delete_target"
                 android:layout_width="wrap_content"

core/res/res/layout-sw600dp-port/keyguard_host_view.xml

@@ -38,7 +38,8 @@
             android:id="@+id/keyguard_widget_pager_delete_target"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:layout_gravity="top|center_horizontal" />
+            android:layout_gravity="top|center_horizontal"
+            androidprv:layout_childType="pageDeleteDropTarget" />
 
         <include layout="@layout/keyguard_widget_pager"
             android:id="@+id/app_widget_container"

core/res/res/layout/keyguard_widget_remove_drop_target.xml

@@ -19,9 +19,9 @@
 <TextView
     xmlns:android="http://schemas.android.com/apk/res/android"
     android:gravity="center"
-    android:padding="20dp"
-    android:paddingLeft="40dp"
-    android:paddingRight="40dp"
+    android:padding="30dp"
+    android:paddingLeft="60dp"
+    android:paddingRight="60dp"
     android:drawableLeft="@drawable/kg_widget_delete_drop_target"
     android:drawablePadding="4dp"
     android:text="@string/kg_reordering_delete_drop_target_text"

core/res/res/values/attrs.xml

@@ -5815,6 +5815,8 @@
         <!-- This is a handle that is used for expanding the
              security challenge container when it is collapsed. -->
         <enum name="expandChallengeHandle" value="6" />
+        <!-- Delete drop target.  This will be the drop target to delete pages. -->
+        <enum name="pageDeleteDropTarget" value="7" />
     </attr>
 
     <declare-styleable name="SlidingChallengeLayout_Layout">

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

@@ -507,14 +507,17 @@ public boolean isWidgetPage(int pageIndex) {
         return false;
     }
 
+    /**
+     * Returns the bounded set of pages that are re-orderable.  The range is fully inclusive.
+     */
     @Override
     void boundByReorderablePages(boolean isReordering, int[] range) {
         if (isReordering) {
             // Remove non-widget pages from the range
-            while (range[1] > range[0] && !isWidgetPage(range[1])) {
+            while (range[1] >= range[0] && !isWidgetPage(range[1])) {
                 range[1]--;
             }
-            while (range[0] < range[1] && !isWidgetPage(range[0])) {
+            while (range[0] <= range[1] && !isWidgetPage(range[0])) {
                 range[0]++;
             }
         }

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

@@ -48,6 +48,7 @@ public class MultiPaneChallengeLayout extends ViewGroup implements ChallengeLayo
     private OnBouncerStateChangedListener mBouncerListener;
 
     private final Rect mTempRect = new Rect();
+    private final Rect mZeroPadding = new Rect();
 
     private final DisplayMetrics mDisplayMetrics;
 
@@ -187,6 +188,8 @@ private int getVirtualHeight(LayoutParams lp, int height, int heightUsed) {
             // on the window. We want to avoid resizing widgets when possible as it can
             // be ugly/expensive. This lets us simply clip them instead.
             return virtualHeight - heightUsed;
+        } else if (lp.childType == LayoutParams.CHILD_TYPE_PAGE_DELETE_DROP_TARGET) {
+            return height;
         }
         return Math.min(virtualHeight - heightUsed, height);
     }
@@ -330,13 +333,17 @@ protected void onLayout(boolean changed, int l, int t, int r, int b) {
         final int count = getChildCount();
         for (int i = 0; i < count; i++) {
             final View child = getChildAt(i);
+            LayoutParams lp = (LayoutParams) child.getLayoutParams();
 
             // We did the user switcher above if we have one.
             if (child == mUserSwitcherView || child.getVisibility() == GONE) continue;
 
             if (child == mScrimView) {
                 child.layout(0, 0, width, height);
                 continue;
+            } else if (lp.childType == LayoutParams.CHILD_TYPE_PAGE_DELETE_DROP_TARGET) {
+                layoutWithGravity(width, height, child, mZeroPadding, false);
+                continue;
             }
 
             layoutWithGravity(width, height, child, padding, false);
@@ -467,6 +474,7 @@ public static class LayoutParams extends MarginLayoutParams {
         public static final int CHILD_TYPE_CHALLENGE = 2;
         public static final int CHILD_TYPE_USER_SWITCHER = 3;
         public static final int CHILD_TYPE_SCRIM = 4;
+        public static final int CHILD_TYPE_PAGE_DELETE_DROP_TARGET = 7;
 
         public int gravity = Gravity.NO_GRAVITY;
 

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

@@ -233,6 +233,7 @@ public abstract class PagedView extends ViewGroup implements ViewGroup.OnHierarc
     private Matrix mTmpInvMatrix = new Matrix();
     private float[] mTmpPoint = new float[2];
     private Rect mTmpRect = new Rect();
+    private Rect mAltTmpRect = new Rect();
 
     // Fling to delete
     private int FLING_TO_DELETE_FADE_OUT_DURATION = 350;
@@ -2455,7 +2456,13 @@ public void onAnimationEnd(Animator animation) {
     /* Drag to delete */
     private boolean isHoveringOverDeleteDropTarget(int x, int y) {
         if (mDeleteDropTarget != null) {
+            mAltTmpRect.set(0, 0, 0, 0);
+            View parent = (View) mDeleteDropTarget.getParent();
+            if (parent != null) {
+                parent.getGlobalVisibleRect(mAltTmpRect);
+            }
             mDeleteDropTarget.getGlobalVisibleRect(mTmpRect);
+            mTmpRect.offset(-mAltTmpRect.left, -mAltTmpRect.top);
             return mTmpRect.contains(x, y);
         }
         return false;