Merge branch 'main' into rl.agp.8.3.2

Author: rlazo

firebase-components/firebase-dynamic-module-support/firebase-dynamic-module-support.gradle.kts

@@ -49,8 +49,8 @@ android {
 }
 
 dependencies {
-  implementation("com.google.android.play:feature-delivery:2.0.0")
+  implementation(libs.feature.delivery)
   implementation(libs.kotlin.stdlib.jdk8)
-  api("com.google.firebase:firebase-common:21.0.0")
-  api("com.google.firebase:firebase-components:18.0.0")
+  api(libs.firebase.common)
+  api(libs.firebase.components)
 }

firebase-crashlytics/CHANGELOG.md

@@ -1,5 +1,5 @@
 # Unreleased
-
+* [fixed] Execute failure listener outside the main thread  [#6535]
 
 # 19.2.1
 * [changed] Updated protobuf dependency to `3.25.5` to fix
@@ -634,4 +634,3 @@ The following release notes describe changes in the new SDK.
   from your `AndroidManifest.xml` file.
 * [removed] The `fabric.properties` and `crashlytics.properties` files are no
   longer supported. Remove them from your app.
-

firebase-crashlytics/src/main/java/com/google/firebase/crashlytics/CrashlyticsRegistrar.java

@@ -18,6 +18,7 @@
 import com.google.firebase.analytics.connector.AnalyticsConnector;
 import com.google.firebase.annotations.concurrent.Background;
 import com.google.firebase.annotations.concurrent.Blocking;
+import com.google.firebase.annotations.concurrent.Lightweight;
 import com.google.firebase.components.Component;
 import com.google.firebase.components.ComponentContainer;
 import com.google.firebase.components.ComponentRegistrar;
@@ -42,6 +43,8 @@ public class CrashlyticsRegistrar implements ComponentRegistrar {
       Qualified.qualified(Background.class, ExecutorService.class);
   private final Qualified<ExecutorService> blockingExecutorService =
       Qualified.qualified(Blocking.class, ExecutorService.class);
+  private final Qualified<ExecutorService> lightweightExecutorService =
+      Qualified.qualified(Lightweight.class, ExecutorService.class);
 
   static {
     // Add Crashlytics as a dependency of Sessions when this class is loaded into memory.
@@ -57,6 +60,7 @@ public List<Component<?>> getComponents() {
             .add(Dependency.required(FirebaseInstallationsApi.class))
             .add(Dependency.required(backgroundExecutorService))
             .add(Dependency.required(blockingExecutorService))
+            .add(Dependency.required(lightweightExecutorService))
             .add(Dependency.deferred(CrashlyticsNativeComponent.class))
             .add(Dependency.deferred(AnalyticsConnector.class))
             .add(Dependency.deferred(FirebaseRemoteConfigInterop.class))
@@ -79,7 +83,8 @@ private FirebaseCrashlytics buildCrashlytics(ComponentContainer container) {
             container.getDeferred(AnalyticsConnector.class),
             container.getDeferred(FirebaseRemoteConfigInterop.class),
             container.get(backgroundExecutorService),
-            container.get(blockingExecutorService));
+            container.get(blockingExecutorService),
+            container.get(lightweightExecutorService));
 
     long duration = System.currentTimeMillis() - startTime;
     if (duration > 16) {

firebase-crashlytics/src/main/java/com/google/firebase/crashlytics/FirebaseCrashlytics.java

@@ -67,7 +67,8 @@ public class FirebaseCrashlytics {
       @NonNull Deferred<AnalyticsConnector> analyticsConnector,
       @NonNull Deferred<FirebaseRemoteConfigInterop> remoteConfigInteropDeferred,
       ExecutorService backgroundExecutorService,
-      ExecutorService blockingExecutorService) {
+      ExecutorService blockingExecutorService,
+      ExecutorService lightExecutorService) {
 
     Context context = app.getApplicationContext();
     final String appIdentifier = context.getPackageName();
@@ -160,7 +161,8 @@ public class FirebaseCrashlytics {
     // Kick off actually fetching the settings.
     settingsController
         .loadSettingsData(crashlyticsWorkers)
-        .addOnFailureListener(ex -> Logger.getLogger().e("Error fetching settings.", ex));
+        .addOnFailureListener(
+            lightExecutorService, ex -> Logger.getLogger().e("Error fetching settings.", ex));
 
     final boolean finishCoreInBackground = core.onPreExecute(appData, settingsController);
 

firebase-functions/CHANGELOG.md

@@ -1,4 +1,7 @@
 # Unreleased
+* [fixed] Fixed HttpsCallableResult.data resolution in Kotlin
+
+# 21.1.0
 * [changed] Migrated to Kotlin
 
 # 21.0.0

firebase-functions/api.txt

@@ -110,6 +110,7 @@ package com.google.firebase.functions {
 
   public final class HttpsCallableResult {
     method @Nullable public Object getData();
+    field @Nullable public final Object data;
   }
 
   public final class Serializer {

firebase-functions/src/androidTest/java/com/google/firebase/functions/ktx/CallTests.kt

@@ -64,7 +64,7 @@ class CallTests {
       )
 
     var function = functions.getHttpsCallable("dataTest")
-    val actual = Tasks.await(function.call(input)).getData()
+    val actual = Tasks.await(function.call(input)).data
 
     assertThat(actual).isInstanceOf(Map::class.java)
     @Suppress("UNCHECKED_CAST") val map = actual as Map<String, *>
@@ -77,7 +77,7 @@ class CallTests {
   fun testNullDataCall() {
     val functions = Firebase.functions(app)
     var function = functions.getHttpsCallable("nullTest")
-    val actual = Tasks.await(function.call(null)).getData()
+    val actual = Tasks.await(function.call(null)).data
 
     assertThat(actual).isNull()
   }
@@ -86,7 +86,7 @@ class CallTests {
   fun testEmptyDataCall() {
     val functions = Firebase.functions(app)
     var function = functions.getHttpsCallable("nullTest")
-    val actual = Tasks.await(function.call()).getData()
+    val actual = Tasks.await(function.call()).data
 
     assertThat(actual).isNull()
   }

firebase-functions/src/main/java/com/google/firebase/functions/HttpsCallableResult.kt

@@ -16,13 +16,21 @@ package com.google.firebase.functions
 /** The result of calling a HttpsCallableReference function. */
 public class HttpsCallableResult
 internal constructor( // The actual result data, as generic types decoded from JSON.
-private val data: Any?) {
+  /**
+   * The data that was returned from the Callable HTTPS trigger.
+   *
+   * The data is in the form of native Java objects. For example, if your trigger returned an array,
+   * this object would be a `List<Object>`. If your trigger returned a JavaScript object with keys
+   * and values, this object would be a `Map<String, Object>`.
+   */
+  @JvmField public val data: Any?
+) {
   /**
    * Returns the data that was returned from the Callable HTTPS trigger.
    *
    * The data is in the form of native Java objects. For example, if your trigger returned an array,
-   * this object would be a List<Object>. If your trigger returned a JavaScript object with keys and
-   * values, this object would be a Map<String, Object>.
+   * this object would be a `List<Object>`. If your trigger returned a JavaScript object with keys
+   * and values, this object would be a `Map<String, Object>`.
    */
   public fun getData(): Any? {
     return data

gradle/libs.versions.toml

@@ -7,6 +7,9 @@ android-lint = "31.3.2"
 autovalue = "1.10.1"
 coroutines = "1.7.3"
 dagger = "2.43.2"
+featureDelivery = "2.1.0"
+firebaseCommon = "21.0.0"
+firebaseComponents = "18.0.0"
 grpc = "1.62.2"
 grpcKotlin = "1.4.1"
 javalite = "3.25.5"
@@ -45,7 +48,10 @@ autovalue-annotations = { module = "com.google.auto.value:auto-value-annotations
 dagger-dagger = { module = "com.google.dagger:dagger", version.ref = "dagger" }
 dagger-compiler = { module = "com.google.dagger:dagger-compiler", version.ref = "dagger" }
 errorprone-annotations = { module = "com.google.errorprone:error_prone_annotations", version = "2.26.0" }
+feature-delivery = { module = "com.google.android.play:feature-delivery", version.ref = "featureDelivery" }
 findbugs-jsr305 = { module = "com.google.code.findbugs:jsr305", version = "3.0.2" }
+firebase-common = { module = "com.google.firebase:firebase-common", version.ref = "firebaseCommon" }
+firebase-components = { module = "com.google.firebase:firebase-components", version.ref = "firebaseComponents" }
 grpc-android = { module = "io.grpc:grpc-android", version.ref = "grpc" }
 grpc-kotlin-stub = { module = "io.grpc:grpc-kotlin-stub", version.ref = "grpcKotlin" }
 grpc-okhttp = { module = "io.grpc:grpc-okhttp", version.ref = "grpc" }