Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f6989da7 authored by Jeff Brown's avatar Jeff Brown
Browse files

Allow batching samples onto the pending motion event.

This enlarges the window of opportunity for batching to
encompass time spent for the window to become ready (while it is
busy processing the last event).

Change-Id: I3fb5a394ab1b85d6591192678168ca6e35dd9d53
parent 5487500c
Loading
Loading
Loading
Loading
+28 −0
Original line number Diff line number Diff line
@@ -2524,6 +2524,34 @@ void InputDispatcher::notifyMotion(nsecs_t eventTime, int32_t deviceId, uint32_t
                return; // done!
            }

            // BATCHING ONTO PENDING EVENT CASE
            //
            // Try to append a move sample to the currently pending event, if there is one.
            // We can do this as long as we are still waiting to find the targets for the
            // event.  Once the targets are locked-in we can only do streaming.
            if (mPendingEvent
                    && (!mPendingEvent->dispatchInProgress || !mCurrentInputTargetsValid)
                    && mPendingEvent->type == EventEntry::TYPE_MOTION) {
                MotionEntry* motionEntry = static_cast<MotionEntry*>(mPendingEvent);
                if (motionEntry->deviceId == deviceId && motionEntry->source == source) {
                    if (motionEntry->action != action
                            || motionEntry->pointerCount != pointerCount
                            || motionEntry->isInjected()) {
                        // Pending event is not compatible for appending new samples.  Stop here.
                        goto NoBatchingOrStreaming;
                    }

                    // The pending motion event is a move and is compatible for appending.
                    // Do the batching magic.
                    mAllocator.appendMotionSample(motionEntry, eventTime, pointerCoords);
#if DEBUG_BATCHING
                    LOGD("Appended motion sample onto batch for the pending motion event.");
#endif
                    mLock.unlock();
                    return; // done!
                }
            }

            // STREAMING CASE
            //
            // There is no pending motion event (of any kind) for this device in the inbound queue.