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

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

Issue 2443613002: Rename classes related to top controls (Closed)
Patch Set: fix test Created 4 years, 2 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 2ea2af190ca905cdca9d4dbb2a60f0f63fba6bde..29df68e3d65cf04f8a7515b2f4b505e0ca778a95 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
@@ -28,10 +28,10 @@ class StackAnimationLandscape extends StackAnimation {
/**
* Only Constructor.
*/
- public StackAnimationLandscape(float width, float height, float heightMinusTopControls,
+ public StackAnimationLandscape(float width, float height, float heightMinusBrowserControls,
float borderFramePaddingTop, float borderFramePaddingTopOpaque,
float borderFramePaddingLeft) {
- super(width, height, heightMinusTopControls, borderFramePaddingTop,
+ super(width, height, heightMinusBrowserControls, borderFramePaddingTop,
borderFramePaddingTopOpaque, borderFramePaddingLeft);
}
@@ -53,8 +53,8 @@ class StackAnimationLandscape extends StackAnimation {
final float scrollOffset = StackTab.screenToScroll(i * spacing, warpSize);
addAnimation(set, tab.getLayoutTab(), MAX_CONTENT_HEIGHT,
- tab.getLayoutTab().getUnclampedOriginalContentHeight(), mHeightMinusTopControls,
- ENTER_STACK_ANIMATION_DURATION, 0);
+ tab.getLayoutTab().getUnclampedOriginalContentHeight(),
+ mHeightMinusBrowserControls, ENTER_STACK_ANIMATION_DURATION, 0);
if (i < focusIndex) {
addAnimation(set, tab, SCROLL_OFFSET, initialScrollOffset, scrollOffset,
ENTER_STACK_ANIMATION_DURATION, 0);
@@ -135,7 +135,7 @@ class StackAnimationLandscape extends StackAnimation {
tab.getLayoutTab().getMaxContentHeight(),
tab.getLayoutTab().getUnclampedOriginalContentHeight(),
TAB_FOCUSED_ANIMATION_DURATION, 0);
- tab.setYOutOfStack(mHeight - mHeightMinusTopControls - mBorderTopHeight);
+ tab.setYOutOfStack(mHeight - mHeightMinusBrowserControls - mBorderTopHeight);
if (layoutTab.shouldStall()) {
addAnimation(set, layoutTab, SATURATION, 1.0f, 0.0f,

Powered by Google App Engine
This is Rietveld 408576698