commit 2cb8b62ef3264c0b8b0429ba1a86d04307d31ab8
author Ady Abraham <adyabr@google.com> Mon Dec 02 18:55:33 2019 -0800
committer Ady Abraham <adyabr@google.com> Tue Dec 10 16:30:08 2019 -0800
tree b2b1ccb5f7756caf2cff7a5e3e17f1e3e6d12323
parent 3a77a7b627de68a015f778bdefc783aec99ff660

SurfaceFlinger: show refresh rate instead of color in overlay

Change the RefreshRateOverlay to show the actual refresh rate fps
instead of just a color.

Change-Id: Ifd6e419b1534736de466d67e9e95042e62c730c6
Test: adb shell service call SurfaceFlinger 1034 i32 1
Bug: 129297325

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 71ecf88..3403b28 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -4831,15 +4831,15 @@
                 return NO_ERROR;
             }
             case 1034: {
-                // TODO(b/129297325): expose this via developer menu option
                 n = data.readInt32();
-                if (n && !mRefreshRateOverlay &&
-                    mRefreshRateConfigs->refreshRateSwitchingSupported()) {
+                if (n == 1 && !mRefreshRateOverlay) {
                     mRefreshRateOverlay = std::make_unique<RefreshRateOverlay>(*this);
                     auto current = mRefreshRateConfigs->getCurrentRefreshRate();
                     mRefreshRateOverlay->changeRefreshRate(current);
-                } else if (!n) {
+                } else if (n == 0) {
                     mRefreshRateOverlay.reset();
+                } else {
+                    reply->writeBool(mRefreshRateOverlay != nullptr);
                 }
                 return NO_ERROR;
             }