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

Commit 04995744 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Android (Google) Code Review
Browse files

Merge "Delete actionWithSource. Caller can use action directly."

parents 7c9f5bd4 6a607e7c
Loading
Loading
Loading
Loading
+0 −12
Original line number Diff line number Diff line
@@ -29,8 +29,6 @@ import com.android.internal.logging.nano.MetricsProto;
 */
public class EventLogWriter implements LogWriter {

    private final MetricsLogger mMetricsLogger = new MetricsLogger();

    @Override
    public void visible(Context context, int source, int category) {
        final LogMaker logMaker = new LogMaker(category)
@@ -49,16 +47,6 @@ public class EventLogWriter implements LogWriter {
        action(context, category, "", taggedData);
    }

    @Override
    public void actionWithSource(Context context, int source, int category) {
        final LogMaker logMaker = new LogMaker(category)
                .setType(MetricsProto.MetricsEvent.TYPE_ACTION);
        if (source != MetricsProto.MetricsEvent.VIEW_UNKNOWN) {
            logMaker.addTaggedData(MetricsProto.MetricsEvent.FIELD_CONTEXT, source);
        }
        MetricsLogger.action(logMaker);
    }

    @Override
    public void action(Context context, int category, int value) {
        MetricsLogger.action(context, category, value);
+0 −5
Original line number Diff line number Diff line
@@ -38,11 +38,6 @@ public interface LogWriter {
     */
    void action(Context context, int category, Pair<Integer, Object>... taggedData);

    /**
     * Logs an user action.
     */
    void actionWithSource(Context context, int source, int category);

    /**
     * Logs an user action.
     */
+0 −6
Original line number Diff line number Diff line
@@ -79,12 +79,6 @@ public class MetricsFeatureProvider {
        }
    }

    public void actionWithSource(Context context, int source, int category) {
        for (LogWriter writer : mLoggerWriters) {
            writer.actionWithSource(context, source, category);
        }
    }

    public void action(Context context, int category, Pair<Integer, Object>... taggedData) {
        for (LogWriter writer : mLoggerWriters) {
            writer.action(context, category, taggedData);