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

Commit e1dd1626 authored by Chavi Weingarten's avatar Chavi Weingarten Committed by Automerger Merge Worker
Browse files

Merge "Restore old behavior in pulic API setGeometry" into sc-dev am: 81aec8eb

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

Change-Id: I63320870a691583471c562cf019fbb089ed68ca0
parents bd88dc75 81aec8eb
Loading
Loading
Loading
Loading
+28 −11
Original line number Diff line number Diff line
@@ -438,27 +438,44 @@ void ASurfaceTransaction_setGeometry(ASurfaceTransaction* aSurfaceTransaction,
                                     const ARect& destination, int32_t transform) {
    CHECK_NOT_NULL(aSurfaceTransaction);
    CHECK_NOT_NULL(aSurfaceControl);
    CHECK_VALID_RECT(source);
    CHECK_VALID_RECT(destination);

    sp<SurfaceControl> surfaceControl = ASurfaceControl_to_SurfaceControl(aSurfaceControl);
    Transaction* transaction = ASurfaceTransaction_to_Transaction(aSurfaceTransaction);

    Rect sourceRect = static_cast<const Rect&>(source);
    Rect destRect = static_cast<const Rect&>(destination);
    // Adjust the source so its top and left are not negative
    sourceRect.left = std::max(sourceRect.left, 0);
    sourceRect.top = std::max(sourceRect.top, 0);
    LOG_ALWAYS_FATAL_IF(sourceRect.isEmpty(), "invalid arg passed as source argument");

    sp<SurfaceControl> surfaceControl = ASurfaceControl_to_SurfaceControl(aSurfaceControl);
    Transaction* transaction = ASurfaceTransaction_to_Transaction(aSurfaceTransaction);

    if (!sourceRect.isValid()) {
        sourceRect.makeInvalid();
    }
    transaction->setBufferCrop(surfaceControl, sourceRect);

    float dsdx = (destination.right - destination.left) /
            static_cast<float>(sourceRect.right - sourceRect.left);
    float dsdy = (destination.bottom - destination.top) /
            static_cast<float>(sourceRect.bottom - sourceRect.top);
    int destW = destRect.width();
    int destH = destRect.height();
    if (destRect.left < 0) {
        destRect.left = 0;
        destRect.right = destW;
    }
    if (destRect.top < 0) {
        destRect.top = 0;
        destRect.bottom = destH;
    }

    if (!sourceRect.isEmpty()) {
        float sx = destW / static_cast<float>(sourceRect.width());
        float sy = destH / static_cast<float>(sourceRect.height());
        transaction->setPosition(surfaceControl, destRect.left - (sourceRect.left * sx),
                                 destRect.top - (sourceRect.top * sy));
        transaction->setMatrix(surfaceControl, sx, 0, 0, sy);
    } else {
        transaction->setPosition(surfaceControl, destRect.left, destRect.top);
    }

    transaction->setPosition(surfaceControl, destination.left - (sourceRect.left * dsdx),
                             destination.top - (sourceRect.top * dsdy));
    transaction->setMatrix(surfaceControl, dsdx, 0, 0, dsdy);
    transaction->setTransform(surfaceControl, transform);
    bool transformToInverseDisplay = (NATIVE_WINDOW_TRANSFORM_INVERSE_DISPLAY & transform) ==
            NATIVE_WINDOW_TRANSFORM_INVERSE_DISPLAY;