Address review comments for dispatch audio focus with fade API

- Add IntDef definition, @FocusRequestResult, for the result
  value.
- Remove redundant null pointer exception details from the
  Javadoc, as this is documented by the @NonNull annotation.

Bug: 318430127
Bug: 186905459
API-Coverage-Bug: 308666800
Test: atest -c FadeManagerConfigurationUnitTest
Change-Id: I0e7b8944b75b875317d727d4d62155153259e688

1 files changed, 1 insertions, 2 deletions

diff --git a/media/java/android/media/AudioManager.java b/media/java/android/media/AudioManager.java
index 4918289e8b5c..72d87253d404 100644
--- a/media/java/android/media/AudioManager.java
+++ b/media/java/android/media/AudioManager.java
@@ -5163,13 +5163,12 @@ public class AudioManager {
      *     dispatch was successfully sent, or {@link #AUDIOFOCUS_REQUEST_DELAYED} if
      *     the request was successful but the dispatch of focus change was delayed due to a fade
      *     operation.
-     * @throws NullPointerException if the {@link AudioFocusInfo} or {@link AudioPolicy} or list of
-     *     other active {@link AudioFocusInfo} are {@code null}.
      * @hide
      */
     @FlaggedApi(FLAG_ENABLE_FADE_MANAGER_CONFIGURATION)
     @SystemApi
     @RequiresPermission(Manifest.permission.MODIFY_AUDIO_SETTINGS_PRIVILEGED)
+    @FocusRequestResult
     public int dispatchAudioFocusChangeWithFade(@NonNull AudioFocusInfo afi, int focusChange,
             @NonNull AudioPolicy ap, @NonNull List<AudioFocusInfo> otherActiveAfis,
             @Nullable FadeManagerConfiguration transientFadeMgrConfig) {