From 8c6f3f6bd0c5edce7aa9bfd61b64dd2e7dc43deb Mon Sep 17 00:00:00 2001 From: Ana Krulec Date: Thu, 23 Jan 2020 15:48:01 -0800 Subject: SF: Remove refresh_rate_switching flag. ag/9294789 introduce a flag to only allow refresh rate switching on the devices that have that flag set to true, because it broke devices that didn't support multiple refresh rates, and Android TV. LocalDisplayAdapter sets the refresh rate to default when booting the phone, so no need for that in SF anymore. Test: Install on devices that doesn't have multiple refresh rates. Test: Run SF unittests. Test: Turn on all the flags, and run through set of examples: Video playback stays at 60. Camera stays at 60. Maps play at 60. Low brightness stays at 90. Swappy successfully switches between 60 & 90. Test: Turn off all the flags, and rerun the examples from previous. If applications requested the change, the change is honored, otherwise we stay at default (set by DM). Test: Ask Android TV team to test the patch. Bug: 148427603 Change-Id: I44085e91a74902d102bcf9f1491164cdb50650cc --- services/surfaceflinger/SurfaceFlingerProperties.cpp | 8 -------- 1 file changed, 8 deletions(-) (limited to 'services/surfaceflinger/SurfaceFlingerProperties.cpp') diff --git a/services/surfaceflinger/SurfaceFlingerProperties.cpp b/services/surfaceflinger/SurfaceFlingerProperties.cpp index b4716eb61e..768074a6cd 100644 --- a/services/surfaceflinger/SurfaceFlingerProperties.cpp +++ b/services/surfaceflinger/SurfaceFlingerProperties.cpp @@ -226,14 +226,6 @@ int64_t color_space_agnostic_dataspace(Dataspace defaultValue) { return static_cast(defaultValue); } -bool refresh_rate_switching(bool defaultValue) { - auto temp = SurfaceFlingerProperties::refresh_rate_switching(); - if (temp.has_value()) { - return *temp; - } - return defaultValue; -} - int32_t set_idle_timer_ms(int32_t defaultValue) { auto temp = SurfaceFlingerProperties::set_idle_timer_ms(); if (temp.has_value()) { -- cgit v1.2.3-59-g8ed1b