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

Commit 469963c6 authored by Will Brockman's avatar Will Brockman
Browse files

Don't re-use LogMakers in LockscreenGestureLogger. 

Bug: 142268369
Test: atest SystemUiTests
Change-Id: I9ae2d908069f6cb90893477dcb9ad72138b34a98
parent af109096

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

+2 −4
Original line number Diff line number Diff line
@@ -35,8 +35,6 @@ import javax.inject.Singleton; 
@Singleton

public class LockscreenGestureLogger {

    private ArrayMap<Integer, Integer> mLegacyMap;

    private LogMaker mLogMaker = new LogMaker(MetricsEvent.VIEW_UNKNOWN)

            .setType(MetricsEvent.TYPE_ACTION);

    private final MetricsLogger mMetricsLogger = Dependency.get(MetricsLogger.class);



    @Inject
@@ -48,7 +46,7 @@ public class LockscreenGestureLogger { 
    }



    public void write(int gesture, int length, int velocity) {

        mMetricsLogger.write(mLogMaker.setCategory(gesture)

        mMetricsLogger.write(new LogMaker(gesture)

                .setType(MetricsEvent.TYPE_ACTION)

                .addTaggedData(MetricsEvent.FIELD_GESTURE_LENGTH, length)

                .addTaggedData(MetricsEvent.FIELD_GESTURE_VELOCITY, velocity));
@@ -64,7 +62,7 @@ public class LockscreenGestureLogger { 
     */

    public void writeAtFractionalPosition(

            int category, int xPercent, int yPercent, int rotation) {

        mMetricsLogger.write(mLogMaker.setCategory(category)

        mMetricsLogger.write(new LogMaker(category)

                .setType(MetricsEvent.TYPE_ACTION)

                .addTaggedData(MetricsEvent.FIELD_GESTURE_X_PERCENT, xPercent)

                .addTaggedData(MetricsEvent.FIELD_GESTURE_Y_PERCENT, yPercent)