gjf

Author: leotianlizhan

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

@@ -148,8 +148,7 @@ public void onActivityCreated(Activity activity, Bundle savedInstanceState) {
         fragmentActivity
             .getSupportFragmentManager()
             .registerFragmentLifecycleCallbacks(
-                new FragmentStateMonitor(
-                    clock, transportManager, this, frameMetricsAggregator),
+                new FragmentStateMonitor(clock, transportManager, this, frameMetricsAggregator),
                 true);
       }
     }

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

@@ -23,7 +23,6 @@
 import com.google.firebase.perf.transport.TransportManager;
 import com.google.firebase.perf.util.Clock;
 import com.google.firebase.perf.util.Constants;
-
 import java.util.WeakHashMap;
 
 public class FragmentStateMonitor extends FragmentManager.FragmentLifecycleCallbacks {
@@ -60,14 +59,18 @@ public void onFragmentResumed(@NonNull FragmentManager fm, @NonNull Fragment f)
     super.onFragmentResumed(fm, f);
     // Start Fragment screen trace
     logger.debug("FragmentMonitor %s.onFragmentResumed", f.getClass().getSimpleName());
-    Trace fragmentTrace = new Trace(getFragmentScreenTraceName(f), transportManager, clock, appStateMonitor);
+    Trace fragmentTrace =
+        new Trace(getFragmentScreenTraceName(f), transportManager, clock, appStateMonitor);
     fragmentTrace.start();
 
     if (f.getParentFragment() != null) {
-      fragmentTrace.putAttribute(Constants.PARENT_FRAGMENT_ATTRIBUTE_KEY, f.getParentFragment().getClass().getSimpleName());
+      fragmentTrace.putAttribute(
+          Constants.PARENT_FRAGMENT_ATTRIBUTE_KEY,
+          f.getParentFragment().getClass().getSimpleName());
     }
     if (f.getActivity() != null) {
-      fragmentTrace.putAttribute(Constants.ACTIVITY_ATTRIBUTE_KEY, f.getActivity().getClass().getSimpleName());
+      fragmentTrace.putAttribute(
+          Constants.ACTIVITY_ATTRIBUTE_KEY, f.getActivity().getClass().getSimpleName());
     }
 
     fragmentToTraceMap.put(f, fragmentTrace);
@@ -79,7 +82,8 @@ public void onFragmentPaused(@NonNull FragmentManager fm, @NonNull Fragment f) {
     // Stop Fragment screen trace
     logger.debug("FragmentMonitor %s.onFragmentPaused ", f.getClass().getSimpleName());
     if (!fragmentToTraceMap.containsKey(f)) {
-      logger.error("FragmentMonitor: missed a fragment trace from %s", f.getClass().getSimpleName());
+      logger.error(
+          "FragmentMonitor: missed a fragment trace from %s", f.getClass().getSimpleName());
     }
 
     Trace fragmentTrace = fragmentToTraceMap.get(f);

firebase-perf/src/test/java/com/google/firebase/perf/application/FragmentStateMonitorTest.java

@@ -1,19 +1,16 @@
 package com.google.firebase.perf.application;
 
 import static org.mockito.ArgumentMatchers.any;
-import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.doAnswer;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
 import static org.mockito.MockitoAnnotations.initMocks;
 
 import android.app.Activity;
-
 import androidx.appcompat.app.AppCompatActivity;
 import androidx.core.app.FrameMetricsAggregator;
 import androidx.fragment.app.Fragment;
 import androidx.fragment.app.FragmentManager;
-
 import com.google.firebase.perf.FirebasePerformanceTestBase;
 import com.google.firebase.perf.config.ConfigResolver;
 import com.google.firebase.perf.config.DeviceCacheManager;
@@ -22,8 +19,6 @@
 import com.google.firebase.perf.util.Timer;
 import com.google.firebase.perf.v1.TraceMetric;
 import com.google.testing.timing.FakeDirectExecutorService;
-
-import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
@@ -45,8 +40,7 @@ public class FragmentStateMonitorTest extends FirebasePerformanceTestBase {
   @Mock private AppStateMonitor appStateMonitor;
   @Mock private FrameMetricsAggregator fma;
 
-  @Captor
-  private ArgumentCaptor<TraceMetric> argTraceMetric;
+  @Captor private ArgumentCaptor<TraceMetric> argTraceMetric;
 
   private long currentTime = 0;
 
@@ -63,16 +57,16 @@ public void setUp() {
 
     ConfigResolver configResolver = ConfigResolver.getInstance();
     configResolver.setDeviceCacheManager(new DeviceCacheManager(new FakeDirectExecutorService()));
-
   }
 
   @Test
   public void fragmentLifecycleCallbacks_logFragmentScreenTrace() {
-    FragmentStateMonitor monitor = new FragmentStateMonitor(clock, mockTransportManager, appStateMonitor, fma);
+    FragmentStateMonitor monitor =
+        new FragmentStateMonitor(clock, mockTransportManager, appStateMonitor, fma);
     monitor.onFragmentResumed(mockfragmentManager, mockFragment);
     verify(mockTransportManager, times(0)).log(any(TraceMetric.class), any());
 
     monitor.onFragmentPaused(mockfragmentManager, mockFragment);
     verify(mockTransportManager, times(1)).log(any(TraceMetric.class), any());
   }
-}
+}