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

Commit e91389fb authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Create InputTarget in a separate function" into udc-dev am: 5796a176 

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



Change-Id: I558c8e11e70e67a78e3fa97f95e8ac95dba3b7b4
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents c9e11954 5796a176

include/input/Input.h

+0 −1
Original line number Diff line number Diff line
@@ -30,7 +30,6 @@ 
#include <stdint.h>

#include <ui/Transform.h>

#include <utils/BitSet.h>

#include <utils/RefBase.h>

#include <utils/Timers.h>

#include <array>

#include <limits>

services/inputflinger/dispatcher/InputDispatcher.cpp

+28 −17
Original line number Diff line number Diff line
@@ -2811,6 +2811,30 @@ void InputDispatcher::addDragEventLocked(const MotionEntry& entry) { 
    }

}



std::optional<InputTarget> InputDispatcher::createInputTargetLocked(

        const sp<android::gui::WindowInfoHandle>& windowHandle,

        ftl::Flags<InputTarget::Flags> targetFlags,

        std::optional<nsecs_t> firstDownTimeInTarget) const {

    std::shared_ptr<InputChannel> inputChannel = getInputChannelLocked(windowHandle->getToken());

    if (inputChannel == nullptr) {

        ALOGW("Not creating InputTarget for %s, no input channel", windowHandle->getName().c_str());

        return {};

    }

    InputTarget inputTarget;

    inputTarget.inputChannel = inputChannel;

    inputTarget.flags = targetFlags;

    inputTarget.globalScaleFactor = windowHandle->getInfo()->globalScaleFactor;

    inputTarget.firstDownTimeInTarget = firstDownTimeInTarget;

    const auto& displayInfoIt = mDisplayInfos.find(windowHandle->getInfo()->displayId);

    if (displayInfoIt != mDisplayInfos.end()) {

        inputTarget.displayTransform = displayInfoIt->second.transform;

    } else {

        // DisplayInfo not found for this window on display windowInfo->displayId.

        // TODO(b/198444055): Make this an error message after 'setInputWindows' API is removed.

    }

    return inputTarget;

}



void InputDispatcher::addWindowTargetLocked(const sp<WindowInfoHandle>& windowHandle,

                                            ftl::Flags<InputTarget::Flags> targetFlags,

                                            std::bitset<MAX_POINTER_ID + 1> pointerIds,
@@ -2826,25 +2850,12 @@ void InputDispatcher::addWindowTargetLocked(const sp<WindowInfoHandle>& windowHa 
    const WindowInfo* windowInfo = windowHandle->getInfo();



    if (it == inputTargets.end()) {

        InputTarget inputTarget;

        std::shared_ptr<InputChannel> inputChannel =

                getInputChannelLocked(windowHandle->getToken());

        if (inputChannel == nullptr) {

            ALOGW("Window %s already unregistered input channel", windowHandle->getName().c_str());

        std::optional<InputTarget> target =

                createInputTargetLocked(windowHandle, targetFlags, firstDownTimeInTarget);

        if (!target) {

            return;

        }

        inputTarget.inputChannel = inputChannel;

        inputTarget.flags = targetFlags;

        inputTarget.globalScaleFactor = windowInfo->globalScaleFactor;

        inputTarget.firstDownTimeInTarget = firstDownTimeInTarget;

        const auto& displayInfoIt = mDisplayInfos.find(windowInfo->displayId);

        if (displayInfoIt != mDisplayInfos.end()) {

            inputTarget.displayTransform = displayInfoIt->second.transform;

        } else {

            // DisplayInfo not found for this window on display windowInfo->displayId.

            // TODO(b/198444055): Make this an error message after 'setInputWindows' API is removed.

        }

        inputTargets.push_back(inputTarget);

        inputTargets.push_back(*target);

        it = inputTargets.end() - 1;

    }

services/inputflinger/dispatcher/InputDispatcher.h

+4 −0
Original line number Diff line number Diff line
@@ -556,6 +556,10 @@ private: 
    std::vector<Monitor> selectResponsiveMonitorsLocked(

            const std::vector<Monitor>& gestureMonitors) const REQUIRES(mLock);



    std::optional<InputTarget> createInputTargetLocked(

            const sp<android::gui::WindowInfoHandle>& windowHandle,

            ftl::Flags<InputTarget::Flags> targetFlags,

            std::optional<nsecs_t> firstDownTimeInTarget) const REQUIRES(mLock);

    void addWindowTargetLocked(const sp<android::gui::WindowInfoHandle>& windowHandle,

                               ftl::Flags<InputTarget::Flags> targetFlags,

                               std::bitset<MAX_POINTER_ID + 1> pointerIds,