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

Commit bf82514b authored by lijilou's avatar lijilou Committed by Felix Stern
Browse files

fix the NPE problem due to add null InsetsControlTarget object to map . 

We may add null  InsetsControlTarget object to map when call the onPostLayout method in InsetsSourceProvider java file.

Test: OEM monkey test
Flag: EXEMPT bugfix
Bug: 385565551
Change-Id: I466df1d0229b2f71fbed30e88ec7edd5d44370fe
Merged-In: I8be582c42c6ba7524a3d6edc0a36aa4856e10e71
parent a17154f1

services/core/java/com/android/server/wm/ImeInsetsSourceProvider.java

+2 −1
Original line number Diff line number Diff line
@@ -100,7 +100,8 @@ final class ImeInsetsSourceProvider extends InsetsSourceProvider { 
            // isLeashReadyForDispatching (used to dispatch the leash of the control) is

            // depending on mGivenInsetsReady. Therefore, triggering notifyControlChanged here

            // again, so that the control with leash can be eventually dispatched

            if (!mGivenInsetsReady && mServerVisible && !givenInsetsPending) {

            if (!mGivenInsetsReady && mServerVisible && !givenInsetsPending

                    && mControlTarget != null) {

                mGivenInsetsReady = true;

                ImeTracker.forLogging().onProgress(mStatsToken,

                        ImeTracker.PHASE_WM_POST_LAYOUT_NOTIFY_CONTROLS_CHANGED);

services/core/java/com/android/server/wm/InsetsSourceProvider.java

+1 −1
Original line number Diff line number Diff line
@@ -379,7 +379,7 @@ class InsetsSourceProvider { 
        final boolean serverVisibleChanged = mServerVisible != isServerVisible;

        setServerVisible(isServerVisible);

        final boolean positionChanged = updateInsetsControlPosition(windowState);

        if (mControl != null && !positionChanged

        if (mControl != null && mControlTarget != null && !positionChanged

                // The insets hint would be updated if the position is changed. Here updates it for

                // the possible change of the bounds or the server visibility.

                && (updateInsetsHint()

services/core/java/com/android/server/wm/InsetsStateController.java

+1 −1
Original line number Diff line number Diff line
@@ -373,7 +373,7 @@ class InsetsStateController { 
        array.add(provider);

    }



    void notifyControlChanged(InsetsControlTarget target, InsetsSourceProvider provider) {

    void notifyControlChanged(@NonNull InsetsControlTarget target, InsetsSourceProvider provider) {

        addToPendingControlMaps(target, provider);

        notifyPendingInsetsControlChanged();