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

Commit 694d40e8 authored by Fabian Kozynski's avatar Fabian Kozynski Committed by Android (Google) Code Review
Browse files

Merge "Revert "Making mirror slider appear when brightness slider is used in... 

Merge "Revert "Making mirror slider appear when brightness slider is used in split shade"" into sc-v2-dev
parents 510e959a 7b43d589

packages/SystemUI/src/com/android/systemui/qs/QSFragment.java

+2 −5
Original line number Diff line number Diff line
@@ -52,7 +52,6 @@ import com.android.systemui.statusbar.StatusBarState; 
import com.android.systemui.statusbar.notification.stack.StackStateAnimator;

import com.android.systemui.statusbar.phone.KeyguardBypassController;

import com.android.systemui.statusbar.phone.NotificationsQuickSettingsContainer;

import com.android.systemui.statusbar.policy.BrightnessMirrorController;

import com.android.systemui.statusbar.policy.RemoteInputQuickSettingsDisabler;

import com.android.systemui.util.InjectionInflationController;

import com.android.systemui.util.LifecycleFragment;
@@ -411,10 +410,8 @@ public class QSFragment extends LifecycleFragment implements QS, CommandQueue.Ca 
        }

    }



    public void setBrightnessMirrorController(

            BrightnessMirrorController brightnessMirrorController) {

        mQSPanelController.setBrightnessMirror(brightnessMirrorController);

        mQuickQSPanelController.setBrightnessMirror(brightnessMirrorController);

    public QSPanelController getQSPanelController() {

        return mQSPanelController;

    }



    @Override

packages/SystemUI/src/com/android/systemui/qs/QSPanelController.java

+30 −7
Original line number Diff line number Diff line
@@ -40,7 +40,6 @@ import com.android.systemui.qs.customize.QSCustomizerController; 
import com.android.systemui.qs.dagger.QSScope;

import com.android.systemui.qs.logging.QSLogger;

import com.android.systemui.settings.brightness.BrightnessController;

import com.android.systemui.settings.brightness.BrightnessMirrorHandler;

import com.android.systemui.settings.brightness.BrightnessSlider;

import com.android.systemui.statusbar.policy.BrightnessMirrorController;

import com.android.systemui.tuner.TunerService;
@@ -61,9 +60,10 @@ public class QSPanelController extends QSPanelControllerBase<QSPanel> { 
    private final QSTileRevealController.Factory mQsTileRevealControllerFactory;

    private final FalsingManager mFalsingManager;

    private final BrightnessController mBrightnessController;

    private final BrightnessSlider.Factory mBrightnessSliderFactory;

    private final BrightnessSlider mBrightnessSlider;

    private final BrightnessMirrorHandler mBrightnessMirrorHandler;



    private BrightnessMirrorController mBrightnessMirrorController;

    private boolean mGridContentVisible = true;



    private final QSPanel.OnConfigurationChangedListener mOnConfigurationChangedListener =
@@ -75,10 +75,14 @@ public class QSPanelController extends QSPanelControllerBase<QSPanel> { 
            if (mView.isListening()) {

                refreshAllTiles();

            }

            updateBrightnessMirror();

            mView.switchSecurityFooter(mShouldUseSplitNotificationShade);

        }

    };



    private final BrightnessMirrorController.BrightnessMirrorListener mBrightnessMirrorListener =

            mirror -> updateBrightnessMirror();



    private View.OnTouchListener mTileLayoutTouchListener = new View.OnTouchListener() {

        @Override

        public boolean onTouch(View v, MotionEvent event) {
@@ -106,12 +110,12 @@ public class QSPanelController extends QSPanelControllerBase<QSPanel> { 
        mQsTileRevealControllerFactory = qsTileRevealControllerFactory;

        mFalsingManager = falsingManager;

        mQsSecurityFooter.setHostEnvironment(qstileHost);

        mBrightnessSliderFactory = brightnessSliderFactory;



        mBrightnessSlider = brightnessSliderFactory.create(getContext(), mView);

        mBrightnessSlider = mBrightnessSliderFactory.create(getContext(), mView);

        mView.setBrightnessView(mBrightnessSlider.getRootView());



        mBrightnessController = brightnessControllerFactory.create(mBrightnessSlider);

        mBrightnessMirrorHandler = new BrightnessMirrorHandler(mBrightnessController);

    }



    @Override
@@ -138,7 +142,9 @@ public class QSPanelController extends QSPanelControllerBase<QSPanel> { 
        mView.addOnConfigurationChangedListener(mOnConfigurationChangedListener);

        mView.setSecurityFooter(mQsSecurityFooter.getView(), mShouldUseSplitNotificationShade);

        switchTileLayout(true);

        mBrightnessMirrorHandler.onQsPanelAttached();

        if (mBrightnessMirrorController != null) {

            mBrightnessMirrorController.addCallback(mBrightnessMirrorListener);

        }



        ((PagedTileLayout) mView.getOrCreateTileLayout())

                .setOnTouchListener(mTileLayoutTouchListener);
@@ -154,7 +160,9 @@ public class QSPanelController extends QSPanelControllerBase<QSPanel> { 
    protected void onViewDetached() {

        mTunerService.removeTunable(mView);

        mView.removeOnConfigurationChangedListener(mOnConfigurationChangedListener);

        mBrightnessMirrorHandler.onQsPanelDettached();

        if (mBrightnessMirrorController != null) {

            mBrightnessMirrorController.removeCallback(mBrightnessMirrorListener);

        }

        super.onViewDetached();

    }
@@ -188,8 +196,23 @@ public class QSPanelController extends QSPanelControllerBase<QSPanel> { 
        }

    }



    /** */

    public void setBrightnessMirror(BrightnessMirrorController brightnessMirrorController) {

        mBrightnessMirrorHandler.setController(brightnessMirrorController);

        mBrightnessMirrorController = brightnessMirrorController;

        if (mBrightnessMirrorController != null) {

            mBrightnessMirrorController.removeCallback(mBrightnessMirrorListener);

        }

        mBrightnessMirrorController = brightnessMirrorController;

        if (mBrightnessMirrorController != null) {

            mBrightnessMirrorController.addCallback(mBrightnessMirrorListener);

        }

        updateBrightnessMirror();

    }



    private void updateBrightnessMirror() {

        if (mBrightnessMirrorController != null) {

            mBrightnessSlider.setMirrorControllerAndMirror(mBrightnessMirrorController);

        }

    }



    /** Get the QSTileHost this panel uses. */

packages/SystemUI/src/com/android/systemui/qs/QuickQSBrightnessController.kt

+5 −12
Original line number Diff line number Diff line
@@ -19,8 +19,6 @@ package com.android.systemui.qs 
import androidx.annotation.VisibleForTesting

import com.android.systemui.settings.brightness.BrightnessController

import com.android.systemui.settings.brightness.BrightnessSlider

import com.android.systemui.settings.brightness.MirroredBrightnessController

import com.android.systemui.statusbar.policy.BrightnessMirrorController

import javax.inject.Inject



/**
@@ -29,7 +27,7 @@ import javax.inject.Inject 
 */

class QuickQSBrightnessController @VisibleForTesting constructor(

    private val brightnessControllerFactory: () -> BrightnessController

) : MirroredBrightnessController {

) {



    @Inject constructor(

        brightnessControllerFactory: BrightnessController.Factory,
@@ -44,7 +42,6 @@ class QuickQSBrightnessController @VisibleForTesting constructor( 


    private var isListening = false

    private var brightnessController: BrightnessController? = null

    private var mirrorController: BrightnessMirrorController? = null



    fun init(shouldUseSplitNotificationShade: Boolean) {

        refreshVisibility(shouldUseSplitNotificationShade)
@@ -80,11 +77,6 @@ class QuickQSBrightnessController @VisibleForTesting constructor( 
        }

    }



    override fun setMirror(controller: BrightnessMirrorController) {

        mirrorController = controller

        mirrorController?.let { brightnessController?.setMirror(it) }

    }



    private fun hideBrightnessSlider() {

        brightnessController?.hideSlider()

    }
@@ -92,10 +84,11 @@ class QuickQSBrightnessController @VisibleForTesting constructor( 
    private fun showBrightnessSlider() {

        if (brightnessController == null) {

            brightnessController = brightnessControllerFactory()

            mirrorController?.also { brightnessController?.setMirror(it) }

        }

        brightnessController?.showSlider()

        if (!isListening) {

            brightnessController?.registerCallbacks()

            isListening = true

        }

        brightnessController?.showSlider()

    }

}
 
 No newline at end of file

packages/SystemUI/src/com/android/systemui/qs/QuickQSPanelController.java

+0 −10
Original line number Diff line number Diff line
@@ -29,8 +29,6 @@ import com.android.systemui.plugins.qs.QSTile; 
import com.android.systemui.qs.customize.QSCustomizerController;

import com.android.systemui.qs.dagger.QSScope;

import com.android.systemui.qs.logging.QSLogger;

import com.android.systemui.settings.brightness.BrightnessMirrorHandler;

import com.android.systemui.statusbar.policy.BrightnessMirrorController;



import java.util.ArrayList;

import java.util.List;
@@ -52,7 +50,6 @@ public class QuickQSPanelController extends QSPanelControllerBase<QuickQSPanel> 


    // brightness is visible only in split shade

    private final QuickQSBrightnessController mBrightnessController;

    private final BrightnessMirrorHandler mBrightnessMirrorHandler;



    @Inject

    QuickQSPanelController(QuickQSPanel view, QSTileHost qsTileHost,
@@ -66,7 +63,6 @@ public class QuickQSPanelController extends QSPanelControllerBase<QuickQSPanel> 
        super(view, qsTileHost, qsCustomizerController, usingMediaPlayer, mediaHost, metricsLogger,

                uiEventLogger, qsLogger, dumpManager);

        mBrightnessController = quickQSBrightnessController;

        mBrightnessMirrorHandler = new BrightnessMirrorHandler(mBrightnessController);

    }



    @Override
@@ -82,14 +78,12 @@ public class QuickQSPanelController extends QSPanelControllerBase<QuickQSPanel> 
    protected void onViewAttached() {

        super.onViewAttached();

        mView.addOnConfigurationChangedListener(mOnConfigurationChangedListener);

        mBrightnessMirrorHandler.onQsPanelAttached();

    }



    @Override

    protected void onViewDetached() {

        super.onViewDetached();

        mView.removeOnConfigurationChangedListener(mOnConfigurationChangedListener);

        mBrightnessMirrorHandler.onQsPanelDettached();

    }



    @Override
@@ -138,8 +132,4 @@ public class QuickQSPanelController extends QSPanelControllerBase<QuickQSPanel> 
    public int getNumQuickTiles() {

        return mView.getNumQuickTiles();

    }



    public void setBrightnessMirror(BrightnessMirrorController brightnessMirrorController) {

        mBrightnessMirrorHandler.setController(brightnessMirrorController);

    }

}

packages/SystemUI/src/com/android/systemui/settings/brightness/BrightnessController.java

+1 −7
Original line number Diff line number Diff line
@@ -50,13 +50,12 @@ import com.android.settingslib.RestrictedLockUtilsInternal; 
import com.android.systemui.Dependency;

import com.android.systemui.broadcast.BroadcastDispatcher;

import com.android.systemui.settings.CurrentUserTracker;

import com.android.systemui.statusbar.policy.BrightnessMirrorController;



import java.util.ArrayList;



import javax.inject.Inject;



public class BrightnessController implements ToggleSlider.Listener, MirroredBrightnessController {

public class BrightnessController implements ToggleSlider.Listener {

    private static final String TAG = "StatusBar.BrightnessController";

    private static final int SLIDER_ANIMATION_DURATION = 3000;
@@ -110,11 +109,6 @@ public class BrightnessController implements ToggleSlider.Listener, MirroredBrig 


    private ValueAnimator mSliderAnimator;



    @Override

    public void setMirror(BrightnessMirrorController controller) {

        mControl.setMirrorControllerAndMirror(controller);

    }



    public interface BrightnessStateChangeCallback {

        /** Indicates that some of the brightness settings have changed */

        void onBrightnessLevelChanged();