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

Commit 94f7ddb4 authored by Arpit Singh's avatar Arpit Singh Committed by Android (Google) Code Review
Browse files

Merge "InputMapper refactor: SwitchInputMapper"

parents 368adca4 df992eb9

services/inputflinger/reader/InputDevice.cpp

+1 −1
Original line number Diff line number Diff line
@@ -446,7 +446,7 @@ std::vector<std::unique_ptr<InputMapper>> InputDevice::createMappers( 


    // Switch-like devices.

    if (classes.test(InputDeviceClass::SWITCH)) {

        mappers.push_back(std::make_unique<SwitchInputMapper>(contextPtr, readerConfig));

        mappers.push_back(createInputMapper<SwitchInputMapper>(contextPtr, readerConfig));

    }



    // Scroll wheel-like devices.

services/inputflinger/reader/mapper/SwitchInputMapper.h

+6 −2
Original line number Diff line number Diff line
@@ -22,8 +22,10 @@ namespace android { 


class SwitchInputMapper : public InputMapper {

public:

    explicit SwitchInputMapper(InputDeviceContext& deviceContext,

                               const InputReaderConfiguration& readerConfig);

    template <class T, class... Args>

    friend std::unique_ptr<T> createInputMapper(InputDeviceContext& deviceContext,

                                                const InputReaderConfiguration& readerConfig,

                                                Args... args);

    virtual ~SwitchInputMapper();



    virtual uint32_t getSources() const override;
@@ -36,6 +38,8 @@ private: 
    uint32_t mSwitchValues;

    uint32_t mUpdatedSwitchMask;



    explicit SwitchInputMapper(InputDeviceContext& deviceContext,

                               const InputReaderConfiguration& readerConfig);

    void processSwitch(int32_t switchCode, int32_t switchValue);

    [[nodiscard]] std::list<NotifyArgs> sync(nsecs_t when);

};

services/inputflinger/tests/InputReader_test.cpp

+3 −3
Original line number Diff line number Diff line
@@ -2603,13 +2603,13 @@ protected: 
};















TEST_F(SwitchInputMapperTest, GetSources) {













    SwitchInputMapper& mapper = addMapperAndConfigure<SwitchInputMapper>();













    SwitchInputMapper& mapper = constructAndAddMapper<SwitchInputMapper>();





























    ASSERT_EQ(uint32_t(AINPUT_SOURCE_SWITCH), mapper.getSources());















}





























TEST_F(SwitchInputMapperTest, GetSwitchState) {













    SwitchInputMapper& mapper = addMapperAndConfigure<SwitchInputMapper>();













    SwitchInputMapper& mapper = constructAndAddMapper<SwitchInputMapper>();





























    mFakeEventHub->setSwitchState(EVENTHUB_ID, SW_LID, 1);















    ASSERT_EQ(1, mapper.getSwitchState(AINPUT_SOURCE_ANY, SW_LID));









@@ -2619,7 +2619,7 @@ TEST_F(SwitchInputMapperTest, GetSwitchState) {













}





























TEST_F(SwitchInputMapperTest, Process) {













    SwitchInputMapper& mapper = addMapperAndConfigure<SwitchInputMapper>();













    SwitchInputMapper& mapper = constructAndAddMapper<SwitchInputMapper>();















    std::list<NotifyArgs> out;















    out = process(mapper, ARBITRARY_TIME, READ_TIME, EV_SW, SW_LID, 1);















    ASSERT_TRUE(out.empty());