am 06be5777: am 55387ba4: am 9fbeeb60: Merge "Revert "[HW7] Introduce consumed events"" into lmp-dev

* commit '06be57772f2fe5e3b92bfa137d90962fc792d258':
  Revert "[HW7] Introduce consumed events"
main
Ken Wakasa 2014-07-30 17:17:08 +00:00 committed by Android Git Automerger
commit 849164f223
5 changed files with 15 additions and 31 deletions

View File

@ -84,8 +84,6 @@ public class CombinerChain {
* Process an event through the combining chain, and return a processed event to apply. * Process an event through the combining chain, and return a processed event to apply.
* @param previousEvents the list of previous events in this composition * @param previousEvents the list of previous events in this composition
* @param newEvent the new event to process * @param newEvent the new event to process
* @return the processed event. It may be the same event, or a consumed event, or a completely
* new event. However it may never be null.
*/ */
public Event processEvent(final ArrayList<Event> previousEvents, final Event newEvent) { public Event processEvent(final ArrayList<Event> previousEvents, final Event newEvent) {
final ArrayList<Event> modifiablePreviousEvents = new ArrayList<>(previousEvents); final ArrayList<Event> modifiablePreviousEvents = new ArrayList<>(previousEvents);

View File

@ -32,11 +32,12 @@ public class DeadKeyCombiner implements Combiner {
@Override @Override
public Event processEvent(final ArrayList<Event> previousEvents, final Event event) { public Event processEvent(final ArrayList<Event> previousEvents, final Event event) {
if (null == event) return null; // Just in case some combiner is broken
if (TextUtils.isEmpty(mDeadSequence)) { if (TextUtils.isEmpty(mDeadSequence)) {
if (event.isDead()) { if (event.isDead()) {
mDeadSequence.appendCodePoint(event.mCodePoint); mDeadSequence.appendCodePoint(event.mCodePoint);
} }
return Event.createConsumedEvent(event); return event;
} else { } else {
// TODO: Allow combining for several dead chars rather than only the first one. // TODO: Allow combining for several dead chars rather than only the first one.
// The framework doesn't know how to do this now. // The framework doesn't know how to do this now.

View File

@ -67,8 +67,6 @@ public class Event {
final private static int FLAG_DEAD = 0x1; final private static int FLAG_DEAD = 0x1;
// This event is coming from a key repeat, software or hardware. // This event is coming from a key repeat, software or hardware.
final private static int FLAG_REPEAT = 0x2; final private static int FLAG_REPEAT = 0x2;
// This event has already been consumed.
final private static int FLAG_CONSUMED = 0x4;
final private int mEventType; // The type of event - one of the constants above final private int mEventType; // The type of event - one of the constants above
// The code point associated with the event, if relevant. This is a unicode code point, and // The code point associated with the event, if relevant. This is a unicode code point, and
@ -221,17 +219,6 @@ public class Event {
null /* next */); null /* next */);
} }
/**
* Creates an event identical to the passed event, but that has already been consumed.
* @param source the event to copy the properties of.
* @return an identical event marked as consumed.
*/
public static Event createConsumedEvent(final Event source) {
return new Event(source.mEventType, source.mText, source.mCodePoint, source.mKeyCode,
source.mX, source.mY, source.mSuggestedWordInfo, source.mFlags | FLAG_CONSUMED,
source.mNextEvent);
}
public static Event createNotHandledEvent() { public static Event createNotHandledEvent() {
return new Event(EVENT_TYPE_NOT_HANDLED, null /* text */, NOT_A_CODE_POINT, NOT_A_KEY_CODE, return new Event(EVENT_TYPE_NOT_HANDLED, null /* text */, NOT_A_CODE_POINT, NOT_A_KEY_CODE,
Constants.NOT_A_COORDINATE, Constants.NOT_A_COORDINATE, Constants.NOT_A_COORDINATE, Constants.NOT_A_COORDINATE,
@ -254,8 +241,6 @@ public class Event {
return 0 != (FLAG_REPEAT & mFlags); return 0 != (FLAG_REPEAT & mFlags);
} }
public boolean isConsumed() { return 0 != (FLAG_CONSUMED & mFlags); }
// Returns whether this is a fake key press from the suggestion strip. This happens with // Returns whether this is a fake key press from the suggestion strip. This happens with
// punctuation signs selected from the suggestion strip. // punctuation signs selected from the suggestion strip.
public boolean isSuggestionStripPress() { public boolean isSuggestionStripPress() {

View File

@ -111,7 +111,7 @@ public class MyanmarReordering implements Combiner {
* Clears the currently combining stream of events and returns the resulting software text * Clears the currently combining stream of events and returns the resulting software text
* event corresponding to the stream. Optionally adds a new event to the cleared stream. * event corresponding to the stream. Optionally adds a new event to the cleared stream.
* @param newEvent the new event to add to the stream. null if none. * @param newEvent the new event to add to the stream. null if none.
* @return the resulting software text event. Never null. * @return the resulting software text event. Null if none.
*/ */
private Event clearAndGetResultingEvent(final Event newEvent) { private Event clearAndGetResultingEvent(final Event newEvent) {
final CharSequence combinedText; final CharSequence combinedText;
@ -124,7 +124,7 @@ public class MyanmarReordering implements Combiner {
if (null != newEvent) { if (null != newEvent) {
mCurrentEvents.add(newEvent); mCurrentEvents.add(newEvent);
} }
return null == combinedText ? Event.createConsumedEvent(newEvent) return null == combinedText ? null
: Event.createSoftwareTextEvent(combinedText, Event.NOT_A_KEY_CODE); : Event.createSoftwareTextEvent(combinedText, Event.NOT_A_KEY_CODE);
} }
@ -135,7 +135,7 @@ public class MyanmarReordering implements Combiner {
final Event lastEvent = getLastEvent(); final Event lastEvent = getLastEvent();
if (null == lastEvent) { if (null == lastEvent) {
mCurrentEvents.add(newEvent); mCurrentEvents.add(newEvent);
return Event.createConsumedEvent(newEvent); return null;
} else if (isConsonantOrMedial(lastEvent.mCodePoint)) { } else if (isConsonantOrMedial(lastEvent.mCodePoint)) {
final Event resultingEvent = clearAndGetResultingEvent(null); final Event resultingEvent = clearAndGetResultingEvent(null);
mCurrentEvents.add(Event.createSoftwareKeypressEvent(ZERO_WIDTH_NON_JOINER, mCurrentEvents.add(Event.createSoftwareKeypressEvent(ZERO_WIDTH_NON_JOINER,
@ -151,7 +151,7 @@ public class MyanmarReordering implements Combiner {
final Event lastEvent = getLastEvent(); final Event lastEvent = getLastEvent();
if (null == lastEvent) { if (null == lastEvent) {
mCurrentEvents.add(newEvent); mCurrentEvents.add(newEvent);
return Event.createConsumedEvent(newEvent); return null;
} else if (VOWEL_E == lastEvent.mCodePoint) { } else if (VOWEL_E == lastEvent.mCodePoint) {
final int eventSize = mCurrentEvents.size(); final int eventSize = mCurrentEvents.size();
if (eventSize >= 2 if (eventSize >= 2
@ -162,7 +162,7 @@ public class MyanmarReordering implements Combiner {
mCurrentEvents.remove(eventSize - 2); mCurrentEvents.remove(eventSize - 2);
mCurrentEvents.add(newEvent); mCurrentEvents.add(newEvent);
mCurrentEvents.add(lastEvent); mCurrentEvents.add(lastEvent);
return Event.createConsumedEvent(newEvent); return null;
} }
// If there is already a consonant, then we are starting a new syllable. // If there is already a consonant, then we are starting a new syllable.
for (int i = eventSize - 2; i >= 0; --i) { for (int i = eventSize - 2; i >= 0; --i) {
@ -174,7 +174,7 @@ public class MyanmarReordering implements Combiner {
mCurrentEvents.remove(eventSize - 1); mCurrentEvents.remove(eventSize - 1);
mCurrentEvents.add(newEvent); mCurrentEvents.add(newEvent);
mCurrentEvents.add(lastEvent); mCurrentEvents.add(lastEvent);
return Event.createConsumedEvent(newEvent); return null;
} else { // lastCodePoint is a consonant/medial. But if it's something else it's fine } else { // lastCodePoint is a consonant/medial. But if it's something else it's fine
return clearAndGetResultingEvent(newEvent); return clearAndGetResultingEvent(newEvent);
} }
@ -182,7 +182,7 @@ public class MyanmarReordering implements Combiner {
final Event lastEvent = getLastEvent(); final Event lastEvent = getLastEvent();
if (null == lastEvent) { if (null == lastEvent) {
mCurrentEvents.add(newEvent); mCurrentEvents.add(newEvent);
return Event.createConsumedEvent(newEvent); return null;
} else if (VOWEL_E == lastEvent.mCodePoint) { } else if (VOWEL_E == lastEvent.mCodePoint) {
final int eventSize = mCurrentEvents.size(); final int eventSize = mCurrentEvents.size();
// If there is already a consonant, then we are in the middle of a syllable, and we // If there is already a consonant, then we are in the middle of a syllable, and we
@ -198,7 +198,7 @@ public class MyanmarReordering implements Combiner {
mCurrentEvents.remove(eventSize - 1); mCurrentEvents.remove(eventSize - 1);
mCurrentEvents.add(newEvent); mCurrentEvents.add(newEvent);
mCurrentEvents.add(lastEvent); mCurrentEvents.add(lastEvent);
return Event.createConsumedEvent(newEvent); return null;
} }
// Otherwise, we just commit everything. // Otherwise, we just commit everything.
return clearAndGetResultingEvent(null); return clearAndGetResultingEvent(null);
@ -228,10 +228,10 @@ public class MyanmarReordering implements Combiner {
mCurrentEvents.remove(eventSize - 1); mCurrentEvents.remove(eventSize - 1);
} }
} }
return Event.createConsumedEvent(newEvent); return null;
} else if (eventSize > 0) { } else if (eventSize > 0) {
mCurrentEvents.remove(eventSize - 1); mCurrentEvents.remove(eventSize - 1);
return Event.createConsumedEvent(newEvent); return null;
} }
} }
} }

View File

@ -177,7 +177,7 @@ public final class WordComposer {
/** /**
* Process an event and return an event, and return a processed event to apply. * Process an event and return an event, and return a processed event to apply.
* @param event the unprocessed event. * @param event the unprocessed event.
* @return the processed event. Never null, but may be marked as consumed. * @return the processed event.
*/ */
public Event processEvent(final Event event) { public Event processEvent(final Event event) {
final Event processedEvent = mCombinerChain.processEvent(mEvents, event); final Event processedEvent = mCombinerChain.processEvent(mEvents, event);
@ -191,14 +191,14 @@ public final class WordComposer {
* All input events should be supported, including software/hardware events, characters as well * All input events should be supported, including software/hardware events, characters as well
* as deletions, multiple inputs and gestures. * as deletions, multiple inputs and gestures.
* *
* @param event the event to apply. Must not be null. * @param event the event to apply.
*/ */
public void applyProcessedEvent(final Event event) { public void applyProcessedEvent(final Event event) {
mCombinerChain.applyProcessedEvent(event);
final int primaryCode = event.mCodePoint; final int primaryCode = event.mCodePoint;
final int keyX = event.mX; final int keyX = event.mX;
final int keyY = event.mY; final int keyY = event.mY;
final int newIndex = size(); final int newIndex = size();
mCombinerChain.applyProcessedEvent(event);
refreshTypedWordCache(); refreshTypedWordCache();
mCursorPositionWithinWord = mCodePointSize; mCursorPositionWithinWord = mCodePointSize;
// We may have deleted the last one. // We may have deleted the last one.