diff options
3 files changed, 110 insertions, 13 deletions
diff --git a/services/core/java/com/android/server/app/GameManagerService.java b/services/core/java/com/android/server/app/GameManagerService.java index 62e2a9b8d40c..4013acefa366 100644 --- a/services/core/java/com/android/server/app/GameManagerService.java +++ b/services/core/java/com/android/server/app/GameManagerService.java @@ -1324,7 +1324,7 @@ public final class GameManagerService extends IGameManagerService.Stub { try { final float fps = 0.0f; final int uid = mPackageManager.getPackageUidAsUser(packageName, userId); - nativeSetOverrideFrameRate(uid, fps); + setOverrideFrameRate(uid, fps); } catch (PackageManager.NameNotFoundException e) { return; } @@ -1410,7 +1410,7 @@ public final class GameManagerService extends IGameManagerService.Stub { try { final float fps = modeConfig.getFps(); final int uid = mPackageManager.getPackageUidAsUser(packageName, userId); - nativeSetOverrideFrameRate(uid, fps); + setOverrideFrameRate(uid, fps); } catch (PackageManager.NameNotFoundException e) { return; } @@ -1419,20 +1419,16 @@ public final class GameManagerService extends IGameManagerService.Stub { private void updateInterventions(String packageName, @GameMode int gameMode, @UserIdInt int userId) { + final GamePackageConfiguration packageConfig = getConfig(packageName); if (gameMode == GameManager.GAME_MODE_STANDARD - || gameMode == GameManager.GAME_MODE_UNSUPPORTED) { + || gameMode == GameManager.GAME_MODE_UNSUPPORTED || packageConfig == null + || packageConfig.willGamePerformOptimizations(gameMode)) { disableCompatScale(packageName); resetFps(packageName, userId); - return; - } - final GamePackageConfiguration packageConfig = getConfig(packageName); - if (packageConfig == null) { - disableCompatScale(packageName); - Slog.v(TAG, "Package configuration not found for " + packageName); - return; - } - if (packageConfig.willGamePerformOptimizations(gameMode)) { - return; + if (packageConfig == null) { + Slog.v(TAG, "Package configuration not found for " + packageName); + return; + } } updateCompatModeDownscale(packageConfig, packageName, gameMode); updateFps(packageConfig, packageName, gameMode, userId); @@ -1876,6 +1872,11 @@ public final class GameManagerService extends IGameManagerService.Stub { return handlerThread; } + @VisibleForTesting + void setOverrideFrameRate(int uid, float frameRate) { + nativeSetOverrideFrameRate(uid, frameRate); + } + /** * load dynamic library for frame rate overriding JNI calls */ diff --git a/services/tests/mockingservicestests/res/xml/game_manager_service_metadata_config_interventions_enabled_all_opt_in.xml b/services/tests/mockingservicestests/res/xml/game_manager_service_metadata_config_interventions_enabled_all_opt_in.xml new file mode 100644 index 000000000000..96d28785ba0a --- /dev/null +++ b/services/tests/mockingservicestests/res/xml/game_manager_service_metadata_config_interventions_enabled_all_opt_in.xml @@ -0,0 +1,9 @@ +<?xml version="1.0" encoding="UTF-8"?> +<game-mode-config + xmlns:android="http://schemas.android.com/apk/res/android" + android:supportsPerformanceGameMode="true" + android:supportsBatteryGameMode="true" + android:allowGameAngleDriver="true" + android:allowGameDownscaling="true" + android:allowGameFpsOverride="true" +/>
\ No newline at end of file diff --git a/services/tests/mockingservicestests/src/com/android/server/app/GameManagerServiceTests.java b/services/tests/mockingservicestests/src/com/android/server/app/GameManagerServiceTests.java index adc4b843e24c..cfb80148f166 100644 --- a/services/tests/mockingservicestests/src/com/android/server/app/GameManagerServiceTests.java +++ b/services/tests/mockingservicestests/src/com/android/server/app/GameManagerServiceTests.java @@ -68,7 +68,9 @@ import org.junit.After; import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; +import org.mockito.ArgumentMatchers; import org.mockito.Mock; +import org.mockito.Mockito; import org.mockito.MockitoSession; import org.mockito.quality.Strictness; @@ -88,6 +90,7 @@ public class GameManagerServiceTests { private static final String PACKAGE_NAME_INVALID = "com.android.app"; private static final int USER_ID_1 = 1001; private static final int USER_ID_2 = 1002; + private static final int DEFAULT_PACKAGE_UID = 12345; private MockitoSession mMockingSession; private String mPackageName; @@ -195,6 +198,8 @@ public class GameManagerServiceTests { .thenReturn(packages); when(mMockPackageManager.getApplicationInfoAsUser(anyString(), anyInt(), anyInt())) .thenReturn(applicationInfo); + when(mMockPackageManager.getPackageUidAsUser(mPackageName, USER_ID_1)).thenReturn( + DEFAULT_PACKAGE_UID); LocalServices.addService(PowerManagerInternal.class, mMockPowerManager); } @@ -375,6 +380,12 @@ public class GameManagerServiceTests { "res/xml/game_manager_service_metadata_config_interventions_enabled_no_opt_in.xml"); } + private void mockInterventionsEnabledAllOptInFromXml() throws Exception { + seedGameManagerServiceMetaDataFromFile(mPackageName, 123, + "res/xml/game_manager_service_metadata_config_interventions_enabled_all_opt_in" + + ".xml"); + } + private void mockInterventionsDisabledNoOptInFromXml() throws Exception { seedGameManagerServiceMetaDataFromFile(mPackageName, 123, "res/xml/game_manager_service_metadata_config_interventions_disabled_no_opt_in" @@ -1477,4 +1488,80 @@ public class GameManagerServiceTests { assertEquals(splitLine[6], "angle=0,scaling=0.7,fps=30"); } + + @Test + public void testResetInterventions_onDeviceConfigReset() throws Exception { + mockModifyGameModeGranted(); + String configStringBefore = + "mode=2,downscaleFactor=1.0,fps=90"; + when(DeviceConfig.getProperty(anyString(), anyString())) + .thenReturn(configStringBefore); + mockInterventionsEnabledNoOptInFromXml(); + GameManagerService gameManagerService = Mockito.spy(new GameManagerService(mMockContext, + mTestLooper.getLooper())); + startUser(gameManagerService, USER_ID_1); + gameManagerService.setGameMode(mPackageName, GameManager.GAME_MODE_PERFORMANCE, USER_ID_1); + Mockito.verify(gameManagerService).setOverrideFrameRate( + ArgumentMatchers.eq(DEFAULT_PACKAGE_UID), + ArgumentMatchers.eq(90.0f)); + checkFps(gameManagerService, GameManager.GAME_MODE_PERFORMANCE, 90); + + String configStringAfter = ""; + when(DeviceConfig.getProperty(anyString(), anyString())) + .thenReturn(configStringAfter); + gameManagerService.updateConfigsForUser(USER_ID_1, false, mPackageName); + Mockito.verify(gameManagerService).setOverrideFrameRate( + ArgumentMatchers.eq(DEFAULT_PACKAGE_UID), + ArgumentMatchers.eq(0.0f)); + } + + @Test + public void testResetInterventions_onInterventionsDisabled() throws Exception { + mockModifyGameModeGranted(); + String configStringBefore = + "mode=2,downscaleFactor=1.0,fps=90"; + when(DeviceConfig.getProperty(anyString(), anyString())) + .thenReturn(configStringBefore); + mockInterventionsEnabledNoOptInFromXml(); + GameManagerService gameManagerService = Mockito.spy(new GameManagerService(mMockContext, + mTestLooper.getLooper())); + startUser(gameManagerService, USER_ID_1); + gameManagerService.setGameMode(mPackageName, GameManager.GAME_MODE_PERFORMANCE, USER_ID_1); + Mockito.verify(gameManagerService).setOverrideFrameRate( + ArgumentMatchers.eq(DEFAULT_PACKAGE_UID), + ArgumentMatchers.eq(90.0f)); + checkFps(gameManagerService, GameManager.GAME_MODE_PERFORMANCE, 90); + + mockInterventionsDisabledNoOptInFromXml(); + gameManagerService.updateConfigsForUser(USER_ID_1, false, mPackageName); + Mockito.verify(gameManagerService).setOverrideFrameRate( + ArgumentMatchers.eq(DEFAULT_PACKAGE_UID), + ArgumentMatchers.eq(0.0f)); + checkFps(gameManagerService, GameManager.GAME_MODE_PERFORMANCE, 0); + } + + @Test + public void testResetInterventions_onGameModeOptedIn() throws Exception { + mockModifyGameModeGranted(); + String configStringBefore = + "mode=2,downscaleFactor=1.0,fps=90"; + when(DeviceConfig.getProperty(anyString(), anyString())) + .thenReturn(configStringBefore); + mockInterventionsEnabledNoOptInFromXml(); + GameManagerService gameManagerService = Mockito.spy(new GameManagerService(mMockContext, + mTestLooper.getLooper())); + startUser(gameManagerService, USER_ID_1); + + gameManagerService.setGameMode(mPackageName, GameManager.GAME_MODE_PERFORMANCE, USER_ID_1); + Mockito.verify(gameManagerService).setOverrideFrameRate( + ArgumentMatchers.eq(DEFAULT_PACKAGE_UID), + ArgumentMatchers.eq(90.0f)); + checkFps(gameManagerService, GameManager.GAME_MODE_PERFORMANCE, 90); + + mockInterventionsEnabledAllOptInFromXml(); + gameManagerService.updateConfigsForUser(USER_ID_1, false, mPackageName); + Mockito.verify(gameManagerService).setOverrideFrameRate( + ArgumentMatchers.eq(DEFAULT_PACKAGE_UID), + ArgumentMatchers.eq(0.0f)); + } } |