am a49d3264: Merge "[IL129] Change some confusing method names."

* commit 'a49d32642676d06a9f474dc97d71a871141ce594':
  [IL129] Change some confusing method names.
main
Jean Chalard 2014-04-02 09:22:34 +00:00 committed by Android Git Automerger
commit 22db22e1c9
3 changed files with 25 additions and 20 deletions

View File

@ -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.

View File

@ -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) {

View File

@ -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,