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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheet.java

Issue 2829333002: [Home] Fix invalid state crash (Closed)
Patch Set: revert cancel changes Created 3 years, 8 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheet.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheet.java b/chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheet.java
index 97b0e44271340b5a7b3db3bac1c4d4d713fb4d54..8f4e4426c7cc8fe73e065d5eb313ad04adb01a33 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheet.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheet.java
@@ -812,7 +812,7 @@ public class BottomSheet
* Creates the sheet's animation to a target state.
* @param targetState The target state.
*/
- private void createSettleAnimation(@SheetState int targetState) {
+ private void createSettleAnimation(@SheetState final int targetState) {
mTargetState = targetState;
mSettleAnimator = ValueAnimator.ofFloat(
getSheetOffsetFromBottom(), getSheetHeightForState(targetState));
@@ -824,7 +824,7 @@ public class BottomSheet
@Override
public void onAnimationEnd(Animator animator) {
mSettleAnimator = null;
- setInternalCurrentState(mTargetState);
+ setInternalCurrentState(targetState);
mTargetState = SHEET_STATE_NONE;
}
});
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698