Merge "AudioService: fix volume group setting name" into tm-qpr-dev (74e8c779) · Commits · e / os / android_frameworks_base

services/core/java/com/android/server/audio/AudioService.java

+32 −6

Original line number	Diff line number	Diff line
		@@ -2251,8 +2251,8 @@ public class AudioService extends IAudioService.Stub
		synchronized (VolumeStreamState.class) {
		mStreamStates[AudioSystem.STREAM_DTMF]
		.setAllIndexes(mStreamStates[dtmfStreamAlias], caller);
		mStreamStates[AudioSystem.STREAM_ACCESSIBILITY].mVolumeIndexSettingName =
		System.VOLUME_SETTINGS_INT[a11yStreamAlias];
		mStreamStates[AudioSystem.STREAM_ACCESSIBILITY].setSettingName(
		System.VOLUME_SETTINGS_INT[a11yStreamAlias]);
		mStreamStates[AudioSystem.STREAM_ACCESSIBILITY].setAllIndexes(
		mStreamStates[a11yStreamAlias], caller);
		}
		@@ -7505,7 +7505,7 @@ public class AudioService extends IAudioService.Stub
		private int mPublicStreamType = AudioSystem.STREAM_MUSIC;
		private AudioAttributes mAudioAttributes = AudioProductStrategy.getDefaultAttributes();
		private boolean mIsMuted = false;
		private final String mSettingName;
		private String mSettingName;

		// No API in AudioSystem to get a device from strategy or from attributes.
		// Need a valid public stream type to use current API getDeviceForStream
		@@ -7834,15 +7834,19 @@ public class AudioService extends IAudioService.Stub
		}

		private void persistVolumeGroup(int device) {
		if (mUseFixedVolume) {
		// No need to persist the index if the volume group is backed up
		// by a public stream type as this is redundant
		if (mUseFixedVolume \|\| mHasValidStreamType) {
		return;
		}
		if (DEBUG_VOL) {
		Log.v(TAG, "persistVolumeGroup: storing index " + getIndex(device) + " for group "
		+ mAudioVolumeGroup.name()
		+ ", device " + AudioSystem.getOutputDeviceName(device)
		+ " and User=" + getCurrentUserId());
		+ " and User=" + getCurrentUserId()
		+ " mSettingName: " + mSettingName);
		}

		boolean success = mSettings.putSystemIntForUser(mContentResolver,
		getSettingNameForDevice(device),
		getIndex(device),
		@@ -7905,6 +7909,14 @@ public class AudioService extends IAudioService.Stub
		return mSettingName + "_" + AudioSystem.getOutputDeviceName(device);
		}

		void setSettingName(String settingName) {
		mSettingName = settingName;
		}

		String getSettingName() {
		return mSettingName;
		}

		private void dump(PrintWriter pw) {
		pw.println("- VOLUME GROUP " + mAudioVolumeGroup.name() + ":");
		pw.print(" Muted: ");
		@@ -8029,6 +8041,9 @@ public class AudioService extends IAudioService.Stub
		*/
		public void setVolumeGroupState(VolumeGroupState volumeGroupState) {
		mVolumeGroupState = volumeGroupState;
		if (mVolumeGroupState != null) {
		mVolumeGroupState.setSettingName(mVolumeIndexSettingName);
		}
		}
		/**
		* Update the minimum index that can be used without MODIFY_AUDIO_SETTINGS permission
		@@ -8099,6 +8114,17 @@ public class AudioService extends IAudioService.Stub
		return (mVolumeIndexSettingName != null && !mVolumeIndexSettingName.isEmpty());
		}

		void setSettingName(String settingName) {
		mVolumeIndexSettingName = settingName;
		if (mVolumeGroupState != null) {
		mVolumeGroupState.setSettingName(mVolumeIndexSettingName);
		}
		}

		String getSettingName() {
		return mVolumeIndexSettingName;
		}

		public void readSettings() {
		synchronized (mSettingsLock) {
		synchronized (VolumeStreamState.class) {