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

Commit 333e0a0a authored by Dan Sandler's avatar Dan Sandler
Browse files

Verbose touch debugging for the status bar. 

This resurrects some truly ancient code (2012!) to
understand how users interact with the shade open gesture.
It turns out to be a convenient way to get logs for b/178277858, in which the notification shade simply won't be pulled down in some situations.

Bug: 178277858
Test: touch status bar; watch `adb logcat -b events`
Test: adb shell dumpsys activity service com.android.systemui ; \
      adb pull /data/user_de/0/com.android.systemui/cache/statusbar_gestures.dat
Change-Id: I2a0d66159c23dd68437b5e750309ae134e3797b7
parent 13a11f78

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

+4 −2
Original line number Diff line number Diff line
@@ -207,7 +207,7 @@ public class GestureRecorder { 
            sb.append(g.toJson());

            count++;

        }

        mLastSaveLen = count;

        mLastSaveLen += count;

        sb.append("]");

        return sb.toString();

    }
@@ -249,7 +249,9 @@ public class GestureRecorder { 
    public void dump(FileDescriptor fd, PrintWriter pw, String[] args) {

        save();

        if (mLastSaveLen >= 0) {

            pw.println(String.valueOf(mLastSaveLen) + " gestures written to " + mLogfile);

            pw.println(String.valueOf(mLastSaveLen)

                    + " gestures since last dump written to " + mLogfile);

            mLastSaveLen = 0;

        } else {

            pw.println("error writing gestures");

        }

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

+9 −5
Original line number Diff line number Diff line
@@ -78,6 +78,7 @@ import android.media.AudioAttributes; 
import android.metrics.LogMaker;

import android.net.Uri;

import android.os.AsyncTask;

import android.os.Build;

import android.os.Bundle;

import android.os.Handler;

import android.os.Looper;
@@ -276,7 +277,8 @@ public class StatusBar extends SystemUI implements DemoMode, 
    public static final boolean DEBUG = false;

    public static final boolean SPEW = false;

    public static final boolean DUMPTRUCK = true; // extra dumpsys info

    public static final boolean DEBUG_GESTURES = false;

    public static final boolean DEBUG_GESTURES = Build.IS_DEBUGGABLE; // TODO(b/178277858)

    public static final boolean DEBUG_GESTURES_VERBOSE = true;

    public static final boolean DEBUG_MEDIA_FAKE_ARTWORK = false;

    public static final boolean DEBUG_CAMERA_LIFT = false;
@@ -456,9 +458,7 @@ public class StatusBar extends SystemUI implements DemoMode, 
    private final DisplayMetrics mDisplayMetrics;



    // XXX: gesture research

    private final GestureRecorder mGestureRec = DEBUG_GESTURES

        ? new GestureRecorder("/sdcard/statusbar_gestures.dat")

        : null;

    private GestureRecorder mGestureRec = null;



    private final ScreenPinningRequest mScreenPinningRequest;
@@ -856,6 +856,10 @@ public class StatusBar extends SystemUI implements DemoMode, 


        mActivityIntentHelper = new ActivityIntentHelper(mContext);

        DateTimeView.setReceiverHandler(timeTickHandler);



        if (DEBUG_GESTURES) {

            mGestureRec = new GestureRecorder(mContext.getCacheDir() + "/statusbar_gestures.dat");

        }

    }



    @Override
@@ -2267,7 +2271,7 @@ public class StatusBar extends SystemUI implements DemoMode, 


    public boolean interceptTouchEvent(MotionEvent event) {

        if (DEBUG_GESTURES) {

            if (event.getActionMasked() != MotionEvent.ACTION_MOVE) {

            if (DEBUG_GESTURES_VERBOSE || event.getActionMasked() != MotionEvent.ACTION_MOVE) {

                EventLog.writeEvent(EventLogTags.SYSUI_STATUSBAR_TOUCH,

                        event.getActionMasked(), (int) event.getX(), (int) event.getY(),

                        mDisabled1, mDisabled2);