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

Commit 4119e227 authored by Sally Qi's avatar Sally Qi
Browse files

[SF] Change getOverlaySupport function body due to graphics AIDL 

interface change.

- Due to splitting dataspace into {stardard, transfer, range} tuple in
  aidl interace.

Bug: 242588489
Test: build and flash
Change-Id: Iaa50bd5fdeb69ae37279a4ad9d29afc487ca5a39
parent 470c15ab

libs/gui/aidl/android/gui/OverlayProperties.aidl

+3 −1
Original line number Diff line number Diff line
@@ -20,7 +20,9 @@ package android.gui; 
parcelable OverlayProperties {

    parcelable SupportedBufferCombinations {

        int[] pixelFormats;

        int[] dataspaces;

        int[] standards;

        int[] transfers;

        int[] ranges;

    }

    SupportedBufferCombinations[] combinations;

services/surfaceflinger/SurfaceFlinger.cpp

+17 −5
Original line number Diff line number Diff line
@@ -1482,14 +1482,26 @@ status_t SurfaceFlinger::getOverlaySupport(gui::OverlayProperties* outProperties 
        std::transform(combination.pixelFormats.cbegin(), combination.pixelFormats.cend(),

                       std::back_inserter(pixelFormats),

                       [](const auto& val) { return static_cast<int32_t>(val); });

        std::vector<int32_t> dataspaces;

        dataspaces.reserve(combination.dataspaces.size());

        std::transform(combination.dataspaces.cbegin(), combination.dataspaces.cend(),

                       std::back_inserter(dataspaces),

        std::vector<int32_t> standards;

        standards.reserve(combination.standards.size());

        std::transform(combination.standards.cbegin(), combination.standards.cend(),

                       std::back_inserter(standards),

                       [](const auto& val) { return static_cast<int32_t>(val); });

        std::vector<int32_t> transfers;

        transfers.reserve(combination.transfers.size());

        std::transform(combination.transfers.cbegin(), combination.transfers.cend(),

                       std::back_inserter(transfers),

                       [](const auto& val) { return static_cast<int32_t>(val); });

        std::vector<int32_t> ranges;

        ranges.reserve(combination.ranges.size());

        std::transform(combination.ranges.cbegin(), combination.ranges.cend(),

                       std::back_inserter(ranges),

                       [](const auto& val) { return static_cast<int32_t>(val); });

        gui::OverlayProperties::SupportedBufferCombinations outCombination;

        outCombination.pixelFormats = std::move(pixelFormats);

        outCombination.dataspaces = std::move(dataspaces);

        outCombination.standards = std::move(standards);

        outCombination.transfers = std::move(transfers);

        outCombination.ranges = std::move(ranges);

        outProperties->combinations.emplace_back(outCombination);

    }

    outProperties->supportMixedColorSpaces = aidlProperties.supportMixedColorSpaces;