From afd850182d79df08b03bbc24eb2b070980fb2826 Mon Sep 17 00:00:00 2001 From: Alec Mouri Date: Thu, 15 May 2025 16:39:49 +0000 Subject: [SP 2025-09-01] Don't blur too many layers An application requesting lots and lots of blurs: a. Enables pixel stealing by measuring how long it takes to perform a blur across windows b. Probably isn't very valid anyways. So, just arbitrarily pick an upper bound for blur requests that a display is allowed to manage (10), and disable everything else. Arbitrarily, pick the 10 "front-most" blurs to be respected. Bug: 399120953 Flag: EXEMPT security Test: Security PoC no longer PoCs (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:fbcb9ae5eb45e2273be05d5366b47bd8436c1718) Merged-In: Ie7195eb852b52aff2f58da8bd095d8684baceef6 Change-Id: Ie7195eb852b52aff2f58da8bd095d8684baceef6 --- .../surfaceflinger/CompositionEngine/tests/planner/CachedSetTest.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'services/surfaceflinger/CompositionEngine/tests') diff --git a/services/surfaceflinger/CompositionEngine/tests/planner/CachedSetTest.cpp b/services/surfaceflinger/CompositionEngine/tests/planner/CachedSetTest.cpp index d61d7ba574..5b33407146 100644 --- a/services/surfaceflinger/CompositionEngine/tests/planner/CachedSetTest.cpp +++ b/services/surfaceflinger/CompositionEngine/tests/planner/CachedSetTest.cpp @@ -1011,12 +1011,12 @@ TEST_F(CachedSetTest, addBlur) { EXPECT_CALL(*layerFE1, prepareClientComposition(ClientCompositionTargetSettingsBlurSettingsEq( compositionengine::LayerFE::ClientCompositionTargetSettings::BlurSetting:: - Enabled))) + Disabled))) .WillOnce(Return(clientComp1)); EXPECT_CALL(*layerFE2, prepareClientComposition(ClientCompositionTargetSettingsBlurSettingsEq( compositionengine::LayerFE::ClientCompositionTargetSettings::BlurSetting:: - Enabled))) + Disabled))) .WillOnce(Return(clientComp2)); EXPECT_CALL(*layerFE3, prepareClientComposition(ClientCompositionTargetSettingsBlurSettingsEq( -- cgit v1.2.3-59-g8ed1b