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/tests/unittests/TimeStatsTest.cpp

diff --git a/services/surfaceflinger/tests/unittests/TimeStatsTest.cpp b/services/surfaceflinger/tests/unittests/TimeStatsTest.cpp
index 7a1c7c6..63a34af 100644
--- a/services/surfaceflinger/tests/unittests/TimeStatsTest.cpp
+++ b/services/surfaceflinger/tests/unittests/TimeStatsTest.cpp
@@ -833,6 +833,15 @@
     ASSERT_EQ(0, globalProto.stats_size());
 }
 
+TEST_F(TimeStatsTest, noInfInAverageFPS) {
+    EXPECT_TRUE(inputCommand(InputCommand::ENABLE, FMT_STRING).empty());
+    insertTimeRecord(NORMAL_SEQUENCE, LAYER_ID_0, 1, 1000000);
+    insertTimeRecord(NORMAL_SEQUENCE, LAYER_ID_0, 2, 1000000);
+
+    const std::string result(inputCommand(InputCommand::DUMP_ALL, FMT_STRING));
+    EXPECT_THAT(result, HasSubstr("averageFPS = 0.000"));
+}
+
 namespace {
 std::string buildExpectedHistogramBytestring(const std::vector<int32_t>& times,
                                              const std::vector<int32_t>& frameCounts) {