Fix an NPE in recapitalize
Bug: 8657736 Change-Id: I459d1c200826c1c394f8207475ecf60a4f356793main
parent
530fd15eff
commit
49007de565
|
@ -1970,9 +1970,12 @@ public final class LatinIME extends InputMethodService implements KeyboardAction
|
||||||
// If we have a recapitalize in progress, use it; otherwise, create a new one.
|
// If we have a recapitalize in progress, use it; otherwise, create a new one.
|
||||||
if (!mRecapitalizeStatus.isActive()
|
if (!mRecapitalizeStatus.isActive()
|
||||||
|| !mRecapitalizeStatus.isSetAt(mLastSelectionStart, mLastSelectionEnd)) {
|
|| !mRecapitalizeStatus.isSetAt(mLastSelectionStart, mLastSelectionEnd)) {
|
||||||
|
final CharSequence selectedText =
|
||||||
|
mConnection.getSelectedText(0 /* flags, 0 for no styles */);
|
||||||
|
if (TextUtils.isEmpty(selectedText)) return; // Race condition with the input connection
|
||||||
mRecapitalizeStatus.initialize(mLastSelectionStart, mLastSelectionEnd,
|
mRecapitalizeStatus.initialize(mLastSelectionStart, mLastSelectionEnd,
|
||||||
mConnection.getSelectedText(0 /* flags, 0 for no styles */).toString(),
|
selectedText.toString(), mSettings.getCurrentLocale(),
|
||||||
mSettings.getCurrentLocale(), mSettings.getWordSeparators());
|
mSettings.getWordSeparators());
|
||||||
// We trim leading and trailing whitespace.
|
// We trim leading and trailing whitespace.
|
||||||
mRecapitalizeStatus.trim();
|
mRecapitalizeStatus.trim();
|
||||||
// Trimming the object may have changed the length of the string, and we need to
|
// Trimming the object may have changed the length of the string, and we need to
|
||||||
|
|
Loading…
Reference in New Issue