Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 8289cf50 authored by Raj Goparaju's avatar Raj Goparaju
Browse files

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
parent b28005e7

media/java/android/media/AudioManager.java

+1 −2
Original line number Diff line number Diff line
@@ -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) {