Skip to content

Commit c08ebb7

Browse files
Fix hasFrameMetricsAggregator's value not being set.
1 parent 2fed9e4 commit c08ebb7

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

firebase-perf/src/main/java/com/google/firebase/perf/application/AppStateMonitor.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ public class AppStateMonitor implements ActivityLifecycleCallbacks {
5353
"androidx.core.app.FrameMetricsAggregator";
5454

5555
private static volatile AppStateMonitor instance;
56+
private static boolean hasFrameMetricsAggregator = false;
5657

5758
private final WeakHashMap<Activity, Boolean> activityToResumedMap = new WeakHashMap<>();
5859
private final WeakHashMap<Activity, Trace> activityToScreenTraceMap = new WeakHashMap<>();
@@ -76,7 +77,6 @@ public class AppStateMonitor implements ActivityLifecycleCallbacks {
7677

7778
private boolean isRegisteredForLifecycleCallbacks = false;
7879
private boolean isColdStart = true;
79-
private boolean hasFrameMetricsAggregator = false;
8080

8181
public static AppStateMonitor getInstance() {
8282
if (instance == null) {
@@ -450,8 +450,10 @@ protected boolean isScreenTraceSupported() {
450450
private static boolean hasFrameMetricsAggregatorClass() {
451451
try {
452452
Class<?> initializerClass = Class.forName(FRAME_METRICS_AGGREGATOR_CLASSNAME);
453+
hasFrameMetricsAggregator = true;
453454
return true;
454455
} catch (ClassNotFoundException e) {
456+
hasFrameMetricsAggregator = false;
455457
return false;
456458
}
457459
}

firebase-perf/src/main/java/com/google/firebase/perf/application/FragmentStateMonitor.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,7 @@ public void onFragmentPaused(@NonNull FragmentManager fm, @NonNull Fragment f) {
8383
// Stop Fragment screen trace
8484
logger.debug("FragmentMonitor %s.onFragmentPaused ", f.getClass().getSimpleName());
8585
if (!fragmentToTraceMap.containsKey(f)) {
86-
logger.warn(
87-
"FragmentMonitor: missed a fragment trace from %s", f.getClass().getSimpleName());
86+
logger.warn("FragmentMonitor: missed a fragment trace from %s", f.getClass().getSimpleName());
8887
return;
8988
}
9089

0 commit comments

Comments
 (0)