diff options
author | Tom Taylor <tomtaylor@google.com> | 2013-01-30 17:11:51 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2013-01-30 17:11:51 +0000 |
commit | 2549ff535f2f79af2ee22ab9697ea726d9fe3671 (patch) | |
tree | e39382ec5674f3a08172b4b7fbd3e897c51fa5f0 | |
parent | 888ebca785d828646c78e7e7fc048c83d49e2109 (diff) | |
parent | 56d79f319079895a186a85d54e6afae83cbb5f81 (diff) | |
download | Mms-2549ff535f2f79af2ee22ab9697ea726d9fe3671.tar.gz |
Merge "Fix NullPointerException in RecipientsEditor."
-rw-r--r-- | src/com/android/mms/ui/RecipientsEditor.java | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/src/com/android/mms/ui/RecipientsEditor.java b/src/com/android/mms/ui/RecipientsEditor.java index 0baeff88..ae1ae34d 100644 --- a/src/com/android/mms/ui/RecipientsEditor.java +++ b/src/com/android/mms/ui/RecipientsEditor.java @@ -280,6 +280,13 @@ public class RecipientsEditor extends RecipientEditTextView { } private int pointToPosition(int x, int y) { + // Check layout before getExtendedPaddingTop(). + // mLayout is used in getExtendedPaddingTop(). + Layout layout = getLayout(); + if (layout == null) { + return -1; + } + x -= getCompoundPaddingLeft(); y -= getExtendedPaddingTop(); @@ -287,11 +294,6 @@ public class RecipientsEditor extends RecipientEditTextView { x += getScrollX(); y += getScrollY(); - Layout layout = getLayout(); - if (layout == null) { - return -1; - } - int line = layout.getLineForVertical(y); int off = layout.getOffsetForHorizontal(line, x); |