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

Commit 648c4f96 authored by Winson Chung's avatar Winson Chung Committed by Android (Google) Code Review
Browse files

Merge "Don't update nav bar tint when in gesture nav" into rvc-dev

parents 9a172417 68e33e03

packages/SystemUI/src/com/android/systemui/statusbar/NavigationBarController.java

+3 −1
Original line number Diff line number Diff line
@@ -44,6 +44,7 @@ import com.android.systemui.statusbar.phone.BarTransitions.TransitionMode; 
import com.android.systemui.statusbar.phone.LightBarController;

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

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

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

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



import javax.inject.Inject;
@@ -139,7 +140,8 @@ public class NavigationBarController implements Callbacks { 
                    ? Dependency.get(LightBarController.class)

                    : new LightBarController(context,

                            Dependency.get(DarkIconDispatcher.class),

                            Dependency.get(BatteryController.class));

                            Dependency.get(BatteryController.class),

                            Dependency.get(NavigationModeController.class));

            navBar.setLightBarController(lightBarController);



            // TODO(b/118592525): to support multi-display, we start to add something which is

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

+8 −2
Original line number Diff line number Diff line
@@ -33,6 +33,7 @@ import com.android.internal.view.AppearanceRegion; 
import com.android.systemui.Dumpable;

import com.android.systemui.R;

import com.android.systemui.plugins.DarkIconDispatcher;

import com.android.systemui.shared.system.QuickStepContract;

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



import java.io.FileDescriptor;
@@ -58,6 +59,7 @@ public class LightBarController implements BatteryController.BatteryStateChangeC 
    private AppearanceRegion[] mAppearanceRegions = new AppearanceRegion[0];

    private int mStatusBarMode;

    private int mNavigationBarMode;

    private int mNavigationMode;

    private final Color mDarkModeColor;



    /**
@@ -84,11 +86,14 @@ public class LightBarController implements BatteryController.BatteryStateChangeC 


    @Inject

    public LightBarController(Context ctx, DarkIconDispatcher darkIconDispatcher,

            BatteryController batteryController) {

            BatteryController batteryController, NavigationModeController navModeController) {

        mDarkModeColor = Color.valueOf(ctx.getColor(R.color.dark_mode_icon_color_single_tone));

        mStatusBarIconController = (SysuiDarkIconDispatcher) darkIconDispatcher;

        mBatteryController = batteryController;

        mBatteryController.addCallback(this);

        mNavigationMode = navModeController.addListener((mode) -> {

            mNavigationMode = mode;

        });

    }



    public void setNavigationBar(LightBarTransitionsController navigationBar) {
@@ -234,7 +239,8 @@ public class LightBarController implements BatteryController.BatteryStateChangeC 
    }



    private void updateNavigation() {

        if (mNavigationBarController != null) {

        if (mNavigationBarController != null

                && !QuickStepContract.isGesturalMode(mNavigationMode)) {

            mNavigationBarController.setIconsDark(mNavigationLight, animateChange());

        }

    }

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

+0 −5
Original line number Diff line number Diff line
@@ -148,11 +148,6 @@ public class RegionSamplingHelper implements View.OnAttachStateChangeListener, 
        updateSamplingRect();

    }



    private void postUpdateSamplingListener() {

        mHandler.removeCallbacks(mUpdateSamplingListener);

        mHandler.post(mUpdateSamplingListener);

    }



    private void updateSamplingListener() {

        boolean isSamplingEnabled = mSamplingEnabled

                && !mSamplingRequestBounds.isEmpty()

packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/LightBarControllerTest.java

+1 −1
Original line number Diff line number Diff line
@@ -56,7 +56,7 @@ public class LightBarControllerTest extends SysuiTestCase { 
        when(mStatusBarIconController.getTransitionsController()).thenReturn(

                mLightBarTransitionsController);

        mLightBarController = new LightBarController(mContext, mStatusBarIconController,

                mock(BatteryController.class));

                mock(BatteryController.class), mock(NavigationModeController.class));

    }



    @Test