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

Commit 0c252708 authored by Dominik Laskowski's avatar Dominik Laskowski
Browse files

SF: Clean up RefreshRateConfigs tests 

Finish renaming `config` to `mode`, and reduce repetition in preparation
of RefreshRateConfigs::RefreshRate removal.

Bug: 185535769
Bug: 129481165
Test: libsurfaceflinger_unittest
Change-Id: I9a87cec76387edbdfaeb31c1f24cff341489e45a
parent b13f2c8f

services/surfaceflinger/Scheduler/RefreshRateConfigs.h

+4 −4
Original line number Diff line number Diff line
@@ -310,6 +310,9 @@ public: 
                                        .idleTimerTimeoutMs = 0,

                                        .supportKernelIdleTimer = false});



    RefreshRateConfigs(const RefreshRateConfigs&) = delete;

    RefreshRateConfigs& operator=(const RefreshRateConfigs&) = delete;



    // Returns whether switching modes (refresh rate or resolution) is possible.

    // TODO(b/158780872): Consider HAL support, and skip frame rate detection if the modes only

    // differ in resolution.
@@ -391,11 +394,8 @@ public: 


    void dump(std::string& result) const EXCLUDES(mLock);



    RefreshRateConfigs(const RefreshRateConfigs&) = delete;

    void operator=(const RefreshRateConfigs&) = delete;



private:

    friend class RefreshRateConfigsTest;

    friend struct TestableRefreshRateConfigs;



    void constructAvailableRefreshRates() REQUIRES(mLock);

services/surfaceflinger/tests/unittests/RefreshRateConfigsTest.cpp

+665 −851

File changed.

Preview size limit exceeded, changes collapsed.