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

Commit 7f2537da authored by Melody Hsu's avatar Melody Hsu
Browse files

Revert "Validate position and matrix for txs" 

Revert submission 34305219-invalidate_inf_position_matrix

Reason for revert: b/430341531, b/430351590

Reverted changes: /q/submissionid:34305219-invalidate_inf_position_matrix

Change-Id: Ie6dbd1d688ddbda91c19112b11fb338f58befd39
parent ebc2e2eb

include/android/surface_control.h

+0 −2
Original line number Diff line number Diff line
@@ -496,7 +496,6 @@ void ASurfaceTransaction_setCrop(ASurfaceTransaction* _Nonnull transaction, 


/**

 * Specifies the position in the parent's space where the surface will be drawn.

 * Only permits finite values to be set.

 *

 * \param x The x position to render the surface.

 * \param y The y position to render the surface.
@@ -520,7 +519,6 @@ void ASurfaceTransaction_setBufferTransform(ASurfaceTransaction* _Nonnull transa 


/**

 * Sets an x and y scale of a surface with (0, 0) as the centerpoint of the scale.

 * Only permits finite values to be set.

 *

 * \param xScale The scale in the x direction. Must be greater than 0.

 * \param yScale The scale in the y direction. Must be greater than 0.

libs/gui/SurfaceComposerClient.cpp

+0 −3
Original line number Diff line number Diff line
@@ -1322,7 +1322,6 @@ SurfaceComposerClient::Transaction& SurfaceComposerClient::Transaction::setPosit 
        mStatus = BAD_INDEX;

        return *this;

    }

    LOG_ALWAYS_FATAL_IF(!isfinite(x) || !isfinite(y), "setPosition called with infinite values");

    s->what |= layer_state_t::ePositionChanged;

    s->x = x;

    s->y = y;
@@ -1463,8 +1462,6 @@ SurfaceComposerClient::Transaction& SurfaceComposerClient::Transaction::setMatri 
        mStatus = BAD_INDEX;

        return *this;

    }

    LOG_ALWAYS_FATAL_IF(!isfinite(dsdx) || !isfinite(dtdx) || !isfinite(dtdy) || !isfinite(dsdy),

                        "setMatrix called with infinite values");

    s->what |= layer_state_t::eMatrixChanged;

    layer_state_t::matrix22_t matrix;

    matrix.dsdx = dsdx;