Update logging message with proper logging levels. (#2706)

* Update logging levels.

* Update NetworkRequestMetricBuilder.java

* Address comments.

Author: jeremyjiang-dev

firebase-perf/src/main/java/com/google/firebase/perf/FirebasePerformance.java

@@ -193,7 +193,7 @@ public static FirebasePerformance getInstance() {
 
     mPerformanceCollectionForceEnabledState = configResolver.getIsPerformanceCollectionEnabled();
     if (isPerformanceCollectionEnabled()) {
-      logger.debug(
+      logger.info(
           String.format(
               "Firebase Performance Monitoring is successfully initialized! In a minute, visit the Firebase console to view your data: %s",
               ConsoleUrlGenerator.generateDashboardUrl(

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

@@ -70,7 +70,7 @@ public final class NetworkRequestMetricBuilder extends AppStateUpdateHandler
   public void updateSession(PerfSession session) {
     // Note(b/152218504): Being defensive to fix the NPE
     if (session == null) {
-      logger.info("Unable to add new SessionId to the Network Trace. Continuing without it.");
+      logger.warn("Unable to add new SessionId to the Network Trace. Continuing without it.");
       return;
     }
 
@@ -293,7 +293,7 @@ public NetworkRequestMetricBuilder setResponseContentType(@Nullable String conte
     if (isValidContentType(contentType)) {
       builder.setResponseContentType(contentType);
     } else {
-      logger.info("The content type of the response is not a valid content-type:" + contentType);
+      logger.warn("The content type of the response is not a valid content-type:" + contentType);
     }
     return this;
   }
@@ -332,7 +332,7 @@ public NetworkRequestMetric build() {
     }
 
     if (isManualNetworkRequestMetric) {
-      logger.info(
+      logger.debug(
           "This metric has already been queued for transmission.  "
               + "Please create a new HttpMetric for each request/response");
     }

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

@@ -75,7 +75,7 @@ public class Trace extends AppStateUpdateHandler
   public void updateSession(PerfSession session) {
     // Note(b/152218504): Being defensive to fix the NPE
     if (session == null) {
-      logger.info("Unable to add new SessionId to the Trace. Continuing without it.");
+      logger.warn("Unable to add new SessionId to the Trace. Continuing without it.");
       return;
     }
 
@@ -205,7 +205,7 @@ private Trace(@NonNull Parcel in, boolean isDataOnly) {
   @Keep
   public void start() {
     if (!ConfigResolver.getInstance().isPerformanceMonitoringEnabled()) {
-      logger.info("Trace feature is disabled.");
+      logger.debug("Trace feature is disabled.");
       return;
     }
 

firebase-perf/src/main/java/com/google/firebase/perf/metrics/validator/FirebasePerfNetworkValidator.java

@@ -48,87 +48,87 @@ final class FirebasePerfNetworkValidator extends PerfMetricValidator {
    */
   public boolean isValidPerfMetric() {
     if (isEmptyUrl(networkMetric.getUrl())) {
-      logger.info("URL is missing:" + networkMetric.getUrl());
+      logger.warn("URL is missing:" + networkMetric.getUrl());
       return false;
     }
     URI uri = getResultUrl(networkMetric.getUrl());
     if (uri == null) {
-      logger.info("URL cannot be parsed");
+      logger.warn("URL cannot be parsed");
       return false;
     }
 
     if (!isAllowlisted(uri, appContext)) {
-      logger.info("URL fails allowlist rule: " + uri);
+      logger.warn("URL fails allowlist rule: " + uri);
       return false;
     }
     if (!isValidHost(uri.getHost())) {
-      logger.info("URL host is null or invalid");
+      logger.warn("URL host is null or invalid");
       return false;
     }
     if (!isValidScheme(uri.getScheme())) {
-      logger.info("URL scheme is null or invalid");
+      logger.warn("URL scheme is null or invalid");
       return false;
     }
     if (!isValidUserInfo(uri.getUserInfo())) {
-      logger.info("URL user info is null");
+      logger.warn("URL user info is null");
       return false;
     }
     if (!isValidPort(uri.getPort())) {
-      logger.info("URL port is less than or equal to 0");
+      logger.warn("URL port is less than or equal to 0");
       return false;
     }
     if (!isValidHttpMethod(networkMetric.hasHttpMethod() ? networkMetric.getHttpMethod() : null)) {
-      logger.info("HTTP Method is null or invalid: " + networkMetric.getHttpMethod());
+      logger.warn("HTTP Method is null or invalid: " + networkMetric.getHttpMethod());
       return false;
     }
     if (networkMetric.hasHttpResponseCode()
         && !isValidHttpResponseCode(networkMetric.getHttpResponseCode())) {
-      logger.info("HTTP ResponseCode is a negative value:" + networkMetric.getHttpResponseCode());
+      logger.warn("HTTP ResponseCode is a negative value:" + networkMetric.getHttpResponseCode());
       return false;
     }
     if (networkMetric.hasRequestPayloadBytes()
         && !isValidPayload(networkMetric.getRequestPayloadBytes())) {
-      logger.info("Request Payload is a negative value:" + networkMetric.getRequestPayloadBytes());
+      logger.warn("Request Payload is a negative value:" + networkMetric.getRequestPayloadBytes());
       return false;
     }
     if (networkMetric.hasResponsePayloadBytes()
         && !isValidPayload(networkMetric.getResponsePayloadBytes())) {
-      logger.info(
+      logger.warn(
           "Response Payload is a negative value:" + networkMetric.getResponsePayloadBytes());
       return false;
     }
     if (!networkMetric.hasClientStartTimeUs() || networkMetric.getClientStartTimeUs() <= 0) {
-      logger.info(
+      logger.warn(
           "Start time of the request is null, or zero, or a negative value:"
               + networkMetric.getClientStartTimeUs());
       return false;
     }
     if (networkMetric.hasTimeToRequestCompletedUs()
         && !isValidTime(networkMetric.getTimeToRequestCompletedUs())) {
-      logger.info(
+      logger.warn(
           "Time to complete the request is a negative value:"
               + networkMetric.getTimeToRequestCompletedUs());
       return false;
     }
     if (networkMetric.hasTimeToResponseInitiatedUs()
         && !isValidTime(networkMetric.getTimeToResponseInitiatedUs())) {
-      logger.info(
+      logger.warn(
           "Time from the start of the request to the start of the response is null or a "
               + "negative value:"
               + networkMetric.getTimeToResponseInitiatedUs());
       return false;
     }
     if (!networkMetric.hasTimeToResponseCompletedUs()
         || networkMetric.getTimeToResponseCompletedUs() <= 0) {
-      logger.info(
+      logger.warn(
           "Time from the start of the request to the end of the response is null, negative or "
               + "zero:"
               + networkMetric.getTimeToResponseCompletedUs());
       return false;
     }
     // Don't log any requests with a connection error set
     if (!networkMetric.hasHttpResponseCode()) {
-      logger.info("Did not receive a HTTP Response Code");
+      logger.warn("Did not receive a HTTP Response Code");
       return false;
     }
     return true;

firebase-perf/src/main/java/com/google/firebase/perf/util/URLAllowlist.java

@@ -36,7 +36,7 @@ public static boolean isURLAllowlisted(@NonNull URI uri, @NonNull Context appCon
     }
 
     AndroidLogger.getInstance()
-        .info("Detected domain allowlist, only allowlisted domains will be measured.");
+        .debug("Detected domain allowlist, only allowlisted domains will be measured.");
     if (allowlistedDomains == null) {
       allowlistedDomains = resources.getStringArray(resourceId);
     }