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

Commit 46da9999 authored by Caitlin Shkuratov's avatar Caitlin Shkuratov
Browse files

[Table logging] Add tracing to the log methods. 

Bug: 272016422
Test: Take systrace -> toggle wifi on and off multiple times -> see logs
in systrace (systrace attached to bug)

Change-Id: I57fbba0acbd4cc15d823e1043e5b94cfe6fb016d
parent dccbda11

packages/SystemUI/src/com/android/systemui/log/table/TableLogBuffer.kt

+7 −1
Original line number Diff line number Diff line
@@ -16,13 +16,13 @@ 


package com.android.systemui.log.table



import android.os.Trace

import com.android.systemui.Dumpable

import com.android.systemui.plugins.util.RingBuffer

import com.android.systemui.util.time.SystemClock

import java.io.PrintWriter

import java.text.SimpleDateFormat

import java.util.Locale

import kotlinx.coroutines.flow.Flow



/**

 * A logger that logs changes in table format.
@@ -138,18 +138,24 @@ class TableLogBuffer( 
    // timestamps.)



    private fun logChange(timestamp: Long, prefix: String, columnName: String, value: String?) {

        Trace.beginSection("TableLogBuffer#logChange(string)")

        val change = obtain(timestamp, prefix, columnName)

        change.set(value)

        Trace.endSection()

    }



    private fun logChange(timestamp: Long, prefix: String, columnName: String, value: Boolean) {

        Trace.beginSection("TableLogBuffer#logChange(boolean)")

        val change = obtain(timestamp, prefix, columnName)

        change.set(value)

        Trace.endSection()

    }



    private fun logChange(timestamp: Long, prefix: String, columnName: String, value: Int?) {

        Trace.beginSection("TableLogBuffer#logChange(int)")

        val change = obtain(timestamp, prefix, columnName)

        change.set(value)

        Trace.endSection()

    }



    // TODO(b/259454430): Add additional change types here.