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

Commit ad1a3d06 authored by Biswarup Pal's avatar Biswarup Pal
Browse files

Revert^2 "Migrate usages of ViewConfiguration static methods" 

This reverts commit fbef669f.

Reason for revert: Original change was not a culprit of b/418347997

Test: presubmit
Bug: 370720522
Change-Id: I7f5fc3ed3fd99d213905dff055f2474ca7c18d63
parent 7c78aba6

core/java/android/inputmethodservice/KeyboardView.java

+10 −3
Original line number Diff line number Diff line
@@ -16,6 +16,7 @@ 


package android.inputmethodservice;



import android.companion.virtualdevice.flags.Flags;

import android.compat.annotation.UnsupportedAppUsage;

import android.content.Context;

import android.content.res.TypedArray;
@@ -220,6 +221,7 @@ public class KeyboardView extends View implements View.OnClickListener { 
    private SwipeTracker mSwipeTracker = new SwipeTracker();

    private int mSwipeThreshold;

    private boolean mDisambiguateSwipe;

    private final int mLongPressTimeoutMillis;



    // Variables for dealing with multiple pointers

    private int mOldPointerCount = 1;
@@ -231,7 +233,6 @@ public class KeyboardView extends View implements View.OnClickListener { 


    private static final int REPEAT_INTERVAL = 50; // ~20 keys per second

    private static final int REPEAT_START_DELAY = 400;

    private static final int LONGPRESS_TIMEOUT = ViewConfiguration.getLongPressTimeout();



    private static int MAX_NEARBY_KEYS = 12;

    private int[] mDistances = new int[MAX_NEARBY_KEYS];
@@ -368,6 +369,12 @@ public class KeyboardView extends View implements View.OnClickListener { 
        mAccessibilityManager = AccessibilityManager.getInstance(context);

        mAudioManager = (AudioManager) context.getSystemService(Context.AUDIO_SERVICE);



        if (Flags.viewconfigurationApis()) {

            mLongPressTimeoutMillis = ViewConfiguration.get(context).getLongPressTimeoutMillis();

        } else {

            mLongPressTimeoutMillis = ViewConfiguration.getLongPressTimeout();

        }



        resetMultiTap();

    }
@@ -1292,7 +1299,7 @@ public class KeyboardView extends View implements View.OnClickListener { 
                }

                if (mCurrentKey != NOT_A_KEY) {

                    Message msg = mHandler.obtainMessage(MSG_LONGPRESS, me);

                    mHandler.sendMessageDelayed(msg, LONGPRESS_TIMEOUT);

                    mHandler.sendMessageDelayed(msg, mLongPressTimeoutMillis);

                }

                showPreview(keyIndex);

                break;
@@ -1325,7 +1332,7 @@ public class KeyboardView extends View implements View.OnClickListener { 
                    // Start new longpress if key has changed

                    if (keyIndex != NOT_A_KEY) {

                        Message msg = mHandler.obtainMessage(MSG_LONGPRESS, me);

                        mHandler.sendMessageDelayed(msg, LONGPRESS_TIMEOUT);

                        mHandler.sendMessageDelayed(msg, mLongPressTimeoutMillis);

                    }

                }

                showPreview(mCurrentKey);

core/java/android/inputmethodservice/navigationbar/KeyButtonView.java

+7 −1
Original line number Diff line number Diff line
@@ -74,6 +74,7 @@ public class KeyButtonView extends ImageView implements ButtonInterface { 
    private final Paint mOvalBgPaint = new Paint(Paint.ANTI_ALIAS_FLAG | Paint.FILTER_BITMAP_FLAG);

    private float mDarkIntensity;

    private boolean mHasOvalBg = false;

    private final int mLongPressTimeoutMillis;



    /** Runnable for checking whether the long click action should be performed. */

    private final Runnable mCheckLongPress = new Runnable() {
@@ -107,6 +108,11 @@ public class KeyButtonView extends ImageView implements ButtonInterface { 
        setBackground(mRipple);

        setWillNotDraw(false);

        forceHasOverlappingRendering(false);



        mLongPressTimeoutMillis =

                android.companion.virtualdevice.flags.Flags.viewconfigurationApis()

                        ? ViewConfiguration.get(context).getLongPressTimeoutMillis()

                        : ViewConfiguration.getLongPressTimeout();

    }



    @Override
@@ -210,7 +216,7 @@ public class KeyButtonView extends ImageView implements ButtonInterface { 
                }

                if (Flags.imeSwitcherRevamp() && isLongClickable()) {

                    removeCallbacks(mCheckLongPress);

                    postDelayed(mCheckLongPress, ViewConfiguration.getLongPressTimeout());

                    postDelayed(mCheckLongPress, mLongPressTimeoutMillis);

                }

                break;

            case MotionEvent.ACTION_MOVE:

core/java/android/view/GestureDetector.java

+6 −3
Original line number Diff line number Diff line
@@ -250,6 +250,7 @@ public class GestureDetector { 
    private int mTapTimeout;

    private int mDoubleTapTimeout;

    private int mDoubleTapMinTime;

    private int mLongPressTimeoutMillis;



    // constants for Message.what used by GestureHandler below

    private static final int SHOW_PRESS = 1;
@@ -514,10 +515,12 @@ public class GestureDetector { 
                mTapTimeout = configuration.getTapTimeoutMillis();

                mDoubleTapTimeout = configuration.getDoubleTapTimeoutMillis();

                mDoubleTapMinTime = configuration.getDoubleTapMinTimeMillis();

                mLongPressTimeoutMillis = configuration.getLongPressTimeoutMillis();

            } else {

                mTapTimeout = ViewConfiguration.getTapTimeout();

                mDoubleTapTimeout = ViewConfiguration.getDoubleTapTimeout();

                mDoubleTapMinTime = ViewConfiguration.getDoubleTapMinTime();

                mLongPressTimeoutMillis = ViewConfiguration.getLongPressTimeout();

            }

        }

        mTouchSlopSquare = touchSlop * touchSlop;
@@ -687,7 +690,7 @@ public class GestureDetector { 
                                    TOUCH_GESTURE_CLASSIFIED__CLASSIFICATION__LONG_PRESS,

                                    0 /* arg2 */),

                            mCurrentDownEvent.getDownTime()

                                    + ViewConfiguration.getLongPressTimeout());

                                    + mLongPressTimeoutMillis);

                }

                mHandler.sendEmptyMessageAtTime(SHOW_PRESS,

                        mCurrentDownEvent.getDownTime() + mTapTimeout);
@@ -728,14 +731,14 @@ public class GestureDetector { 
                            // will happen in response to user input. To prevent this,

                            // reschedule long press with a modified timeout.

                            mHandler.removeMessages(LONG_PRESS);

                            final long longPressTimeout = ViewConfiguration.getLongPressTimeout();

                            mHandler.sendMessageAtTime(

                                    mHandler.obtainMessage(

                                            LONG_PRESS,

                                            TOUCH_GESTURE_CLASSIFIED__CLASSIFICATION__LONG_PRESS,

                                            0 /* arg2 */),

                                    ev.getDownTime()

                                        + (long) (longPressTimeout * mAmbiguousGestureMultiplier));

                                        + (long) (mLongPressTimeoutMillis

                                            * mAmbiguousGestureMultiplier));

                        }

                        // Inhibit default scroll. If a gesture is ambiguous, we prevent scroll

                        // until the gesture is resolved.

core/java/android/view/HandwritingInitiator.java

+4 −1
Original line number Diff line number Diff line
@@ -149,7 +149,10 @@ public class HandwritingInitiator { 
    public HandwritingInitiator(@NonNull ViewConfiguration viewConfiguration,

            @NonNull InputMethodManager inputMethodManager) {

        mHandwritingSlop = viewConfiguration.getScaledHandwritingSlop();

        mHandwritingTimeoutInMillis = ViewConfiguration.getLongPressTimeout();

        mHandwritingTimeoutInMillis =

                android.companion.virtualdevice.flags.Flags.viewconfigurationApis()

                        ? viewConfiguration.getLongPressTimeoutMillis()

                        : ViewConfiguration.getLongPressTimeout();

        mImm = inputMethodManager;

    }

core/java/android/view/View.java

+38 −45
Original line number Diff line number Diff line
@@ -5519,21 +5519,6 @@ public class View implements Drawable.Callback, KeyEvent.Callback, 
     */

    private ViewTreeObserver mFloatingTreeObserver;













    /**













     * Cache the touch slop from the context that created the view.













     */













    private int mTouchSlop;

























    /**













     * Cache the tap timeout from the context that created the view.













     */













    private int mTapTimeoutMillis;

























    /**













     * Cache the ambiguous gesture multiplier from the context that created the view.













     */













    private float mAmbiguousGestureMultiplier;



























    /**















     * Object that handles automatic animation of view properties.















     */










@@ -5905,9 +5890,10 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













    private float mFrameContentVelocity = -1;





























    @Nullable



























    private ViewTranslationResponse mViewTranslationResponse;



























    private final ViewConfiguration mViewConfiguration;



























    /**















     * The size in DP that is considered small for VRR purposes, if square.















     */










@@ -6010,11 +5996,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













                (PFLAG2_TEXT_ALIGNMENT_RESOLVED_DEFAULT) |















                (IMPORTANT_FOR_ACCESSIBILITY_DEFAULT << PFLAG2_IMPORTANT_FOR_ACCESSIBILITY_SHIFT);



























        final ViewConfiguration configuration = ViewConfiguration.get(context);













        mTouchSlop = configuration.getScaledTouchSlop();













        mTapTimeoutMillis = Flags.viewconfigurationApis()













                ? configuration.getTapTimeoutMillis() : ViewConfiguration.getTapTimeout();













        mAmbiguousGestureMultiplier = configuration.getScaledAmbiguousGestureMultiplier();













        mViewConfiguration = ViewConfiguration.get(context);





























        setOverScrollMode(OVER_SCROLL_IF_CONTENT_SCROLLS);















        mUserPaddingStart = UNDEFINED_PADDING;










@@ -7018,6 +7000,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













    View() {















        mResources = null;















        mRenderNode = RenderNode.create(getClass().getName(), new ViewAnimationHostBridge(this));













        mViewConfiguration = new ViewConfiguration();















    }





























    /**










@@ -7274,7 +7257,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,



























        mScrollCache.fadingEdgeLength = a.getDimensionPixelSize(















                R.styleable.View_fadingEdgeLength,













                ViewConfiguration.get(mContext).getScaledFadingEdgeLength());













            mViewConfiguration.getScaledFadingEdgeLength());















    }





























    /**










@@ -7517,7 +7500,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,



























        scrollabilityCache.scrollBarSize = a.getDimensionPixelSize(















                com.android.internal.R.styleable.View_scrollbarSize,













                ViewConfiguration.get(mContext).getScaledScrollBarSize());













                mViewConfiguration.getScaledScrollBarSize());





























        Drawable track = a.getDrawable(R.styleable.View_scrollbarTrackHorizontal);















        scrollabilityCache.scrollBar.setHorizontalTrackDrawable(track);










@@ -7676,7 +7659,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













     */















    private void initScrollCache() {















        if (mScrollCache == null) {













            mScrollCache = new ScrollabilityCache(ViewConfiguration.get(mContext), this);













            mScrollCache = new ScrollabilityCache(mViewConfiguration, this);















        }















    }
























@@ -13326,7 +13309,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













    List<Rect> collectPreferKeepClearRects() {















        ListenerInfo info = mListenerInfo;















        boolean keepClearForFocus = isFocused()













                && ViewConfiguration.get(mContext).isPreferKeepClearForFocusEnabled();













                && mViewConfiguration.isPreferKeepClearForFocusEnabled();















        boolean keepBoundsClear = (info != null && info.mPreferKeepClear) || keepClearForFocus;















        boolean hasCustomKeepClearRects = info != null && info.mKeepClearRects != null;























@@ -13349,7 +13332,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













    }





























    private void updatePreferKeepClearForFocus() {













        if (ViewConfiguration.get(mContext).isPreferKeepClearForFocusEnabled()) {













        if (mViewConfiguration.isPreferKeepClearForFocusEnabled()) {















            updatePositionUpdateListener();















            post(this::updateKeepClearRects);















        }










@@ -16971,8 +16954,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













            // 2) Limits latency from the `ViewConfiguration` API, which may be slow due to feature















            //    flag querying.















            if ((mPrivateFlags4 & PFLAG4_ROTARY_HAPTICS_DETERMINED) == 0) {













                if (ViewConfiguration.get(mContext)













                        .isViewBasedRotaryEncoderHapticScrollFeedbackEnabled()) {













                if (mViewConfiguration.isViewBasedRotaryEncoderHapticScrollFeedbackEnabled()) {















                    mPrivateFlags4 |= PFLAG4_ROTARY_HAPTICS_ENABLED;















                }















                mPrivateFlags4 |= PFLAG4_ROTARY_HAPTICS_DETERMINED;










@@ -17573,7 +17555,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













                        setPressed(true, x, y);















                    }















                    checkForLongClick(













                            ViewConfiguration.getLongPressTimeout(),













                            getLongPressTimeoutMillis(),















                            x,















                            y,















                            // This is not a touch gesture -- do not classify it as one.










@@ -18381,7 +18363,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,



























                    if (!clickable) {















                        checkForLongClick(













                                ViewConfiguration.getLongPressTimeout(),













                                getLongPressTimeoutMillis(),















                                x,















                                y,















                                TOUCH_GESTURE_CLASSIFIED__CLASSIFICATION__LONG_PRESS);









@@ -18404,12 +18386,12 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













                        }















                        mPendingCheckForTap.x = event.getX();















                        mPendingCheckForTap.y = event.getY();













                        postDelayed(mPendingCheckForTap, mTapTimeoutMillis);













                        postDelayed(mPendingCheckForTap, getTapTimeoutMillis());















                    } else {















                        // Not inside a scrolling container, so show the feedback right away















                        setPressed(true, x, y);















                        checkForLongClick(













                                ViewConfiguration.getLongPressTimeout(),













                                getLongPressTimeoutMillis(),















                                x,















                                y,















                                TOUCH_GESTURE_CLASSIFIED__CLASSIFICATION__LONG_PRESS);









@@ -18436,15 +18418,17 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













                    final int motionClassification = event.getClassification();















                    final boolean ambiguousGesture =















                            motionClassification == MotionEvent.CLASSIFICATION_AMBIGUOUS_GESTURE;













                    int touchSlop = mTouchSlop;













                    int touchSlop = mViewConfiguration.getScaledTouchSlop();















                    if (ambiguousGesture && hasPendingLongPressCallback()) {













                        float ambiguousGestureMultiplier =













                                mViewConfiguration.getScaledAmbiguousGestureMultiplier();















                        if (!pointInView(x, y, touchSlop)) {















                            // The default action here is to cancel long press. But instead, we















                            // just extend the timeout here, in case the classification















                            // stays ambiguous.















                            removeLongPressCallback();













                            long delay = (long) (ViewConfiguration.getLongPressTimeout()













                                    * mAmbiguousGestureMultiplier);













                            long delay = (long) (getLongPressTimeoutMillis()













                                    * ambiguousGestureMultiplier);















                            // Subtract the time already spent















                            delay -= event.getEventTime() - event.getDownTime();















                            checkForLongClick(









@@ -18453,7 +18437,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













                                    y,















                                    TOUCH_GESTURE_CLASSIFIED__CLASSIFICATION__LONG_PRESS);















                        }













                        touchSlop *= mAmbiguousGestureMultiplier;













                        touchSlop *= ambiguousGestureMultiplier;















                    }





























                    // Be lenient about moving outside of buttons










@@ -18531,6 +18515,18 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













        }















    }



























    private int getLongPressTimeoutMillis() {













        return Flags.viewconfigurationApis()













                ? mViewConfiguration.getLongPressTimeoutMillis()













                : ViewConfiguration.getLongPressTimeout();













    }

























    private int getTapTimeoutMillis() {













        return Flags.viewconfigurationApis()













                ? mViewConfiguration.getTapTimeoutMillis()













                : ViewConfiguration.getTapTimeout();













    }



























    /**















     * Return true if the long press callback is scheduled to run sometime in the future.















     * Return false if there is no scheduled long press callback at the moment.










@@ -18941,15 +18937,14 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













    private HapticScrollFeedbackProvider getScrollFeedbackProvider() {















        if (mScrollFeedbackProvider == null) {















            mScrollFeedbackProvider = new HapticScrollFeedbackProvider(this,













                    ViewConfiguration.get(mContext), /* isFromView= */ true);













                    mViewConfiguration, /* isFromView= */ true);















        }















        return mScrollFeedbackProvider;















    }





























    private void doRotaryProgressForScrollHaptics(MotionEvent rotaryEvent) {















        final float axisScrollValue = rotaryEvent.getAxisValue(MotionEvent.AXIS_SCROLL);













        final float verticalScrollFactor =













                ViewConfiguration.get(mContext).getScaledVerticalScrollFactor();













        final float verticalScrollFactor = mViewConfiguration.getScaledVerticalScrollFactor();















        final int scrollAmount = -Math.round(axisScrollValue * verticalScrollFactor);















        getScrollFeedbackProvider().onScrollProgress(















                rotaryEvent.getDeviceId(), InputDevice.SOURCE_ROTARY_ENCODER,










@@ -22138,7 +22133,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













     */















    @InspectableProperty(name = "scrollbarSize")















    public int getScrollBarSize() {













        return mScrollCache == null ? ViewConfiguration.get(mContext).getScaledScrollBarSize() :













        return mScrollCache == null ? mViewConfiguration.getScaledScrollBarSize() :















                mScrollCache.scrollBarSize;















    }
























@@ -24425,8 +24420,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













        final boolean use32BitCache = attachInfo != null && attachInfo.mUse32BitDrawingCache;





























        final long projectedBitmapSize = width * height * (opaque && !use32BitCache ? 2 : 4);













        final long drawingCacheSize =













                ViewConfiguration.get(mContext).getScaledMaximumDrawingCacheSize();













        final long drawingCacheSize = mViewConfiguration.getScaledMaximumDrawingCacheSize();















        if (width <= 0 || height <= 0 || projectedBitmapSize > drawingCacheSize) {















            if (width > 0 && height > 0) {















                Log.w(VIEW_LOG_TAG, getClass().getSimpleName() + " not displayed because it is"










@@ -31769,8 +31763,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













        public void run() {















            mPrivateFlags &= ~PFLAG_PREPRESSED;















            setPressed(true, x, y);













            final long delay =













                    (long) ViewConfiguration.getLongPressTimeout() - mTapTimeoutMillis;













            final int delay = getLongPressTimeoutMillis() - getTapTimeoutMillis();















            checkForLongClick(delay, x, y, TOUCH_GESTURE_CLASSIFIED__CLASSIFICATION__LONG_PRESS);















        }















    }










@@ -33760,7 +33753,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,













                mTooltipInfo = new TooltipInfo();















                mTooltipInfo.mShowTooltipRunnable = this::showHoverTooltip;















                mTooltipInfo.mHideTooltipRunnable = this::hideTooltip;













                mTooltipInfo.mHoverSlop = ViewConfiguration.get(mContext).getScaledHoverSlop();













                mTooltipInfo.mHoverSlop = mViewConfiguration.getScaledHoverSlop();















                mTooltipInfo.clearAnchorPos();















            }















            mTooltipInfo.mTooltipText = tooltipText;