send event and RRC mitigation

Author: leotianlizhan

firebase-perf/src/main/java/com/google/firebase/perf/config/ConfigResolver.java

@@ -23,8 +23,8 @@
 import com.google.firebase.perf.BuildConfig;
 import com.google.firebase.perf.config.ConfigurationConstants.CollectionDeactivated;
 import com.google.firebase.perf.config.ConfigurationConstants.CollectionEnabled;
-import com.google.firebase.perf.config.ConfigurationConstants.FragmentSamplingRate;
 import com.google.firebase.perf.config.ConfigurationConstants.ExperimentTTID;
+import com.google.firebase.perf.config.ConfigurationConstants.FragmentSamplingRate;
 import com.google.firebase.perf.config.ConfigurationConstants.LogSourceName;
 import com.google.firebase.perf.config.ConfigurationConstants.NetworkEventCountBackground;
 import com.google.firebase.perf.config.ConfigurationConstants.NetworkEventCountForeground;
@@ -115,11 +115,21 @@ public void setMetadataBundle(ImmutableBundle bundle) {
 
   /** Default API to call for whether performance monitoring is currently silent. */
   public boolean isPerformanceMonitoringEnabled() {
+    saveExperimentFlagToDeviceCache();
     Boolean isPerformanceCollectionEnabled = getIsPerformanceCollectionEnabled();
     return (isPerformanceCollectionEnabled == null || isPerformanceCollectionEnabled == true)
         && getIsServiceCollectionEnabled();
   }
 
+  // TODO: remove after _experiment_as_ttid is finished
+  private void saveExperimentFlagToDeviceCache() {
+    ExperimentTTID config = ExperimentTTID.getInstance();
+    Optional<Boolean> rcValue = getRemoteConfigBoolean(config);
+    if (rcValue.isAvailable()) {
+      deviceCacheManager.setValue(config.getDeviceCacheFlag(), rcValue.get());
+    }
+  }
+
   /** Returns whether developers have enabled Firebase Performance event collection. */
   @Nullable
   public Boolean getIsPerformanceCollectionEnabled() {
@@ -772,7 +782,7 @@ public float getFragmentSamplingRate() {
   public boolean getIsExperimentTTIDEnabled() {
     // Order of precedence is:
     // 1. If the value exists in Android Manifest, return this value.
-    // 2. If the value exists through Firebase Remote Config, cache and return this value.
+    // 2. Cannot read value from RC because it's not initalized yet
     // 3. If the value exists in device cache, return this value.
     // 4. Otherwise, return default value.
     ExperimentTTID config = ExperimentTTID.getInstance();
@@ -783,13 +793,7 @@ public boolean getIsExperimentTTIDEnabled() {
       return metadataValue.get();
     }
 
-    // 2. Reads value from Firebase Remote Config, saves this value in cache layer if valid.
-    Optional<Boolean> rcValue = getRemoteConfigBoolean(config);
-    if (rcValue.isAvailable()) {
-      deviceCacheManager.setValue(config.getDeviceCacheFlag(), rcValue.get());
-      return rcValue.get();
-    }
-
+    // 2. Cannot read value from RC because it's not initialized yet.
     // 3. Reads value from cache layer.
     Optional<Boolean> deviceCacheValue = getDeviceCacheBoolean(config);
     if (deviceCacheValue.isAvailable()) {

firebase-perf/src/main/java/com/google/firebase/perf/metrics/AppStartTrace.java

@@ -22,7 +22,6 @@
 import android.os.Bundle;
 import android.os.Process;
 import android.view.View;
-
 import androidx.annotation.Keep;
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
@@ -41,8 +40,6 @@
 import com.google.firebase.perf.v1.TraceMetric;
 import java.lang.ref.WeakReference;
 import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collections;
 import java.util.List;
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.LinkedBlockingQueue;
@@ -205,7 +202,8 @@ private void recordFirstDrawDone() {
       return;
     }
     this.firstDrawDone = clock.getTime();
-    executorService.execute(() -> this.logColdStart(getStartTimer(), this.firstDrawDone, this.startSession));
+    executorService.execute(
+        () -> this.logColdStart(getStartTimer(), this.firstDrawDone, this.startSession));
 
     if (isRegisteredForLifecycleCallbacks) {
       // After AppStart trace is queued to be logged, we can unregister this callback.
@@ -241,8 +239,7 @@ public synchronized void onActivityStarted(Activity activity) {
 
   @Override
   public synchronized void onActivityResumed(Activity activity) {
-    if (isStartedFromBackground
-        || isTooLateToInitUI) {
+    if (isStartedFromBackground || isTooLateToInitUI) {
       return;
     }
 
@@ -281,10 +278,10 @@ private void logColdStart(Timer start, Timer end, PerfSession session) {
             .setDurationUs(start.getDurationMicros(end));
 
     TraceMetric.Builder subtrace =
-            TraceMetric.newBuilder()
-                    .setName("_experiment_classLoadTime")
-                    .setClientStartTimeUs(FirebasePerfProvider.getAppStartTime().getMicros())
-                    .setDurationUs(FirebasePerfProvider.getAppStartTime().getDurationMicros(end));
+        TraceMetric.newBuilder()
+            .setName("_experiment_classLoadTime")
+            .setClientStartTimeUs(FirebasePerfProvider.getAppStartTime().getMicros())
+            .setDurationUs(FirebasePerfProvider.getAppStartTime().getDurationMicros(end));
 
     metric.addSubtraces(subtrace).addPerfSessions(this.startSession.build());