[IL129] Change some confusing method names.
Bug: 8636060 Change-Id: I2f55fc23d7e05b63582a3f0660b58635eae5ec59main
parent
543fea98a4
commit
a777c1c460
|
@ -156,7 +156,7 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
|
||||||
private static final int MSG_SHOW_GESTURE_PREVIEW_AND_SUGGESTION_STRIP = 3;
|
private static final int MSG_SHOW_GESTURE_PREVIEW_AND_SUGGESTION_STRIP = 3;
|
||||||
private static final int MSG_RESUME_SUGGESTIONS = 4;
|
private static final int MSG_RESUME_SUGGESTIONS = 4;
|
||||||
private static final int MSG_REOPEN_DICTIONARIES = 5;
|
private static final int MSG_REOPEN_DICTIONARIES = 5;
|
||||||
private static final int MSG_ON_END_BATCH_INPUT = 6;
|
private static final int MSG_UPDATE_TAIL_BATCH_INPUT_COMPLETED = 6;
|
||||||
private static final int MSG_RESET_CACHES = 7;
|
private static final int MSG_RESET_CACHES = 7;
|
||||||
// Update this when adding new messages
|
// Update this when adding new messages
|
||||||
private static final int MSG_LAST = MSG_RESET_CACHES;
|
private static final int MSG_LAST = MSG_RESET_CACHES;
|
||||||
|
@ -220,8 +220,9 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
|
||||||
// get any suggestions. Wait one frame.
|
// get any suggestions. Wait one frame.
|
||||||
postUpdateSuggestionStrip();
|
postUpdateSuggestionStrip();
|
||||||
break;
|
break;
|
||||||
case MSG_ON_END_BATCH_INPUT:
|
case MSG_UPDATE_TAIL_BATCH_INPUT_COMPLETED:
|
||||||
latinIme.mInputLogic.endBatchInputInternal(latinIme.mSettings.getCurrent(),
|
latinIme.mInputLogic.onUpdateTailBatchInputCompleted(
|
||||||
|
latinIme.mSettings.getCurrent(),
|
||||||
(SuggestedWords) msg.obj, latinIme.mKeyboardSwitcher);
|
(SuggestedWords) msg.obj, latinIme.mKeyboardSwitcher);
|
||||||
break;
|
break;
|
||||||
case MSG_RESET_CACHES:
|
case MSG_RESET_CACHES:
|
||||||
|
@ -304,8 +305,8 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
|
||||||
ARG1_NOT_GESTURE_INPUT, ARG2_UNUSED, suggestedWords).sendToTarget();
|
ARG1_NOT_GESTURE_INPUT, ARG2_UNUSED, suggestedWords).sendToTarget();
|
||||||
}
|
}
|
||||||
|
|
||||||
public void onEndBatchInput(final SuggestedWords suggestedWords) {
|
public void showTailBatchInputResult(final SuggestedWords suggestedWords) {
|
||||||
obtainMessage(MSG_ON_END_BATCH_INPUT, suggestedWords).sendToTarget();
|
obtainMessage(MSG_UPDATE_TAIL_BATCH_INPUT_COMPLETED, suggestedWords).sendToTarget();
|
||||||
}
|
}
|
||||||
|
|
||||||
// Working variables for the following methods.
|
// Working variables for the following methods.
|
||||||
|
|
|
@ -601,7 +601,7 @@ public final class InputLogic {
|
||||||
}
|
}
|
||||||
|
|
||||||
public void onEndBatchInput(final InputPointers batchPointers) {
|
public void onEndBatchInput(final InputPointers batchPointers) {
|
||||||
mInputLogicHandler.onEndBatchInput(batchPointers, mAutoCommitSequenceNumber);
|
mInputLogicHandler.updateTailBatchInput(batchPointers, mAutoCommitSequenceNumber);
|
||||||
++mAutoCommitSequenceNumber;
|
++mAutoCommitSequenceNumber;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1794,7 +1794,7 @@ public final class InputLogic {
|
||||||
* @param settingsValues the current values of the settings.
|
* @param settingsValues the current values of the settings.
|
||||||
* @param suggestedWords suggestedWords to use.
|
* @param suggestedWords suggestedWords to use.
|
||||||
*/
|
*/
|
||||||
public void endBatchInputInternal(final SettingsValues settingsValues,
|
public void onUpdateTailBatchInputCompleted(final SettingsValues settingsValues,
|
||||||
final SuggestedWords suggestedWords,
|
final SuggestedWords suggestedWords,
|
||||||
// TODO: remove this argument
|
// TODO: remove this argument
|
||||||
final KeyboardSwitcher keyboardSwitcher) {
|
final KeyboardSwitcher keyboardSwitcher) {
|
||||||
|
|
|
@ -54,7 +54,8 @@ class InputLogicHandler implements Handler.Callback {
|
||||||
@Override
|
@Override
|
||||||
public void onCancelBatchInput() {}
|
public void onCancelBatchInput() {}
|
||||||
@Override
|
@Override
|
||||||
public void onEndBatchInput(final InputPointers batchPointers, final int sequenceNumber) {}
|
public void updateTailBatchInput(final InputPointers batchPointers,
|
||||||
|
final int sequenceNumber) {}
|
||||||
@Override
|
@Override
|
||||||
public void getSuggestedWords(final int sessionId, final int sequenceNumber,
|
public void getSuggestedWords(final int sessionId, final int sequenceNumber,
|
||||||
final OnGetSuggestedWordsCallback callback) {}
|
final OnGetSuggestedWordsCallback callback) {}
|
||||||
|
@ -106,7 +107,7 @@ class InputLogicHandler implements Handler.Callback {
|
||||||
* Fetch suggestions corresponding to an update of a batch input.
|
* Fetch suggestions corresponding to an update of a batch input.
|
||||||
* @param batchPointers the updated pointers, including the part that was passed last time.
|
* @param batchPointers the updated pointers, including the part that was passed last time.
|
||||||
* @param sequenceNumber the sequence number associated with this batch input.
|
* @param sequenceNumber the sequence number associated with this batch input.
|
||||||
* @param forEnd true if this is the end of a batch input, false if it's an update.
|
* @param isTailBatchInput true if this is the end of a batch input, false if it's an update.
|
||||||
*/
|
*/
|
||||||
// This method can be called from any thread and will see to it that the correct threads
|
// This method can be called from any thread and will see to it that the correct threads
|
||||||
// are used for parts that require it. This method will send a message to the Non-UI handler
|
// are used for parts that require it. This method will send a message to the Non-UI handler
|
||||||
|
@ -115,7 +116,7 @@ class InputLogicHandler implements Handler.Callback {
|
||||||
// send a message to the UI handler in LatinIME so that showing suggestions can be done on
|
// send a message to the UI handler in LatinIME so that showing suggestions can be done on
|
||||||
// the UI thread.
|
// the UI thread.
|
||||||
private void updateBatchInput(final InputPointers batchPointers,
|
private void updateBatchInput(final InputPointers batchPointers,
|
||||||
final int sequenceNumber, final boolean forEnd) {
|
final int sequenceNumber, final boolean isTailBatchInput) {
|
||||||
synchronized (mLock) {
|
synchronized (mLock) {
|
||||||
if (!mInBatchInput) {
|
if (!mInBatchInput) {
|
||||||
// Batch input has ended or canceled while the message was being delivered.
|
// Batch input has ended or canceled while the message was being delivered.
|
||||||
|
@ -136,12 +137,12 @@ class InputLogicHandler implements Handler.Callback {
|
||||||
suggestedWords = mInputLogic.mSuggestedWords;
|
suggestedWords = mInputLogic.mSuggestedWords;
|
||||||
}
|
}
|
||||||
mLatinIME.mHandler.showGesturePreviewAndSuggestionStrip(suggestedWords,
|
mLatinIME.mHandler.showGesturePreviewAndSuggestionStrip(suggestedWords,
|
||||||
forEnd /* dismissGestureFloatingPreviewText */);
|
isTailBatchInput /* dismissGestureFloatingPreviewText */);
|
||||||
if (forEnd) {
|
if (isTailBatchInput) {
|
||||||
mInBatchInput = false;
|
mInBatchInput = false;
|
||||||
// The following call schedules onEndBatchInputInternal
|
// The following call schedules onEndBatchInputInternal
|
||||||
// to be called on the UI thread.
|
// to be called on the UI thread.
|
||||||
mLatinIME.mHandler.onEndBatchInput(suggestedWords);
|
mLatinIME.mHandler.showTailBatchInputResult(suggestedWords);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
@ -159,13 +160,13 @@ class InputLogicHandler implements Handler.Callback {
|
||||||
// Called on the UI thread by InputLogic.
|
// Called on the UI thread by InputLogic.
|
||||||
public void onUpdateBatchInput(final InputPointers batchPointers,
|
public void onUpdateBatchInput(final InputPointers batchPointers,
|
||||||
final int sequenceNumber) {
|
final int sequenceNumber) {
|
||||||
updateBatchInput(batchPointers, sequenceNumber, false /* forEnd */);
|
updateBatchInput(batchPointers, sequenceNumber, false /* isTailBatchInput */);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Cancel a batch input.
|
* Cancel a batch input.
|
||||||
*
|
*
|
||||||
* Note that as opposed to onEndBatchInput, we do the UI side of this immediately on the
|
* Note that as opposed to updateTailBatchInput, we do the UI side of this immediately on the
|
||||||
* same thread, rather than get this to call a method in LatinIME. This is because
|
* same thread, rather than get this to call a method in LatinIME. This is because
|
||||||
* canceling a batch input does not necessitate the long operation of pulling suggestions.
|
* canceling a batch input does not necessitate the long operation of pulling suggestions.
|
||||||
*/
|
*/
|
||||||
|
@ -177,17 +178,20 @@ class InputLogicHandler implements Handler.Callback {
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Finish a batch input.
|
* Trigger an update for a tail batch input.
|
||||||
*
|
*
|
||||||
* This fetches suggestions, updates the suggestion strip and commits the first suggestion.
|
* A tail batch input is the last update for a gesture, the one that is triggered after the
|
||||||
* It also dismisses the floating text preview.
|
* user lifts their finger. This method schedules fetching suggestions on the non-UI thread,
|
||||||
|
* then when the suggestions are computed it comes back on the UI thread to update the
|
||||||
|
* suggestion strip, commit the first suggestion, and dismiss the floating text preview.
|
||||||
*
|
*
|
||||||
* @param batchPointers the updated batch pointers.
|
* @param batchPointers the updated batch pointers.
|
||||||
* @param sequenceNumber the sequence number associated with this batch input.
|
* @param sequenceNumber the sequence number associated with this batch input.
|
||||||
*/
|
*/
|
||||||
// Called on the UI thread by InputLogic.
|
// Called on the UI thread by InputLogic.
|
||||||
public void onEndBatchInput(final InputPointers batchPointers, final int sequenceNumber) {
|
public void updateTailBatchInput(final InputPointers batchPointers,
|
||||||
updateBatchInput(batchPointers, sequenceNumber, true /* forEnd */);
|
final int sequenceNumber) {
|
||||||
|
updateBatchInput(batchPointers, sequenceNumber, true /* isTailBatchInput */);
|
||||||
}
|
}
|
||||||
|
|
||||||
public void getSuggestedWords(final int sessionId, final int sequenceNumber,
|
public void getSuggestedWords(final int sessionId, final int sequenceNumber,
|
||||||
|
|
Loading…
Reference in New Issue