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

Commit 9f70d02d authored by Siarhei Vishniakou's avatar Siarhei Vishniakou
Browse files

Use KEYCODE_UNKNOWN constant instead of zero 

Refactor KeyButtonView to use a constant name instead of value.

Bug: 128553584
Test: presubmit only
Change-Id: Id2cbd6810de3dbf8c3083f3670b60539b23aca56
parent 5a0a5e72

packages/SystemUI/src/com/android/systemui/statusbar/policy/KeyButtonView.java

+9 −8
Original line number Diff line number Diff line
@@ -17,6 +17,7 @@ 
package com.android.systemui.statusbar.policy;



import static android.view.Display.INVALID_DISPLAY;

import static android.view.KeyEvent.KEYCODE_UNKNOWN;

import static android.view.accessibility.AccessibilityNodeInfo.ACTION_CLICK;

import static android.view.accessibility.AccessibilityNodeInfo.ACTION_LONG_CLICK;
@@ -101,7 +102,7 @@ public class KeyButtonView extends ImageView implements ButtonInterface { 
        TypedArray a = context.obtainStyledAttributes(attrs, R.styleable.KeyButtonView,

                defStyle, 0);



        mCode = a.getInteger(R.styleable.KeyButtonView_keyCode, 0);

        mCode = a.getInteger(R.styleable.KeyButtonView_keyCode, KEYCODE_UNKNOWN);



        mSupportsLongpress = a.getBoolean(R.styleable.KeyButtonView_keyRepeat, true);

        mPlaySounds = a.getBoolean(R.styleable.KeyButtonView_playSound, true);
@@ -124,7 +125,7 @@ public class KeyButtonView extends ImageView implements ButtonInterface { 


    @Override

    public boolean isClickable() {

        return mCode != 0 || super.isClickable();

        return mCode != KEYCODE_UNKNOWN || super.isClickable();

    }



    public void setCode(int code) {
@@ -163,7 +164,7 @@ public class KeyButtonView extends ImageView implements ButtonInterface { 
    @Override

    public void onInitializeAccessibilityNodeInfo(AccessibilityNodeInfo info) {

        super.onInitializeAccessibilityNodeInfo(info);

        if (mCode != 0) {

        if (mCode != KEYCODE_UNKNOWN) {

            info.addAction(new AccessibilityNodeInfo.AccessibilityAction(ACTION_CLICK, null));

            if (mSupportsLongpress || isLongClickable()) {

                info.addAction(
@@ -182,13 +183,13 @@ public class KeyButtonView extends ImageView implements ButtonInterface { 


    @Override

    public boolean performAccessibilityActionInternal(int action, Bundle arguments) {

        if (action == ACTION_CLICK && mCode != 0) {

        if (action == ACTION_CLICK && mCode != KEYCODE_UNKNOWN) {

            sendEvent(KeyEvent.ACTION_DOWN, 0, SystemClock.uptimeMillis());

            sendEvent(KeyEvent.ACTION_UP, 0);

            sendAccessibilityEvent(AccessibilityEvent.TYPE_VIEW_CLICKED);

            playSoundEffect(SoundEffectConstants.CLICK);

            return true;

        } else if (action == ACTION_LONG_CLICK && mCode != 0) {

        } else if (action == ACTION_LONG_CLICK && mCode != KEYCODE_UNKNOWN) {

            sendEvent(KeyEvent.ACTION_DOWN, KeyEvent.FLAG_LONG_PRESS);

            sendEvent(KeyEvent.ACTION_UP, 0);

            sendAccessibilityEvent(AccessibilityEvent.TYPE_VIEW_LONG_CLICKED);
@@ -218,7 +219,7 @@ public class KeyButtonView extends ImageView implements ButtonInterface { 
                // Use raw X and Y to detect gestures in case a parent changes the x and y values

                mTouchDownX = (int) ev.getRawX();

                mTouchDownY = (int) ev.getRawY();

                if (mCode != 0) {

                if (mCode != KEYCODE_UNKNOWN) {

                    sendEvent(KeyEvent.ACTION_DOWN, 0, mDownTime);

                } else {

                    // Provide the same haptic feedback that the system offers for virtual keys.
@@ -249,7 +250,7 @@ public class KeyButtonView extends ImageView implements ButtonInterface { 
                break;

            case MotionEvent.ACTION_CANCEL:

                setPressed(false);

                if (mCode != 0) {

                if (mCode != KEYCODE_UNKNOWN) {

                    sendEvent(KeyEvent.ACTION_UP, KeyEvent.FLAG_CANCELED);

                }

                removeCallbacks(mCheckLongPress);
@@ -269,7 +270,7 @@ public class KeyButtonView extends ImageView implements ButtonInterface { 
                    // and it feels weird to sometimes get a release haptic and other times not.

                    performHapticFeedback(HapticFeedbackConstants.VIRTUAL_KEY_RELEASE);

                }

                if (mCode != 0) {

                if (mCode != KEYCODE_UNKNOWN) {

                    if (doIt) {

                        sendEvent(KeyEvent.ACTION_UP, 0);

                        sendAccessibilityEvent(AccessibilityEvent.TYPE_VIEW_CLICKED);