commit cba8d6c1795d803b35158211c2f9ef39b6a10ec8
author Ady Abraham <adyabr@google.com> Thu Jun 03 18:05:04 2021 -0700
committer Ady Abraham <adyabr@google.com> Thu Jun 24 18:57:38 2021 -0700
tree f5eeb4e844e5185a7cd6376e2f2b80a44519bd1c
parent 40e122da8f75c39d81c5e848a01c460348af3650

SF: bound refresh rate overlay size

The overlay is too big on larger displays.

Bug: 182909799
Test: Enable refresh rate overlay
Change-Id: Ie2eb39ba998aad199e7a5f7d57eff625d382fc68

services/surfaceflinger/RefreshRateOverlay.cpp

diff --git a/services/surfaceflinger/RefreshRateOverlay.cpp b/services/surfaceflinger/RefreshRateOverlay.cpp
index 663e62a..9cfc37d 100644
--- a/services/surfaceflinger/RefreshRateOverlay.cpp
+++ b/services/surfaceflinger/RefreshRateOverlay.cpp
@@ -240,8 +240,11 @@
 }
 
 void RefreshRateOverlay::setViewport(ui::Size viewport) {
-    Rect frame((3 * viewport.width) >> 4, viewport.height >> 5);
-    frame.offsetBy(viewport.width >> 5, viewport.height >> 4);
+    constexpr int32_t kMaxWidth = 1000;
+    const auto width = std::min(kMaxWidth, std::min(viewport.width, viewport.height));
+    const auto height = 2 * width;
+    Rect frame((3 * width) >> 4, height >> 5);
+    frame.offsetBy(width >> 5, height >> 4);
 
     layer_state_t::matrix22_t matrix;
     matrix.dsdx = frame.getWidth() / static_cast<float>(SevenSegmentDrawer::getWidth());