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

Commit b5fa4dff authored by Jean-Michel Trivi's avatar Jean-Michel Trivi Committed by Android (Google) Code Review
Browse files

Merge "Fix bug 2594126 Do not crash when ViewRoot is trying to play a UI... 

Merge "Fix bug 2594126 Do not crash when ViewRoot is trying to play a UI sound. This CL doesn't fix why mView can be null in the first place, but prevents a crash when mView is null and a UI sound is to be played." into froyo
parents a69e10db 13b18fd7

core/java/android/view/ViewRoot.java

+26 −20
Original line number Diff line number Diff line
@@ -2580,6 +2580,7 @@ public final class ViewRoot extends Handler implements ViewParent, 
    public void playSoundEffect(int effectId) {

        checkThread();



        try {

            final AudioManager audioManager = getAudioManager();



            switch (effectId) {
@@ -2602,6 +2603,11 @@ public final class ViewRoot extends Handler implements ViewParent, 
                    throw new IllegalArgumentException("unknown effect id " + effectId +

                            " not defined in " + SoundEffectConstants.class.getCanonicalName());

            }

        } catch (IllegalStateException e) {

            // Exception thrown by getAudioManager() when mView is null

            Log.e(TAG, "FATAL EXCEPTION when attempting to play sound effect: " + e);

            e.printStackTrace();

        }

    }



    /**