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

Commit f30d3c1f authored by Prabir Pradhan's avatar Prabir Pradhan Committed by Automerger Merge Worker
Browse files

Merge changes I111361f7,Ic4979b91,Ic289d772 into tm-qpr-dev am: 7fa3483d 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/19758680



Change-Id: I09e6632b8589ad155e1d3678be39a055e148c807
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents b22d75f6 7fa3483d

services/inputflinger/reader/mapper/TouchInputMapper.cpp

+17 −25
Original line number Diff line number Diff line
@@ -44,6 +44,8 @@ static constexpr nsecs_t STYLUS_DATA_LATENCY = ms2ns(10); 


// --- Static Definitions ---



static const DisplayViewport kUninitializedViewport;



template <typename T>

inline static void swap(T& a, T& b) {

    T temp = a;
@@ -390,10 +392,10 @@ void TouchInputMapper::configure(nsecs_t when, const InputReaderConfiguration* c 
    }



    if (changes && resetNeeded) {

        // If device was reset, cancel touch event and update touch spot state.

        cancelTouch(mCurrentRawState.when, mCurrentRawState.readTime);

        mCurrentCookedState.clear();

        updateTouchSpots();

        // If the device needs to be reset, cancel any ongoing gestures and reset the state.

        cancelTouch(when, when);

        reset(when);



        // Send reset, unless this is the first time the device has been configured,

        // in which case the reader will call reset itself after all mappers are ready.

        NotifyDeviceResetArgs args(getContext()->getNextId(), when, getDeviceId());
@@ -881,7 +883,7 @@ void TouchInputMapper::initializeOrientedRanges() { 
}



void TouchInputMapper::configureInputDevice(nsecs_t when, bool* outResetNeeded) {

    DeviceMode oldDeviceMode = mDeviceMode;

    const DeviceMode oldDeviceMode = mDeviceMode;



    resolveExternalStylusPresence();
@@ -910,43 +912,37 @@ void TouchInputMapper::configureInputDevice(nsecs_t when, bool* outResetNeeded) 
        mDeviceMode = DeviceMode::UNSCALED;

    }



    // Ensure we have valid X and Y axes.

    const std::optional<DisplayViewport> newViewportOpt = findViewport();



    // Ensure the device is valid and can be used.

    if (!mRawPointerAxes.x.valid || !mRawPointerAxes.y.valid) {

        ALOGW("Touch device '%s' did not report support for X or Y axis!  "

              "The device will be inoperable.",

              getDeviceName().c_str());

        mDeviceMode = DeviceMode::DISABLED;

        return;

    }



    // Get associated display dimensions.

    std::optional<DisplayViewport> newViewport = findViewport();

    if (!newViewport) {

    } else if (!newViewportOpt) {

        ALOGI("Touch device '%s' could not query the properties of its associated "

              "display.  The device will be inoperable until the display size "

              "becomes available.",

              getDeviceName().c_str());

        mDeviceMode = DeviceMode::DISABLED;

        return;

    }



    if (!newViewport->isActive) {

    } else if (!newViewportOpt->isActive) {

        ALOGI("Disabling %s (device %i) because the associated viewport is not active",

              getDeviceName().c_str(), getDeviceId());

        mDeviceMode = DeviceMode::DISABLED;

        return;

    }



    // Raw width and height in the natural orientation.

    const int32_t rawWidth = mRawPointerAxes.getRawWidth();

    const int32_t rawHeight = mRawPointerAxes.getRawHeight();



    const bool viewportChanged = mViewport != *newViewport;

    const DisplayViewport& newViewport = newViewportOpt.value_or(kUninitializedViewport);

    const bool viewportChanged = mViewport != newViewport;

    bool skipViewportUpdate = false;

    if (viewportChanged) {

        const bool viewportOrientationChanged = mViewport.orientation != newViewport->orientation;

        const bool viewportDisplayIdChanged = mViewport.displayId != newViewport->displayId;

        mViewport = *newViewport;

        const bool viewportOrientationChanged = mViewport.orientation != newViewport.orientation;

        const bool viewportDisplayIdChanged = mViewport.displayId != newViewport.displayId;

        mViewport = newViewport;



        if (mDeviceMode == DeviceMode::DIRECT || mDeviceMode == DeviceMode::POINTER) {

            // Convert rotated viewport to the natural orientation.
@@ -1105,10 +1101,6 @@ void TouchInputMapper::configureInputDevice(nsecs_t when, bool* outResetNeeded) 
            // of the diagonal axis of the touch pad.  Touches that are wider than this are

            // translated into freeform gestures.

            mPointerGestureMaxSwipeWidth = mConfig.pointerGestureSwipeMaxWidthRatio * rawDiagonal;



            // Abort current pointer usages because the state has changed.

            const nsecs_t readTime = when; // synthetic event

            abortPointerUsage(when, readTime, 0 /*policyFlags*/);

        }



        // Inform the dispatcher about the changes.

services/inputflinger/tests/InputReader_test.cpp

+64 −9
Original line number Diff line number Diff line
@@ -6726,6 +6726,61 @@ TEST_F(SingleTouchInputMapperTest, 
    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyDeviceResetWasCalled());

}



TEST_F(SingleTouchInputMapperTest,

       Process_WhenViewportActiveStatusChanged_TouchIsCanceledAndDeviceIsReset) {

    addConfigurationProperty("touch.deviceType", "touchScreen");

    prepareDisplay(DISPLAY_ORIENTATION_0);

    prepareButtons();

    prepareAxes(POSITION);

    SingleTouchInputMapper& mapper = addMapperAndConfigure<SingleTouchInputMapper>();

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyDeviceResetWasCalled());

    NotifyMotionArgs motionArgs;



    // Start a new gesture.

    processDown(mapper, 100, 200);

    processSync(mapper);

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyMotionWasCalled(&motionArgs));

    ASSERT_EQ(AMOTION_EVENT_ACTION_DOWN, motionArgs.action);



    // Make the viewport inactive. This will put the device in disabled mode.

    auto viewport = mFakePolicy->getDisplayViewportByType(ViewportType::INTERNAL);

    viewport->isActive = false;

    mFakePolicy->updateViewport(*viewport);

    configureDevice(InputReaderConfiguration::CHANGE_DISPLAY_INFO);



    // We should receive a cancel event for the ongoing gesture.

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyMotionWasCalled(&motionArgs));

    ASSERT_EQ(AMOTION_EVENT_ACTION_CANCEL, motionArgs.action);

    // Then we should be notified that the device was reset.

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyDeviceResetWasCalled());



    // No events are generated while the viewport is inactive.

    processMove(mapper, 101, 201);

    processSync(mapper);

    processDown(mapper, 102, 202);

    processSync(mapper);

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyMotionWasNotCalled());



    // Make the viewport active again. The device should resume processing events.

    viewport->isActive = true;

    mFakePolicy->updateViewport(*viewport);

    configureDevice(InputReaderConfiguration::CHANGE_DISPLAY_INFO);



    // The device is reset because it changes back to direct mode, without generating any events.

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyDeviceResetWasCalled());

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyMotionWasNotCalled());



    // Start a new gesture.

    processDown(mapper, 100, 200);

    processSync(mapper);

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyMotionWasCalled(&motionArgs));

    ASSERT_EQ(AMOTION_EVENT_ACTION_DOWN, motionArgs.action);



    // No more events.

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyMotionWasNotCalled());

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyDeviceResetWasNotCalled());

}



// --- TouchDisplayProjectionTest ---



class TouchDisplayProjectionTest : public SingleTouchInputMapperTest {
@@ -8583,27 +8638,27 @@ TEST_F(MultiTouchInputMapperTest, Process_DeactivateViewport_AbortTouches) { 
    ASSERT_TRUE(mFakePolicy->updateViewport(displayViewport));

    configureDevice(InputReaderConfiguration::CHANGE_DISPLAY_INFO);



    // Finger move

    // The ongoing touch should be canceled immediately

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyMotionWasCalled(&motionArgs));

    EXPECT_EQ(AMOTION_EVENT_ACTION_CANCEL, motionArgs.action);



    // Finger move is ignored

    x += 10, y += 10;

    processPosition(mapper, x, y);

    processSync(mapper);



    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyMotionWasCalled(&motionArgs));

    EXPECT_EQ(AMOTION_EVENT_ACTION_CANCEL, motionArgs.action);

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyMotionWasNotCalled());



    // Reactivate display viewport

    displayViewport.isActive = true;

    ASSERT_TRUE(mFakePolicy->updateViewport(displayViewport));

    configureDevice(InputReaderConfiguration::CHANGE_DISPLAY_INFO);



    // Finger move again

    // Finger move again starts new gesture

    x += 10, y += 10;

    processPosition(mapper, x, y);

    processSync(mapper);



    // Gesture is aborted, so events after display is activated won't be dispatched until there is

    // no pointer on the touch device.

    mFakeListener->assertNotifyMotionWasNotCalled();

    ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyMotionWasCalled(&motionArgs));

    EXPECT_EQ(AMOTION_EVENT_ACTION_DOWN, motionArgs.action);

}



TEST_F(MultiTouchInputMapperTest, Process_Pointer_ShowTouches) {