diff options
| author | Fabrice Di Meglio <fdimeglio@google.com> | 2013-06-18 21:54:49 +0000 |
|---|---|---|
| committer | Android (Google) Code Review <android-gerrit@google.com> | 2013-06-18 21:54:49 +0000 |
| commit | 012416fdbb279f291b43e9d6bf565750752e6a41 (patch) | |
| tree | a72ea4399816d7a7e311a06e6d4d6c0a241753f2 /core/java/android/widget/ScrollView.java | |
| parent | 2f0396ad568465f40168275beaaca1848535b732 (diff) | |
| parent | d6d543977da14adb3ef9322a0b7ec3fdd027a55e (diff) | |
Merge "Fix bug #9463581 NPE observed while launching News&Weather app from all apps tray"
Diffstat (limited to 'core/java/android/widget/ScrollView.java')
| -rw-r--r-- | core/java/android/widget/ScrollView.java | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/core/java/android/widget/ScrollView.java b/core/java/android/widget/ScrollView.java index b32cfbc04604..3d361f11d7b1 100644 --- a/core/java/android/widget/ScrollView.java +++ b/core/java/android/widget/ScrollView.java @@ -1473,16 +1473,16 @@ public class ScrollView extends FrameLayout { } mChildToScrollTo = null; - // There is only one child - final View child = getChildAt(0); - final int childHeight = child.getMeasuredHeight(); if (!hasLayout()) { - final int scrollRange = Math.max(0, - childHeight - (b - t - mPaddingBottom - mPaddingTop)); if (mSavedState != null) { mScrollY = mSavedState.scrollPosition; mSavedState = null; } // mScrollY default value is "0" + + final int childHeight = (getChildCount() > 0) ? getChildAt(0).getMeasuredHeight() : 0; + final int scrollRange = Math.max(0, + childHeight - (b - t - mPaddingBottom - mPaddingTop)); + // Don't forget to clamp if (mScrollY > scrollRange) { mScrollY = scrollRange; |
