Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "AudioManager: relax exceptions for setCommunicationDevice" into tm-qpr-dev am: 04e08686b6

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

Change-Id: I89fba58f196025bfcc1f17da418893829cc521e5
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Eric Laurent
2023-01-26 14:03:57 +00:00
committed by Automerger Merge Worker
parent c7b0789e72 04e08686b6
commit f95482ef27
2 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
media/java/android/media/AudioManager.java
View File

@@ -7688,7 +7688,8 @@ public class AudioManager {
Objects.requireNonNull(device);
try {
if (device.getId() == 0) {
throw new IllegalArgumentException("In valid device: " + device);
Log.w(TAG, "setCommunicationDevice: device not found: " + device);
return false;
}
return getService().setCommunicationDevice(mICallBack, device.getId());
} catch (RemoteException e) {

3
services/core/java/com/android/server/audio/AudioService.java
View File

@@ -5897,7 +5897,8 @@ public class AudioService extends IAudioService.Stub
if (portId != 0) {
device = AudioManager.getDeviceForPortId(portId, AudioManager.GET_DEVICES_OUTPUTS);
if (device == null) {
throw new IllegalArgumentException("invalid portID " + portId);
Log.w(TAG, "setCommunicationDevice: invalid portID " + portId);
return false;
}
if (!isValidCommunicationDevice(device)) {
throw new IllegalArgumentException("invalid device type " + device.getType());