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

Commit 111b5eae authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Android (Google) Code Review
Browse files

Merge "[Misc] Only create KeyguardStatusBarViewController once inside... 

Merge "[Misc] Only create KeyguardStatusBarViewController once inside NotificationPanelViewController."
parents 21f6c4bd 0c2b77e5

packages/SystemUI/src/com/android/systemui/battery/BatteryMeterViewController.java

+0 −6
Original line number Diff line number Diff line
@@ -107,12 +107,6 @@ public class BatteryMeterViewController extends ViewController<BatteryMeterView> 


    @Override

    protected void onViewDetached() {

        destroy();

    }



    @Override

    public void destroy() {

        super.destroy();

        mConfigurationController.removeCallback(mConfigurationListener);

        unsubscribeFromTunerUpdates();

        mCurrentUserTracker.stopTracking();

packages/SystemUI/src/com/android/systemui/statusbar/phone/KeyguardStatusBarViewController.java

+0 −11
Original line number Diff line number Diff line
@@ -159,17 +159,6 @@ public class KeyguardStatusBarViewController extends ViewController<KeyguardStat 


    @Override

    protected void onViewDetached() {

        destroy();

    }



    @Override

    public void destroy() {

        // Don't receive future #onViewAttached calls so that we don't accidentally have two

        // controllers registered for the same view.

        // TODO(b/194181195): This shouldn't be necessary.

        super.destroy();

        mBatteryMeterViewController.destroy();



        mConfigurationController.removeCallback(mConfigurationListener);

        mAnimationScheduler.removeCallback(mAnimationCallback);

        mUserInfoController.removeCallback(mOnUserInfoChangedListener);

packages/SystemUI/src/com/android/systemui/statusbar/phone/NotificationPanelViewController.java

+6 −13
Original line number Diff line number Diff line
@@ -928,10 +928,14 @@ public class NotificationPanelViewController extends PanelViewController { 
            }

        }



        mKeyguardStatusBarViewController =

                mKeyguardStatusBarViewComponentFactory.build(mKeyguardStatusBar)

                        .getKeyguardStatusBarViewController();

        mKeyguardStatusBarViewController.init();



        updateViewControllers(

                mView.findViewById(R.id.keyguard_status_view),

                userAvatarView,

                mKeyguardStatusBar,

                keyguardUserSwitcherView,

                mCommunalView,

                mView.findViewById(R.id.idle_host_view));
@@ -1025,7 +1029,6 @@ public class NotificationPanelViewController extends PanelViewController { 


    private void updateViewControllers(KeyguardStatusView keyguardStatusView,

            UserAvatarView userAvatarView,

            KeyguardStatusBarView keyguardStatusBarView,

            KeyguardUserSwitcherView keyguardUserSwitcherView,

            CommunalHostView communalView,

            IdleHostView idleHostView) {
@@ -1039,16 +1042,6 @@ public class NotificationPanelViewController extends PanelViewController { 
        mIdleHostViewController = idleViewComponent.getIdleHostViewController();

        mIdleHostViewController.init();



        KeyguardStatusBarViewComponent statusBarViewComponent =

                mKeyguardStatusBarViewComponentFactory.build(keyguardStatusBarView);

        if (mKeyguardStatusBarViewController != null) {

            // TODO(b/194181195): This shouldn't be necessary.

            mKeyguardStatusBarViewController.destroy();

        }

        mKeyguardStatusBarViewController =

                statusBarViewComponent.getKeyguardStatusBarViewController();

        mKeyguardStatusBarViewController.init();



        if (communalView != null) {

            CommunalViewComponent communalViewComponent =

                    mCommunalViewComponentFactory.build(communalView);
@@ -1217,7 +1210,7 @@ public class NotificationPanelViewController extends PanelViewController { 


        mBigClockContainer.removeAllViews();

        updateViewControllers(mView.findViewById(R.id.keyguard_status_view), userAvatarView,

                mKeyguardStatusBar, keyguardUserSwitcherView, mCommunalView,

                keyguardUserSwitcherView, mCommunalView,

                mView.findViewById(R.id.idle_host_view));



        // Update keyguard bottom area

packages/SystemUI/src/com/android/systemui/util/ViewController.java

+0 −10
Original line number Diff line number Diff line
@@ -109,16 +109,6 @@ public abstract class ViewController<T extends View> { 
        }

    }



    /**

     * Destroys this controller so that it never receives view attach and detach events again.

     * Does nothing if the view is null.

     */

    public void destroy() {

        if (mView != null) {

            mView.removeOnAttachStateChangeListener(mOnAttachStateListener);

        }

    }



    /**

     * Called when the view is attached and a call to {@link #init()} has been made in either order.

     */