Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(722)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/phone/stack/StackAnimationLandscape.java

Issue 2761273004: [Home] Improve exit tab switcher animation (Closed)
Patch Set: rebase Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/phone/stack/StackAnimationLandscape.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/phone/stack/StackAnimationLandscape.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/phone/stack/StackAnimationLandscape.java
index 5beb7cf80cf0d71fa322979b109ada4e01d0aa9f..5b37de350b299d39bae6570301ea8b5ee675bba0 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/phone/stack/StackAnimationLandscape.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/phone/stack/StackAnimationLandscape.java
@@ -21,6 +21,7 @@ import static org.chromium.chrome.browser.compositor.layouts.phone.stack.StackTa
import org.chromium.chrome.browser.compositor.layouts.ChromeAnimation;
import org.chromium.chrome.browser.compositor.layouts.ChromeAnimation.Animatable;
import org.chromium.chrome.browser.compositor.layouts.components.LayoutTab;
+import org.chromium.chrome.browser.util.FeatureUtilities;
import org.chromium.chrome.browser.util.MathUtils;
import org.chromium.ui.base.LocalizationUtils;
@@ -28,10 +29,10 @@ class StackAnimationLandscape extends StackAnimation {
/**
* Only Constructor.
*/
- public StackAnimationLandscape(float width, float height, float heightMinusBrowserControls,
- float borderFramePaddingTop, float borderFramePaddingTopOpaque,
- float borderFramePaddingLeft) {
- super(width, height, heightMinusBrowserControls, borderFramePaddingTop,
+ public StackAnimationLandscape(Stack stack, float width, float height,
+ float heightMinusBrowserControls, float borderFramePaddingTop,
+ float borderFramePaddingTopOpaque, float borderFramePaddingLeft) {
+ super(stack, width, height, heightMinusBrowserControls, borderFramePaddingTop,
borderFramePaddingTopOpaque, borderFramePaddingLeft);
}
@@ -54,7 +55,7 @@ class StackAnimationLandscape extends StackAnimation {
addAnimation(set, tab.getLayoutTab(), MAX_CONTENT_HEIGHT,
tab.getLayoutTab().getUnclampedOriginalContentHeight(),
- mHeightMinusBrowserControls, ENTER_STACK_ANIMATION_DURATION, 0);
+ mStack.getMaxTabHeight(), ENTER_STACK_ANIMATION_DURATION, 0);
if (i < focusIndex) {
addAnimation(set, tab, SCROLL_OFFSET, initialScrollOffset, scrollOffset,
ENTER_STACK_ANIMATION_DURATION, 0);
@@ -128,8 +129,11 @@ class StackAnimationLandscape extends StackAnimation {
set, tab, SCALE, tab.getScale(), 1.0f, TAB_FOCUSED_ANIMATION_DURATION, 0);
addAnimation(set, tab, X_IN_STACK_INFLUENCE, tab.getXInStackInfluence(), 0.0f,
TAB_FOCUSED_ANIMATION_DURATION, 0);
+ int tabYInfluenceDuration = FeatureUtilities.isChromeHomeEnabled()
+ ? TAB_FOCUSED_ANIMATION_DURATION
+ : TAB_FOCUSED_Y_STACK_DURATION;
addAnimation(set, tab, Y_IN_STACK_INFLUENCE, tab.getYInStackInfluence(), 0.0f,
- TAB_FOCUSED_Y_STACK_DURATION, 0);
+ tabYInfluenceDuration, 0);
addAnimation(set, tab.getLayoutTab(), MAX_CONTENT_HEIGHT,
tab.getLayoutTab().getMaxContentHeight(),

Powered by Google App Engine
This is Rietveld 408576698