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 am: f95482ef27

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

Change-Id: I0db4d8e11f3d8236d64f64d531f32871dee4718e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Eric Laurent
2023-01-26 15:07:52 +00:00
committed by Automerger Merge Worker
parent 806ee500de f95482ef27
commit c129e33dba
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

@@ -8329,7 +8329,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

@@ -6118,7 +6118,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)) {
if (!device.isSink()) {