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

Commit d604df19 authored by Daniel Norman's avatar Daniel Norman Committed by Android (Google) Code Review
Browse files

Revert "InputDispatcher: check consistency of filtered injected events -- try 2" 

This reverts commit 527a4cfa.

Reason for revert: http://b/336186130#comment10

Change-Id: Iac08deb8db20a6be02e416edb7656ab893aaa495
parent 527a4cfa

services/inputflinger/dispatcher/InputDispatcher.cpp

+0 −26
Original line number Diff line number Diff line
@@ -4551,7 +4551,6 @@ void InputDispatcher::notifySwitch(const NotifySwitchArgs& args) { 
}



void InputDispatcher::notifyDeviceReset(const NotifyDeviceResetArgs& args) {

    // TODO(b/308677868) Remove device reset from the InputListener interface

    if (debugInboundEventDetails()) {

        ALOGD("notifyDeviceReset - eventTime=%" PRId64 ", deviceId=%d", args.eventTime,

              args.deviceId);
@@ -4693,30 +4692,6 @@ InputEventInjectionResult InputDispatcher::injectInputEvent(const InputEvent* ev 
            }



            mLock.lock();



            if (policyFlags & POLICY_FLAG_FILTERED) {

                // The events from InputFilter impersonate real hardware devices. Check these

                // events for consistency and print an error. An inconsistent event sent from

                // InputFilter could cause a crash in the later stages of dispatching pipeline.

                auto [it, _] =

                        mInputFilterVerifiersByDisplay

                                .try_emplace(displayId,

                                             StringPrintf("Injection on %" PRId32, displayId));

                InputVerifier& verifier = it->second;



                Result<void> result =

                        verifier.processMovement(resolvedDeviceId, motionEvent.getSource(),

                                                 motionEvent.getAction(),

                                                 motionEvent.getPointerCount(),

                                                 motionEvent.getPointerProperties(),

                                                 motionEvent.getSamplePointerCoords(), flags);

                if (!result.ok()) {

                    logDispatchStateLocked();

                    LOG(ERROR) << "Inconsistent event: " << motionEvent

                               << ", reason: " << result.error();

                }

            }



            const nsecs_t* sampleEventTimes = motionEvent.getSampleEventTimes();

            const size_t pointerCount = motionEvent.getPointerCount();

            const std::vector<PointerProperties>
@@ -6781,7 +6756,6 @@ void InputDispatcher::displayRemoved(int32_t displayId) { 
        // Remove the associated touch mode state.

        mTouchModePerDisplay.erase(displayId);

        mVerifiersByDisplay.erase(displayId);

        mInputFilterVerifiersByDisplay.erase(displayId);

    } // release lock



    // Wake up poll loop since it may need to make new input dispatching choices.

services/inputflinger/dispatcher/InputDispatcher.h

+1 −2
Original line number Diff line number Diff line
@@ -286,8 +286,7 @@ private: 
    void transformMotionEntryForInjectionLocked(MotionEntry&,

                                                const ui::Transform& injectedTransform) const

            REQUIRES(mLock);

    // Per-display correction of injected events

    std::map</*displayId*/ int32_t, InputVerifier> mInputFilterVerifiersByDisplay GUARDED_BY(mLock);



    std::condition_variable mInjectionSyncFinished;

    void incrementPendingForegroundDispatches(EventEntry& entry);

    void decrementPendingForegroundDispatches(EventEntry& entry);