commit 4a92f55e56693fe21b0e930ba34477cf887a1103
author Marin Shalamanov <shalamanov@google.com> Mon Mar 29 18:38:27 2021 +0200
committer Marin Shalamanov <shalamanov@google.com> Tue Mar 30 14:33:50 2021 +0200
tree 46d85c069226d9e28432d886f47cca88e8551cae
parent ae0b5356aaf26ffae35b082975a2023f497decc9

FramebufferSurface: fix for limitFramebufferSize

This CL fixes limitFramebufferSize and adds a test for the
broken use case. Also limitFramebufferSize is renamed to
limitSize  because "Framebuffer" is implied by the class
name. The implementation is moved to a static
limitSizeInternal which can be tested more easily.

Bug: 180908183
Test: atest FramebufferSurfaceTest
Change-Id: Ic094b36fef6f9a17c87ec95a6c3f5f0a09c99d2f

services/surfaceflinger/tests/unittests/Android.bp

diff --git a/services/surfaceflinger/tests/unittests/Android.bp b/services/surfaceflinger/tests/unittests/Android.bp
index 3c1b9d8..9f94c73 100644
--- a/services/surfaceflinger/tests/unittests/Android.bp
+++ b/services/surfaceflinger/tests/unittests/Android.bp
@@ -55,6 +55,7 @@
         "EventThreadTest.cpp",
         "FpsReporterTest.cpp",
         "FpsTest.cpp",
+        "FramebufferSurfaceTest.cpp",
         "FrameTimelineTest.cpp",
         "HWComposerTest.cpp",
         "OneShotTimerTest.cpp",