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

Commit 71d83f59 authored by Priyanka Advani (xWF)'s avatar Priyanka Advani (xWF)
Browse files

Revert "Make sure StatusBarContentInsetsProvider for correct dis..." 

Revert submission 30466729

Reason for revert: Droidmonitor created revert due to b/380269649. Will be verifying through ABTD before submission.

Reverted changes: /q/submissionid:30466729

Change-Id: I40585decc6ef63a7e53771248e334a0dcb667358
parent db469497

packages/SystemUI/multivalentTests/src/com/android/systemui/shade/QsBatteryModeControllerTest.kt

+1 −1
Original line number Diff line number Diff line
@@ -38,7 +38,7 @@ class QsBatteryModeControllerTest : SysuiTestCase() { 


    private val kosmos = testKosmos()

    private val insetsProviderStore = kosmos.fakeStatusBarContentInsetsProviderStore

    private val insetsProvider = insetsProviderStore.forDisplay(context.displayId)

    private val insetsProvider = insetsProviderStore.defaultDisplay



    @JvmField @Rule val mockitoRule = MockitoJUnit.rule()!!

packages/SystemUI/multivalentTests/src/com/android/systemui/statusbar/phone/KeyguardStatusBarViewControllerTest.kt

+1 −2
Original line number Diff line number Diff line
@@ -160,7 +160,7 @@ class KeyguardStatusBarViewControllerTest : SysuiTestCase() { 


        Mockito.`when`(iconManagerFactory.create(ArgumentMatchers.any(), ArgumentMatchers.any()))

            .thenReturn(iconManager)

        Mockito.`when`(statusBarContentInsetsProviderStore.forDisplay(context.displayId))

        Mockito.`when`(statusBarContentInsetsProviderStore.defaultDisplay)

            .thenReturn(kosmos.statusBarContentInsetsProvider)

        allowTestableLooperAsMainThread()

        looper.runWithLooper {
@@ -178,7 +178,6 @@ class KeyguardStatusBarViewControllerTest : SysuiTestCase() { 
    private fun createController(): KeyguardStatusBarViewController {

        return KeyguardStatusBarViewController(

            kosmos.testDispatcher,

            context,

            keyguardStatusBarView,

            carrierTextController,

            configurationController,

packages/SystemUI/src/com/android/systemui/shade/QsBatteryModeController.kt

+2 −3
Original line number Diff line number Diff line
@@ -15,11 +15,10 @@ class QsBatteryModeController 
@Inject

constructor(

    @ShadeDisplayAware private val context: Context,

    private val insetsProviderStore: StatusBarContentInsetsProviderStore,

    insetsProviderStore: StatusBarContentInsetsProviderStore,

) {



    private val insetsProvider

        get() = insetsProviderStore.forDisplay(context.displayId)

    private val insetsProvider = insetsProviderStore.defaultDisplay



    private companion object {

        // MotionLayout frames are in [0, 100]. Where 0 and 100 are reserved for start and end

packages/SystemUI/src/com/android/systemui/shade/ShadeHeaderController.kt

+1 −4
Original line number Diff line number Diff line
@@ -21,7 +21,6 @@ import android.animation.AnimatorListenerAdapter 
import android.annotation.IdRes

import android.app.PendingIntent

import android.app.StatusBarManager

import android.content.Context

import android.content.Intent

import android.content.res.Configuration

import android.graphics.Insets
@@ -93,7 +92,6 @@ constructor( 
    private val privacyIconsController: HeaderPrivacyIconsController,

    private val insetsProviderStore: StatusBarContentInsetsProviderStore,

    @ShadeDisplayAware private val configurationController: ConfigurationController,

    @ShadeDisplayAware private val shadeDisplayAwareContext: Context,

    private val variableDateViewControllerFactory: VariableDateViewController.Factory,

    @Named(SHADE_HEADER) private val batteryMeterViewController: BatteryMeterViewController,

    private val dumpManager: DumpManager,
@@ -106,8 +104,7 @@ constructor( 
    private val statusOverlayHoverListenerFactory: StatusOverlayHoverListenerFactory,

) : ViewController<View>(header), Dumpable {



    private val insetsProvider

        get() = insetsProviderStore.forDisplay(shadeDisplayAwareContext.displayId)

    private val insetsProvider = insetsProviderStore.defaultDisplay



    companion object {

        /** IDs for transitions and constraints for the [MotionLayout]. */

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

+4 −13
Original line number Diff line number Diff line
@@ -23,7 +23,6 @@ import static com.android.systemui.Flags.updateUserSwitcherBackground; 
import static com.android.systemui.statusbar.StatusBarState.KEYGUARD;

import static com.android.systemui.util.kotlin.JavaAdapterKt.collectFlow;



import android.content.Context;

import android.content.res.Configuration;

import android.content.res.Resources;

import android.database.ContentObserver;
@@ -57,7 +56,6 @@ import com.android.systemui.log.core.LogLevel; 
import com.android.systemui.plugins.statusbar.StatusBarStateController;

import com.android.systemui.res.R;

import com.android.systemui.scene.shared.flag.SceneContainerFlag;

import com.android.systemui.shade.ShadeDisplayAware;

import com.android.systemui.shade.ShadeViewStateProvider;

import com.android.systemui.statusbar.CommandQueue;

import com.android.systemui.statusbar.StatusBarState;
@@ -115,7 +113,6 @@ public class KeyguardStatusBarViewController extends ViewController<KeyguardStat 
            R.id.keyguard_hun_animator_start_tag);



    private final CoroutineDispatcher mCoroutineDispatcher;

    private final Context mContext;

    private final CarrierTextController mCarrierTextController;

    private final ConfigurationController mConfigurationController;

    private final SystemStatusAnimationScheduler mAnimationScheduler;
@@ -131,7 +128,7 @@ public class KeyguardStatusBarViewController extends ViewController<KeyguardStat 
    private final KeyguardStatusBarViewModel mKeyguardStatusBarViewModel;

    private final BiometricUnlockController mBiometricUnlockController;

    private final SysuiStatusBarStateController mStatusBarStateController;

    private final StatusBarContentInsetsProviderStore mInsetsProviderStore;

    private final StatusBarContentInsetsProvider mInsetsProvider;

    private final UserManager mUserManager;

    private final StatusBarUserChipViewModel mStatusBarUserChipViewModel;

    private final SecureSettings mSecureSettings;
@@ -316,7 +313,6 @@ public class KeyguardStatusBarViewController extends ViewController<KeyguardStat 
    @Inject

    public KeyguardStatusBarViewController(

            @Main CoroutineDispatcher dispatcher,

            @ShadeDisplayAware Context context,

            KeyguardStatusBarView view,

            CarrierTextController carrierTextController,

            ConfigurationController configurationController,
@@ -350,7 +346,6 @@ public class KeyguardStatusBarViewController extends ViewController<KeyguardStat 
    ) {

        super(view);

        mCoroutineDispatcher = dispatcher;

        mContext = context;

        mCarrierTextController = carrierTextController;

        mConfigurationController = configurationController;

        mAnimationScheduler = animationScheduler;
@@ -366,7 +361,7 @@ public class KeyguardStatusBarViewController extends ViewController<KeyguardStat 
        mKeyguardStatusBarViewModel = keyguardStatusBarViewModel;

        mBiometricUnlockController = biometricUnlockController;

        mStatusBarStateController = statusBarStateController;

        mInsetsProviderStore = statusBarContentInsetsProviderStore;

        mInsetsProvider = statusBarContentInsetsProviderStore.getDefaultDisplay();

        mUserManager = userManager;

        mStatusBarUserChipViewModel = userChipViewModel;

        mSecureSettings = secureSettings;
@@ -408,10 +403,6 @@ public class KeyguardStatusBarViewController extends ViewController<KeyguardStat 
        mStatusOverlayHoverListenerFactory = statusOverlayHoverListenerFactory;

    }



    private StatusBarContentInsetsProvider insetsProvider() {

        return mInsetsProviderStore.forDisplay(mContext.getDisplayId());

    }



    @Override

    protected void onInit() {

        super.onInit();
@@ -454,7 +445,7 @@ public class KeyguardStatusBarViewController extends ViewController<KeyguardStat 
                .createDarkAwareListener(mSystemIconsContainer, mView.darkChangeFlow());

        mSystemIconsContainer.setOnHoverListener(hoverListener);

        mView.setOnApplyWindowInsetsListener(

                (view, windowInsets) -> mView.updateWindowInsets(windowInsets, insetsProvider()));

                (view, windowInsets) -> mView.updateWindowInsets(windowInsets, mInsetsProvider));

        mSecureSettings.registerContentObserverForUserSync(

                Settings.Secure.STATUS_BAR_SHOW_VIBRATE_ICON,

                false,
@@ -652,7 +643,7 @@ public class KeyguardStatusBarViewController extends ViewController<KeyguardStat 
     * {@code OnApplyWindowInsetsListener}s.

     */

    public void setDisplayCutout(@Nullable DisplayCutout displayCutout) {

        mView.setDisplayCutout(displayCutout, insetsProvider());

        mView.setDisplayCutout(displayCutout, mInsetsProvider);

    }



    /**