Enable global custom attributes on Network Requests (#3399)

Author: visumickey

firebase-perf/CHANGELOG.md

@@ -20,6 +20,8 @@ Refer [GMaven](https://maven.google.com/web/index.html?q=firebase-perf#com.googl
 > **Note:** Refer go/firebase-android-release for `MXX` info.
 
 ## Unreleased
+*   {{feature}} Enable global custom attributes on Network Requests
+*   {{fixed}} Update log statement to differentiate event drop because of rate limiting and sampling.
 
 ## Released
 

firebase-perf/src/main/java/com/google/firebase/perf/transport/TransportManager.java

@@ -502,7 +502,7 @@ private PerfMetric setApplicationInfoAndBuild(
     ApplicationInfo.Builder appInfoBuilder =
         applicationInfoBuilder.setApplicationProcessState(appState);
 
-    if (perfMetricBuilder.hasTraceMetric()) {
+    if (perfMetricBuilder.hasTraceMetric() || perfMetricBuilder.hasNetworkRequestMetric()) {
       appInfoBuilder =
           appInfoBuilder
               .clone() // Needed so that we don't add global custom attributes everywhere

firebase-perf/src/test/java/com/google/firebase/perf/transport/TransportManagerTest.java

@@ -1119,7 +1119,7 @@ public void logTraceMetric_globalCustomAttributesAreAdded() {
   }
 
   @Test
-  public void logNetworkMetric_globalCustomAttributesAreNotAdded() {
+  public void logNetworkMetric_globalCustomAttributesAreAdded() {
     FirebasePerformance.getInstance().putAttribute("test_key1", "test_value1");
     FirebasePerformance.getInstance().putAttribute("test_key2", "test_value2");
     NetworkRequestMetric validNetworkRequest = createValidNetworkRequestMetric();
@@ -1131,7 +1131,12 @@ public void logNetworkMetric_globalCustomAttributesAreNotAdded() {
     assertThat(loggedPerfMetric.getNetworkRequestMetric()).isEqualTo(validNetworkRequest);
     validateApplicationInfo(
         loggedPerfMetric, ApplicationProcessState.APPLICATION_PROCESS_STATE_UNKNOWN);
-    assertThat(loggedPerfMetric.getApplicationInfo().getCustomAttributesCount()).isEqualTo(0);
+
+    Map<String, String> globalCustomAttributes =
+        loggedPerfMetric.getApplicationInfo().getCustomAttributesMap();
+    assertThat(globalCustomAttributes).hasSize(2);
+    assertThat(globalCustomAttributes).containsEntry("test_key1", "test_value1");
+    assertThat(globalCustomAttributes).containsEntry("test_key2", "test_value2");
   }
 
   @Test