[fenix] For perf-frontend-issueshttps://github.com/mozilla-mobile/fenix/pull/33: Clarify VERBOSE logging motivation.

pull/600/head
Michael Comella 5 years ago committed by Michael Comella
parent 3b36ea34c8
commit ed87207990

@ -19,11 +19,14 @@ import android.util.Log
* to add monitoring code (i.e. the first time our application code is called in onRestart). An * to add monitoring code (i.e. the first time our application code is called in onRestart). An
* alternative implementation could measure performance from outside the application. * alternative implementation could measure performance from outside the application.
* *
* To see logs from this class, the user must enable VERBOSE logging for the appropriate tag: * The logs from this class are not visible to users by default. To see logs from this class, the
* user must enable VERBOSE logging for the appropriate tag:
* adb shell setprop log.tag.FenixPerf VERBOSE * adb shell setprop log.tag.FenixPerf VERBOSE
*/ */
class HotStartPerformanceMonitor( class HotStartPerformanceMonitor(
private val log: (String) -> Unit = { Log.v(Performance.TAG, it) }, // android log to minimize overhead. // We use VERBOSE logging so that the logs are not visible to users by default. We use the
// Android Log methods to minimize overhead introduced in a-c logging.
private val log: (String) -> Unit = { Log.v(Performance.TAG, it) },
private val getElapsedRealtime: () -> Long = { SystemClock.elapsedRealtime() } private val getElapsedRealtime: () -> Long = { SystemClock.elapsedRealtime() }
) { ) {

Loading…
Cancel
Save