Merge "Don't animate brightness slider changes when triggered by keyboard" into main

public class BrightnessController implements ToggleSlider.Listener, MirroredBrightnessController {

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

    private static final int SLIDER_ANIMATION_DURATION = 1000;

    private static final int SLIDER_ANIMATION_DURATION = 3000;

    private static final int MSG_UPDATE_SLIDER = 1;

    private static final int MSG_ATTACH_LISTENER = 2;

    };

    private volatile boolean mAutomatic;  // Brightness adjusted automatically using ambient light.

    private boolean mTrackingTouch = false; // Brightness adjusted via touch events.

    private volatile boolean mIsVrModeEnabled;

    private boolean mListening;

    private boolean mExternalChange;

    @Override

    public void onChanged(boolean tracking, int value, boolean stopTracking) {

        mTrackingTouch = tracking;

        if (mExternalChange) return;

        if (mSliderAnimator != null) {

        }

    }

    private boolean triggeredByBrightnessKey() {

        // When the brightness mode is manual and the user isn't changing the brightness via the

        // brightness slider, assume changes are coming from a brightness key.

        return !mAutomatic && !mTrackingTouch;

    }

    private void updateSlider(float brightnessValue, boolean inVrMode) {

        final float min = mBrightnessMin;

        final float max = mBrightnessMax;

    }

    private void animateSliderTo(int target) {

        if (!mControlValueInitialized || !mControl.isVisible()) {

        if (!mControlValueInitialized || !mControl.isVisible() || triggeredByBrightnessKey()) {

            // Don't animate the first value since its default state isn't meaningful to users.

            // We also don't want to animate slider if it's not visible - especially important when

            // two sliders are active at the same time in split shade (one in QS and one in QQS),

            // as this negatively affects transition between them and they share mirror slider -

            // animating it from two different sources causes janky motion

            // animating it from two different sources causes janky motion.

            // Don't animate if the value is changed via the brightness keys of a keyboard.

            mControl.setValue(target);

            mControlValueInitialized = true;

        }