commit 01eeab93f38168153c42e8819221b5417aab79ab
author Vova Sharaienko <sharaienko@google.com> Wed Jun 08 22:55:02 2022 +0000
committer Vova Sharaienko <sharaienko@google.com> Thu Jun 09 02:32:17 2022 +0000
tree 31744a682056fcd99c506d4f508497b16f12039b
parent 7514e3b91a168996e006b3b69e2e6c51ef50a345

Fixed gpuservice_unittest mem leaks

  Activated looper to wait on binder transactions
  Binder transaction initiated by libstatspull on behalf of GpuStats
  See GpuStats::registerStatsdCallbacksIfNeeded()

Bug: 222527290
Test: atest gpuservice_unittest
Change-Id: I3797e7bb22f35eda84bc1196a448674e1123e046

services/gpuservice/tests/unittests/GpuStatsTest.cpp

diff --git a/services/gpuservice/tests/unittests/GpuStatsTest.cpp b/services/gpuservice/tests/unittests/GpuStatsTest.cpp
index 0baf1f9..3c7644f 100644
--- a/services/gpuservice/tests/unittests/GpuStatsTest.cpp
+++ b/services/gpuservice/tests/unittests/GpuStatsTest.cpp
@@ -25,6 +25,7 @@
 #include <gtest/gtest.h>
 #include <stats_pull_atom_callback.h>
 #include <statslog.h>
+#include <utils/Looper.h>
 #include <utils/String16.h>
 #include <utils/Vector.h>
 
@@ -62,8 +63,9 @@
 // clang-format on
 
 class GpuStatsTest : public testing::Test {
+    sp<android::Looper> looper;
 public:
-    GpuStatsTest() {
+    GpuStatsTest() : looper(Looper::prepare(0 /* opts */)) {
         const ::testing::TestInfo* const test_info =
                 ::testing::UnitTest::GetInstance()->current_test_info();
         ALOGD("**** Setting up for %s.%s\n", test_info->test_case_name(), test_info->name());
@@ -73,6 +75,10 @@
         const ::testing::TestInfo* const test_info =
                 ::testing::UnitTest::GetInstance()->current_test_info();
         ALOGD("**** Tearing down after %s.%s\n", test_info->test_case_name(), test_info->name());
+
+        // performs all pending callbacks until all data has been consumed
+        // gives time to process binder transactions by thread pool
+        looper->pollAll(1000);
     }
 
     std::string inputCommand(InputCommand cmd);