diff options
author | Hyunyoung Song <hyunyoungs@google.com> | 2016-07-27 17:08:38 -0700 |
---|---|---|
committer | Hyunyoung Song <hyunyoungs@google.com> | 2016-07-27 17:08:38 -0700 |
commit | dd60ce41b31028a309448df17dcd5b7796af6fef (patch) | |
tree | ad9a0d96f2426bd93bd4b1a5ce36f3412e0d2e43 /src | |
parent | 8fcdee05bb87fee8085048a27c954626d59dbb9b (diff) | |
download | android_packages_apps_Trebuchet-dd60ce41b31028a309448df17dcd5b7796af6fef.tar.gz android_packages_apps_Trebuchet-dd60ce41b31028a309448df17dcd5b7796af6fef.tar.bz2 android_packages_apps_Trebuchet-dd60ce41b31028a309448df17dcd5b7796af6fef.zip |
Remove two redundant layout pass in AllAppsTransitionController.
Change-Id: I48d043820db962c58e97a09d7888303c1d5bf4ad
Diffstat (limited to 'src')
3 files changed, 16 insertions, 18 deletions
diff --git a/src/com/android/launcher3/Launcher.java b/src/com/android/launcher3/Launcher.java index deac73be8..f22219f78 100644 --- a/src/com/android/launcher3/Launcher.java +++ b/src/com/android/launcher3/Launcher.java @@ -997,8 +997,8 @@ public class Launcher extends Activity // Don't update the predicted apps if the user is returning to launcher in the apps // view after launching an app, as they may be depending on the UI to be static to // switch to another app, otherwise, if it was - showAppsView(false /* animated */, false /* resetListToTop */, - !launchedFromApp /* updatePredictedApps */, false /* focusSearchBar */); + showAppsView(false /* animated */, !launchedFromApp /* updatePredictedApps */, + false /* focusSearchBar */); } else if (mOnResumeState == State.WIDGETS) { showWidgetsView(false, false); } @@ -2601,8 +2601,8 @@ public class Launcher extends Activity if (!isAppsViewVisible()) { getUserEventDispatcher().logActionOnControl(LauncherLogProto.Action.TAP, LauncherLogProto.ALL_APPS_BUTTON); - showAppsView(true /* animated */, false /* resetListToTop */, - true /* updatePredictedApps */, false /* focusSearchBar */); + showAppsView(true /* animated */, true /* updatePredictedApps */, + false /* focusSearchBar */); } } @@ -2611,7 +2611,7 @@ public class Launcher extends Activity if (!isAppsViewVisible()) { getUserEventDispatcher().logActionOnControl(LauncherLogProto.Action.LONGPRESS, LauncherLogProto.ALL_APPS_BUTTON); - showAppsView(true /* animated */, false /* resetListToTop */, + showAppsView(true /* animated */, true /* updatePredictedApps */, true /* focusSearchBar */); } } @@ -3355,12 +3355,9 @@ public class Launcher extends Activity /** * Shows the apps view. */ - public void showAppsView(boolean animated, boolean resetListToTop, boolean updatePredictedApps, + public void showAppsView(boolean animated, boolean updatePredictedApps, boolean focusSearchBar) { markAppsViewShown(); - if (resetListToTop) { - mAppsView.scrollToTop(); - } if (updatePredictedApps) { tryAndUpdatePredictedApps(); } @@ -3487,7 +3484,7 @@ public class Launcher extends Activity void exitSpringLoadedDragMode() { if (mState == State.APPS_SPRING_LOADED) { - showAppsView(true /* animated */, false /* resetListToTop */, + showAppsView(true /* animated */, false /* updatePredictedApps */, false /* focusSearchBar */); } else if (mState == State.WIDGETS_SPRING_LOADED) { showWidgetsView(true, false); diff --git a/src/com/android/launcher3/allapps/AllAppsContainerView.java b/src/com/android/launcher3/allapps/AllAppsContainerView.java index 428f78401..d8601893e 100644 --- a/src/com/android/launcher3/allapps/AllAppsContainerView.java +++ b/src/com/android/launcher3/allapps/AllAppsContainerView.java @@ -348,9 +348,10 @@ public class AllAppsContainerView extends BaseContainerView implements DragSourc mAppsRecyclerView.preMeasureViews(mAdapter); mAdapter.setIconFocusListener(focusedItemDecorator.getFocusListener()); - // TODO(hyunyoungs): clean up setting the content and the reveal view. if (FeatureFlags.LAUNCHER3_ALL_APPS_PULL_UP) { getRevealView().setVisibility(View.VISIBLE); + getContentView().setVisibility(View.VISIBLE); + getContentView().setBackground(null); } } diff --git a/src/com/android/launcher3/allapps/AllAppsTransitionController.java b/src/com/android/launcher3/allapps/AllAppsTransitionController.java index df209b56c..0e9cac856 100644 --- a/src/com/android/launcher3/allapps/AllAppsTransitionController.java +++ b/src/com/android/launcher3/allapps/AllAppsTransitionController.java @@ -12,7 +12,6 @@ import android.view.MotionEvent; import android.view.View; import android.view.animation.AccelerateInterpolator; import android.view.animation.AnimationUtils; -import android.view.animation.DecelerateInterpolator; import android.view.animation.Interpolator; import com.android.launcher3.DeviceProfile; @@ -225,7 +224,9 @@ public class AllAppsTransitionController implements TouchController, VerticalPul LauncherLogProto.Action.UP, LauncherLogProto.HOTSEAT); } - mLauncher.showAppsView(true, true, false, false); + mLauncher.showAppsView(true /* animated */, + false /* updatePredictedApps */, + false /* focusSearchBar */); } else { calculateDuration(velocity, Math.abs(mShiftRange - mAppsView.getTranslationY())); mLauncher.showWorkspace(true); @@ -243,7 +244,9 @@ public class AllAppsTransitionController implements TouchController, VerticalPul LauncherLogProto.Action.UP, LauncherLogProto.HOTSEAT); } - mLauncher.showAppsView(true, true, false, false); + mLauncher.showAppsView(true, /* animated */ + false /* updatePredictedApps */, + false /* focusSearchBar */); } } } @@ -260,15 +263,11 @@ public class AllAppsTransitionController implements TouchController, VerticalPul // Initialize values that should not change until #onDragEnd mStatusBarHeight = mLauncher.getDragLayer().getInsets().top; mHotseat.setVisibility(View.VISIBLE); - mHotseat.bringToFront(); if (!mLauncher.isAllAppsVisible()) { mLauncher.tryAndUpdatePredictedApps(); mHotseatBackgroundColor = mHotseat.getBackgroundDrawableColor(); mHotseat.setBackgroundTransparent(true /* transparent */); mAppsView.setVisibility(View.VISIBLE); - mAppsView.getContentView().setVisibility(View.VISIBLE); - mAppsView.getContentView().setBackground(null); - mAppsView.getRevealView().setVisibility(View.VISIBLE); mAppsView.setRevealDrawableColor(mHotseatBackgroundColor); } } @@ -525,6 +524,7 @@ public class AllAppsTransitionController implements TouchController, VerticalPul mCaretAnimator.setDuration(mCaretAnimationDuration); mCaretAnimator.setInterpolator(mCaretInterpolator); mHotseat.addOnLayoutChangeListener(this); + mHotseat.bringToFront(); } @Override |