Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b2c7bb9e authored by Tim Van Patten's avatar Tim Van Patten Committed by Automerger Merge Worker
Browse files

Merge "Rename Settings to GameManagerSettings" into sc-dev am: 1db78385

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/13536005

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: Ide97c336770084fb0c2116416f23564cd4776e71
parents e3814089 1db78385
Loading
Loading
Loading
Loading
+7 −6
Original line number Diff line number Diff line
@@ -61,7 +61,7 @@ public final class GameManagerService extends IGameManagerService.Stub {
    private final Object mLock = new Object();
    private final Handler mHandler;
    @GuardedBy("mLock")
    private final ArrayMap<Integer, Settings> mSettings = new ArrayMap<>();
    private final ArrayMap<Integer, GameManagerSettings> mSettings = new ArrayMap<>();

    public GameManagerService(Context context) {
        this(context, createServiceThread().getLooper());
@@ -99,7 +99,7 @@ public final class GameManagerService extends IGameManagerService.Stub {
                    synchronized (mLock) {
                        removeMessages(WRITE_SETTINGS, msg.obj);
                        if (mSettings.containsKey(userId)) {
                            Settings userSettings = mSettings.get(userId);
                            GameManagerSettings userSettings = mSettings.get(userId);
                            userSettings.writePersistentDataLocked();
                        }
                    }
@@ -123,7 +123,7 @@ public final class GameManagerService extends IGameManagerService.Stub {
                        removeMessages(WRITE_SETTINGS, msg.obj);
                        removeMessages(REMOVE_SETTINGS, msg.obj);
                        if (mSettings.containsKey(userId)) {
                            final Settings userSettings = mSettings.get(userId);
                            final GameManagerSettings userSettings = mSettings.get(userId);
                            mSettings.remove(userId);
                            userSettings.writePersistentDataLocked();
                        }
@@ -190,7 +190,7 @@ public final class GameManagerService extends IGameManagerService.Stub {
            if (!mSettings.containsKey(userId)) {
                return GameManager.GAME_MODE_UNSUPPORTED;
            }
            Settings userSettings = mSettings.get(userId);
            GameManagerSettings userSettings = mSettings.get(userId);
            return userSettings.getGameModeLocked(packageName);
        }
    }
@@ -211,7 +211,7 @@ public final class GameManagerService extends IGameManagerService.Stub {
            if (!mSettings.containsKey(userId)) {
                return;
            }
            Settings userSettings = mSettings.get(userId);
            GameManagerSettings userSettings = mSettings.get(userId);
            userSettings.setGameModeLocked(packageName, gameMode);
            final Message msg = mHandler.obtainMessage(WRITE_SETTINGS);
            msg.obj = userId;
@@ -235,7 +235,8 @@ public final class GameManagerService extends IGameManagerService.Stub {
                return;
            }

            Settings userSettings = new Settings(Environment.getDataSystemDeDirectory(userId));
            GameManagerSettings userSettings =
                    new GameManagerSettings(Environment.getDataSystemDeDirectory(userId));
            mSettings.put(userId, userSettings);
            userSettings.readPersistentDataLocked();
        }
+3 −2
Original line number Diff line number Diff line
@@ -41,7 +41,7 @@ import java.util.Map;
 * Persists all GameService related settings.
 * @hide
 */
public class Settings {
public class GameManagerSettings {

    // The XML file follows the below format:
    // <?xml>
@@ -63,7 +63,7 @@ public class Settings {
    // PackageName -> GameMode
    private final ArrayMap<String, Integer> mGameModes = new ArrayMap<>();

    Settings(File dataDir) {
    GameManagerSettings(File dataDir) {
        mSystemDir = new File(dataDir, "system");
        mSystemDir.mkdirs();
        FileUtils.setPermissions(mSystemDir.toString(),
@@ -144,6 +144,7 @@ public class Settings {
            int type;
            while ((type = parser.next()) != XmlPullParser.START_TAG
                    && type != XmlPullParser.END_DOCUMENT) {
                // Do nothing
            }
            if (type != XmlPullParser.START_TAG) {
                Slog.wtf(GameManagerService.TAG,
+3 −3
Original line number Diff line number Diff line
@@ -90,7 +90,7 @@ public class GameManagerServiceSettingsTests {
        writeGameServiceXml();
    }

    private void verifyGameServiceSettingsData(Settings settings) {
    private void verifyGameServiceSettingsData(GameManagerSettings settings) {
        assertThat(settings.getGameModeLocked(PACKAGE_NAME_1), is(1));
        assertThat(settings.getGameModeLocked(PACKAGE_NAME_2), is(2));
        assertThat(settings.getGameModeLocked(PACKAGE_NAME_3), is(3));
@@ -107,7 +107,7 @@ public class GameManagerServiceSettingsTests {
        /* write out files and read */
        writeOldFiles();
        final Context context = InstrumentationRegistry.getContext();
        Settings settings = new Settings(context.getFilesDir());
        GameManagerSettings settings = new GameManagerSettings(context.getFilesDir());
        assertThat(settings.readPersistentDataLocked(), is(true));
        verifyGameServiceSettingsData(settings);
    }
@@ -118,7 +118,7 @@ public class GameManagerServiceSettingsTests {
        // write out files and read
        writeOldFiles();
        final Context context = InstrumentationRegistry.getContext();
        Settings settings = new Settings(context.getFilesDir());
        GameManagerSettings settings = new GameManagerSettings(context.getFilesDir());
        assertThat(settings.readPersistentDataLocked(), is(true));

        // write out, read back in and verify the same