commit dd221b2978969ad094541a88c20a9bbe6090a048
author Yiwei Zhang <zzyiwei@google.com> Fri Jun 12 11:06:19 2020 -0700
committer Yiwei Zhang <zzyiwei@google.com> Mon Jun 15 23:08:06 2020 +0000
tree 4883831edd44ef0e9db9a28df514bf23f9a446e3
parent 5f43db07b38a2ff94b6ac85cbe15f99306431306

TimeStats: avoid inf value in averageFPS

Bug: 158285824
Test: atest libsurfaceflinger_unittest:TimeStatsTest
Change-Id: Ica9a472c8365e4f9d9a179090de538f75d5de91a

services/surfaceflinger/TimeStats/timestatsproto/TimeStatsHelper.cpp

diff --git a/services/surfaceflinger/TimeStats/timestatsproto/TimeStatsHelper.cpp b/services/surfaceflinger/TimeStats/timestatsproto/TimeStatsHelper.cpp
index c90b1b8..894ee6d 100644
--- a/services/surfaceflinger/TimeStats/timestatsproto/TimeStatsHelper.cpp
+++ b/services/surfaceflinger/TimeStats/timestatsproto/TimeStatsHelper.cpp
@@ -87,7 +87,9 @@
     StringAppendF(&result, "badDesiredPresentFrames = %d\n", badDesiredPresentFrames);
     const auto iter = deltas.find("present2present");
     if (iter != deltas.end()) {
-        StringAppendF(&result, "averageFPS = %.3f\n", 1000.0 / iter->second.averageTime());
+        const float averageTime = iter->second.averageTime();
+        const float averageFPS = averageTime < 1.0f ? 0.0f : 1000.0f / averageTime;
+        StringAppendF(&result, "averageFPS = %.3f\n", averageFPS);
     }
     for (const auto& ele : deltas) {
         StringAppendF(&result, "%s histogram is as below:\n", ele.first.c_str());