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

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

Merge "MotionEvent: Round coordinates to a precision of 0.001" into udc-dev... 

Merge "MotionEvent: Round coordinates to a precision of 0.001" into udc-dev am: f2075052 am: ef9c3e26

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



Change-Id: Idf660bc945b63f34c11fb34f882f99720fea2b93
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 0af98d8b ef9c3e26

include/input/Input.h

+2 −0
Original line number Diff line number Diff line
@@ -855,6 +855,8 @@ public: 
                                               const PointerCoords&);

    static PointerCoords calculateTransformedCoords(uint32_t source, const ui::Transform&,

                                                    const PointerCoords&);

    // The rounding precision for transformed motion events.

    static constexpr float ROUNDING_PRECISION = 0.001f;



protected:

    int32_t mAction;

libs/input/Input.cpp

+17 −4
Original line number Diff line number Diff line
@@ -151,10 +151,23 @@ int32_t IdGenerator::nextId() const { 


// --- InputEvent ---



// Due to precision limitations when working with floating points, transforming - namely

// scaling - floating points can lead to minute errors. We round transformed values to approximately

// three decimal places so that values like 0.99997 show up as 1.0.

inline float roundTransformedCoords(float val) {

    // Use a power to two to approximate three decimal places to potentially reduce some cycles.

    // This should be at least as precise as MotionEvent::ROUNDING_PRECISION.

    return std::round(val * 1024.f) / 1024.f;

}



inline vec2 roundTransformedCoords(vec2 p) {

    return {roundTransformedCoords(p.x), roundTransformedCoords(p.y)};

}



vec2 transformWithoutTranslation(const ui::Transform& transform, const vec2& xy) {

    const vec2 transformedXy = transform.transform(xy);

    const vec2 transformedOrigin = transform.transform(0, 0);

    return transformedXy - transformedOrigin;

    return roundTransformedCoords(transformedXy - transformedOrigin);

}



float transformAngle(const ui::Transform& transform, float angleRadians) {
@@ -606,12 +619,12 @@ std::optional<ui::Rotation> MotionEvent::getSurfaceRotation() const { 


float MotionEvent::getXCursorPosition() const {

    vec2 vals = mTransform.transform(getRawXCursorPosition(), getRawYCursorPosition());

    return vals.x;

    return roundTransformedCoords(vals.x);

}



float MotionEvent::getYCursorPosition() const {

    vec2 vals = mTransform.transform(getRawXCursorPosition(), getRawYCursorPosition());

    return vals.y;

    return roundTransformedCoords(vals.y);

}



void MotionEvent::setCursorPosition(float x, float y) {
@@ -933,7 +946,7 @@ std::string MotionEvent::actionToString(int32_t action) { 
static inline vec2 calculateTransformedXYUnchecked(uint32_t source, const ui::Transform& transform,

                                                   const vec2& xy) {

    return shouldDisregardOffset(source) ? transformWithoutTranslation(transform, xy)

                                         : transform.transform(xy);

                                         : roundTransformedCoords(transform.transform(xy));

}



vec2 MotionEvent::calculateTransformedXY(uint32_t source, const ui::Transform& transform,

libs/input/tests/InputEvent_test.cpp

+195 −133

File changed.

Preview size limit exceeded, changes collapsed.

libs/input/tests/InputPublisherAndConsumer_test.cpp

+12 −8

File changed.

Preview size limit exceeded, changes collapsed.

services/inputflinger/tests/InputDispatcher_test.cpp

+4 −4

File changed.

Preview size limit exceeded, changes collapsed.