Fix screen trace logging by printing the trace name. (#3599)

jeremyjiang-dev · web-flow · commit 84bb05ba9923 · 2022-03-29T13:22:47.000-04:00
diff --git a/firebase-perf/src/main/java/com/google/firebase/perf/metrics/Trace.java b/firebase-perf/src/main/java/com/google/firebase/perf/metrics/Trace.java
@@ -498,7 +498,7 @@ protected void finalize() throws Throwable {
   /** @hide */
   @VisibleForTesting
   @NonNull
-  String getName() {
+  public String getName() {
     return name;
   }
 
diff --git a/firebase-perf/src/main/java/com/google/firebase/perf/util/ScreenTraceUtil.java b/firebase-perf/src/main/java/com/google/firebase/perf/util/ScreenTraceUtil.java
@@ -46,7 +46,7 @@ public static Trace addFrameCounters(Trace screenTrace, PerfFrameMetrics perfFra
     }
     logger.debug(
         "Screen trace: "
-            + screenTrace.getClass().getSimpleName()
+            + screenTrace.getName()
             + " _fr_tot:"
             + perfFrameMetrics.getTotalFrames()
             + " _fr_slo:"

Original file line number	Diff line number	Diff line change
`@@ -498,7 +498,7 @@ protected void finalize() throws Throwable {`
`498`	`498`	`/** @hide */`
`499`	`499`	`@VisibleForTesting`
`500`	`500`	`@NonNull`
`501`		`- String getName() {`
	`501`	`+ public String getName() {`
`502`	`502`	`return name;`
`503`	`503`	`}`
`504`	`504`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ public static Trace addFrameCounters(Trace screenTrace, PerfFrameMetrics perfFra`
`46`	`46`	`}`
`47`	`47`	`logger.debug(`
`48`	`48`	`"Screen trace: "`
`49`		`- + screenTrace.getClass().getSimpleName()`
	`49`	`+ + screenTrace.getName()`
`50`	`50`	`+ " _fr_tot:"`
`51`	`51`	`+ perfFrameMetrics.getTotalFrames()`
`52`	`52`	`+ " _fr_slo:"`