commit ea25ddfe44e3aaa1182761504c13ec0594d1d9ed
author Marin Shalamanov <shalamanov@google.com> Thu Dec 10 15:44:51 2020 +0100
committer Marin Shalamanov <shalamanov@google.com> Thu Dec 24 15:57:01 2020 +0100
tree 4b35fb066d3250f1581eb0ecffc4f5bdf214ced6
parent 9149533a4ad89ca71bb4f9554740ee0ecc1531ad

SF: Remove RefreshRateConfigs::getAllRefreshRates()

Remove this function since it's (almost) unused.

Test: adb shell service call SurfaceFlinger 1035 i32 5
Bug: 159590486
Change-Id: Iaec883eb2b29fd8064885dcb4e590c0e7b6021cf

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 68d2a68..5d08328 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -5309,12 +5309,17 @@
                 return NO_ERROR;
             }
             case 1035: {
-                n = data.readInt32();
+                const int newConfigId = data.readInt32();
                 mDebugDisplayConfigSetByBackdoor = false;
-                const auto numConfigs = mRefreshRateConfigs->getAllRefreshRates().size();
-                if (n >= 0 && n < numConfigs) {
+                const auto displayId = getInternalDisplayId();
+                if (!displayId) {
+                    ALOGE("No internal display found.");
+                    return NO_ERROR;
+                }
+                const auto numConfigs = getHwComposer().getConfigs(*displayId).size();
+                if (newConfigId >= 0 && newConfigId < numConfigs) {
                     const auto displayToken = getInternalDisplayToken();
-                    status_t result = setActiveConfig(displayToken, n);
+                    status_t result = setActiveConfig(displayToken, newConfigId);
                     if (result != NO_ERROR) {
                         return result;
                     }