diff options
| author | Brad Fitzpatrick <bradfitz@android.com> | 2010-10-19 12:19:41 -0700 |
|---|---|---|
| committer | Brad Fitzpatrick <bradfitz@android.com> | 2010-10-19 12:19:41 -0700 |
| commit | 44d2c69ad30878521c6eff66a83893099b3d12ee (patch) | |
| tree | 6b5ff8bb7085241e07ceb8885696be826162d3ad | |
| parent | 987e8d7f9748018d3fee5778645b267a9ee06ff6 (diff) | |
| download | frameworks_base-44d2c69ad30878521c6eff66a83893099b3d12ee.zip frameworks_base-44d2c69ad30878521c6eff66a83893099b3d12ee.tar.gz frameworks_base-44d2c69ad30878521c6eff66a83893099b3d12ee.tar.bz2 | |
Avoid NPE in WebTextView.lineUpScroll -> Touch.getMaxScrollX
Bug: 3110967
Change-Id: I156070956528c8c7d1eb6db5d2e8bb1fe119178d
| -rw-r--r-- | core/java/android/webkit/WebTextView.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/core/java/android/webkit/WebTextView.java b/core/java/android/webkit/WebTextView.java index f7afdb9..4f192b3 100644 --- a/core/java/android/webkit/WebTextView.java +++ b/core/java/android/webkit/WebTextView.java @@ -28,6 +28,7 @@ import android.graphics.Rect; import android.graphics.drawable.Drawable; import android.text.Editable; import android.text.InputFilter; +import android.text.Layout; import android.text.Selection; import android.text.Spannable; import android.text.TextPaint; @@ -292,8 +293,9 @@ import java.util.ArrayList; * Ensure that the underlying textfield is lined up with the WebTextView. */ private void lineUpScroll() { - if (mWebView != null) { - float maxScrollX = Touch.getMaxScrollX(this, getLayout(), mScrollY); + Layout layout = getLayout(); + if (mWebView != null && layout != null) { + float maxScrollX = Touch.getMaxScrollX(this, layout, mScrollY); if (DebugFlags.WEB_TEXT_VIEW) { Log.v(LOGTAG, "onTouchEvent x=" + mScrollX + " y=" + mScrollY + " maxX=" + maxScrollX); |
