From 515b50095b02ca1dcc652ae4f120bf116278050c Mon Sep 17 00:00:00 2001
From: Vlad Popa <pvlad@google.com>
Date: Thu, 16 Jan 2025 19:19:03 -0800
Subject: Use cache for min max volume getter

Using the IpcDataCache to reduce the number of binder calls between
AudioManager and AudioService. Especially for the get min max stream
volume this makes a lot of sense since they do not change that often.

Flag: android.media.audio.cache_get_stream_min_max_volume
Test: atest AudioManagerTest
Bug: 383667500
Change-Id: I040fb20e100099461a6fd31d10726f6ac3b4b02d
---
 .../android/audiopolicytest/AudioManagerTest.java  | 39 ++++++++++++++++++++++
 1 file changed, 39 insertions(+)

(limited to 'media/tests')

diff --git a/media/tests/AudioPolicyTest/src/com/android/audiopolicytest/AudioManagerTest.java b/media/tests/AudioPolicyTest/src/com/android/audiopolicytest/AudioManagerTest.java
index e9a0d3eceba3..017a1029d35c 100644
--- a/media/tests/AudioPolicyTest/src/com/android/audiopolicytest/AudioManagerTest.java
+++ b/media/tests/AudioPolicyTest/src/com/android/audiopolicytest/AudioManagerTest.java
@@ -16,6 +16,15 @@
 
 package com.android.audiopolicytest;
 
+import static android.media.AudioManager.STREAM_ACCESSIBILITY;
+import static android.media.AudioManager.STREAM_ALARM;
+import static android.media.AudioManager.STREAM_DTMF;
+import static android.media.AudioManager.STREAM_MUSIC;
+import static android.media.AudioManager.STREAM_NOTIFICATION;
+import static android.media.AudioManager.STREAM_RING;
+import static android.media.AudioManager.STREAM_SYSTEM;
+import static android.media.AudioManager.STREAM_VOICE_CALL;
+
 import static androidx.test.core.app.ApplicationProvider.getApplicationContext;
 
 import static com.android.audiopolicytest.AudioVolumeTestUtil.DEFAULT_ATTRIBUTES;
@@ -28,11 +37,15 @@ import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertThrows;
 import static org.junit.Assert.assertTrue;
 
+import android.content.Context;
 import android.media.AudioAttributes;
 import android.media.AudioManager;
 import android.media.AudioSystem;
+import android.media.IAudioService;
 import android.media.audiopolicy.AudioProductStrategy;
 import android.media.audiopolicy.AudioVolumeGroup;
+import android.os.IBinder;
+import android.os.ServiceManager;
 import android.platform.test.annotations.Presubmit;
 import android.util.Log;
 
@@ -206,6 +219,32 @@ public class AudioManagerTest {
         }
     }
 
+    //-----------------------------------------------------------------
+    // Test getStreamVolume consistency with AudioService
+    //-----------------------------------------------------------------
+    @Test
+    public void getStreamMinMaxVolume_consistentWithAs() throws Exception {
+        IBinder b = ServiceManager.getService(Context.AUDIO_SERVICE);
+        IAudioService service = IAudioService.Stub.asInterface(b);
+        final int[] streamTypes = {
+                STREAM_VOICE_CALL,
+                STREAM_SYSTEM,
+                STREAM_RING,
+                STREAM_MUSIC,
+                STREAM_ALARM,
+                STREAM_NOTIFICATION,
+                STREAM_DTMF,
+                STREAM_ACCESSIBILITY,
+        };
+
+        for (int streamType : streamTypes) {
+            assertEquals(service.getStreamMinVolume(streamType),
+                    mAudioManager.getStreamMinVolume(streamType));
+            assertEquals(service.getStreamMaxVolume(streamType),
+                    mAudioManager.getStreamMaxVolume(streamType));
+        }
+    }
+
     //-----------------------------------------------------------------
     // Test Volume per Attributes setter/getters
     //-----------------------------------------------------------------
-- 
cgit v1.2.3-59-g8ed1b