Move continuations off main thread for firebase-appcheck-playintegrity.

Author: rosalyntan

appcheck/firebase-appcheck-playintegrity/src/main/java/com/google/firebase/appcheck/playintegrity/FirebaseAppCheckPlayIntegrityRegistrar.java

@@ -17,6 +17,7 @@
 import com.google.android.gms.common.annotation.KeepForSdk;
 import com.google.firebase.FirebaseApp;
 import com.google.firebase.annotations.concurrent.Blocking;
+import com.google.firebase.annotations.concurrent.Lightweight;
 import com.google.firebase.appcheck.playintegrity.internal.PlayIntegrityAppCheckProvider;
 import com.google.firebase.components.Component;
 import com.google.firebase.components.ComponentRegistrar;
@@ -39,17 +40,21 @@ public class FirebaseAppCheckPlayIntegrityRegistrar implements ComponentRegistra
 
   @Override
   public List<Component<?>> getComponents() {
+    Qualified<Executor> liteExecutor = Qualified.qualified(Lightweight.class, Executor.class);
     Qualified<Executor> blockingExecutor = Qualified.qualified(Blocking.class, Executor.class);
 
     return Arrays.asList(
         Component.builder(PlayIntegrityAppCheckProvider.class)
             .name(LIBRARY_NAME)
             .add(Dependency.required(FirebaseApp.class))
+            .add(Dependency.required(liteExecutor))
             .add(Dependency.required(blockingExecutor))
             .factory(
                 (container) ->
                     new PlayIntegrityAppCheckProvider(
-                        container.get(FirebaseApp.class), container.get(blockingExecutor)))
+                        container.get(FirebaseApp.class),
+                        container.get(liteExecutor),
+                        container.get(blockingExecutor)))
             .build(),
         LibraryVersionComponent.create(LIBRARY_NAME, BuildConfig.VERSION_NAME));
   }

appcheck/firebase-appcheck-playintegrity/src/main/java/com/google/firebase/appcheck/playintegrity/internal/PlayIntegrityAppCheckProvider.java

@@ -14,7 +14,6 @@
 
 package com.google.firebase.appcheck.playintegrity.internal;
 
-import android.annotation.SuppressLint;
 import androidx.annotation.NonNull;
 import androidx.annotation.VisibleForTesting;
 import com.google.android.gms.tasks.Task;
@@ -25,6 +24,7 @@
 import com.google.android.play.core.integrity.IntegrityTokenResponse;
 import com.google.firebase.FirebaseApp;
 import com.google.firebase.annotations.concurrent.Blocking;
+import com.google.firebase.annotations.concurrent.Lightweight;
 import com.google.firebase.appcheck.AppCheckProvider;
 import com.google.firebase.appcheck.AppCheckToken;
 import com.google.firebase.appcheck.internal.DefaultAppCheckToken;
@@ -39,15 +39,19 @@ public class PlayIntegrityAppCheckProvider implements AppCheckProvider {
   private final String projectNumber;
   private final IntegrityManager integrityManager;
   private final NetworkClient networkClient;
+  private final Executor liteExecutor;
   private final Executor blockingExecutor;
   private final RetryManager retryManager;
 
   public PlayIntegrityAppCheckProvider(
-      @NonNull FirebaseApp firebaseApp, @Blocking Executor blockingExecutor) {
+      @NonNull FirebaseApp firebaseApp,
+      @Lightweight Executor liteExecutor,
+      @Blocking Executor blockingExecutor) {
     this(
         firebaseApp.getOptions().getGcmSenderId(),
         IntegrityManagerFactory.create(firebaseApp.getApplicationContext()),
         new NetworkClient(firebaseApp),
+        liteExecutor,
         blockingExecutor,
         new RetryManager());
   }
@@ -57,22 +61,23 @@ public PlayIntegrityAppCheckProvider(
       @NonNull String projectNumber,
       @NonNull IntegrityManager integrityManager,
       @NonNull NetworkClient networkClient,
+      @NonNull Executor liteExecutor,
       @NonNull Executor blockingExecutor,
       @NonNull RetryManager retryManager) {
     this.projectNumber = projectNumber;
     this.integrityManager = integrityManager;
     this.networkClient = networkClient;
+    this.liteExecutor = liteExecutor;
     this.blockingExecutor = blockingExecutor;
     this.retryManager = retryManager;
   }
 
-  // TODO(b/261013814): Use an explicit executor in continuations.
-  @SuppressLint("TaskMainThread")
   @NonNull
   @Override
   public Task<AppCheckToken> getToken() {
     return getPlayIntegrityAttestation()
         .onSuccessTask(
+            liteExecutor,
             integrityTokenResponse -> {
               ExchangePlayIntegrityTokenRequest request =
                   new ExchangePlayIntegrityTokenRequest(integrityTokenResponse.token());
@@ -85,32 +90,30 @@ public Task<AppCheckToken> getToken() {
                           retryManager));
             })
         .onSuccessTask(
-            appCheckTokenResponse -> {
-              return Tasks.forResult(
-                  DefaultAppCheckToken.constructFromAppCheckTokenResponse(appCheckTokenResponse));
-            });
+            liteExecutor,
+            appCheckTokenResponse ->
+                Tasks.forResult(
+                    DefaultAppCheckToken.constructFromAppCheckTokenResponse(
+                        appCheckTokenResponse)));
   }
 
-  // TODO(b/261013814): Use an explicit executor in continuations.
-  @SuppressLint("TaskMainThread")
   @NonNull
   private Task<IntegrityTokenResponse> getPlayIntegrityAttestation() {
     GeneratePlayIntegrityChallengeRequest generateChallengeRequest =
         new GeneratePlayIntegrityChallengeRequest();
-    Task<GeneratePlayIntegrityChallengeResponse> generateChallengeTask =
-        Tasks.call(
+    return Tasks.call(
             blockingExecutor,
             () ->
                 GeneratePlayIntegrityChallengeResponse.fromJsonString(
                     networkClient.generatePlayIntegrityChallenge(
-                        generateChallengeRequest.toJsonString().getBytes(UTF_8), retryManager)));
-    return generateChallengeTask.onSuccessTask(
-        generatePlayIntegrityChallengeResponse -> {
-          return integrityManager.requestIntegrityToken(
-              IntegrityTokenRequest.builder()
-                  .setCloudProjectNumber(Long.parseLong(projectNumber))
-                  .setNonce(generatePlayIntegrityChallengeResponse.getChallenge())
-                  .build());
-        });
+                        generateChallengeRequest.toJsonString().getBytes(UTF_8), retryManager)))
+        .onSuccessTask(
+            liteExecutor,
+            generatePlayIntegrityChallengeResponse ->
+                integrityManager.requestIntegrityToken(
+                    IntegrityTokenRequest.builder()
+                        .setCloudProjectNumber(Long.parseLong(projectNumber))
+                        .setNonce(generatePlayIntegrityChallengeResponse.getChallenge())
+                        .build()));
   }
 }

appcheck/firebase-appcheck-playintegrity/src/test/java/com/google/firebase/appcheck/playintegrity/FirebaseAppCheckPlayIntegrityRegistrarTest.java

@@ -18,6 +18,7 @@
 
 import com.google.firebase.FirebaseApp;
 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.Dependency;
 import com.google.firebase.components.Qualified;
@@ -40,6 +41,7 @@ public void testGetComponents() {
     assertThat(appCheckPlayIntegrityComponent.getDependencies())
         .containsExactly(
             Dependency.required(FirebaseApp.class),
+            Dependency.required(Qualified.qualified(Lightweight.class, Executor.class)),
             Dependency.required(Qualified.qualified(Blocking.class, Executor.class)));
     assertThat(appCheckPlayIntegrityComponent.isLazy()).isTrue();
   }

appcheck/firebase-appcheck-playintegrity/src/test/java/com/google/firebase/appcheck/playintegrity/internal/PlayIntegrityAppCheckProviderTest.java

@@ -72,7 +72,7 @@ public class PlayIntegrityAppCheckProviderTest {
   @Captor private ArgumentCaptor<byte[]> exchangePlayIntegrityTokenRequestCaptor;
 
   // TODO(b/258273630): Use TestOnlyExecutors instead of MoreExecutors.directExecutor().
-  private Executor backgroundExecutor = MoreExecutors.directExecutor();
+  private Executor executor = MoreExecutors.directExecutor();
 
   @Before
   public void setup() {
@@ -87,7 +87,8 @@ public void testPublicConstructor_nullFirebaseApp_expectThrows() {
     assertThrows(
         NullPointerException.class,
         () -> {
-          new PlayIntegrityAppCheckProvider(null, TestOnlyExecutors.blocking());
+          new PlayIntegrityAppCheckProvider(
+              null, TestOnlyExecutors.lite(), TestOnlyExecutors.blocking());
         });
   }
 
@@ -106,7 +107,8 @@ public void getToken_onSuccess_setsTaskResult() throws Exception {
             PROJECT_NUMBER,
             mockIntegrityManager,
             mockNetworkClient,
-            backgroundExecutor,
+            /* liteExecutor= */ executor,
+            /* blockingExecutor= */ executor,
             mockRetryManager);
     Task<AppCheckToken> task = provider.getToken();
 
@@ -141,7 +143,8 @@ public void getToken_generateChallengeFails_setsTaskException() throws Exception
             PROJECT_NUMBER,
             mockIntegrityManager,
             mockNetworkClient,
-            backgroundExecutor,
+            /* liteExecutor= */ executor,
+            /* blockingExecutor= */ executor,
             mockRetryManager);
     Task<AppCheckToken> task = provider.getToken();
 
@@ -165,7 +168,8 @@ public void getToken_requestIntegrityTokenFails_setsTaskException() throws Excep
             PROJECT_NUMBER,
             mockIntegrityManager,
             mockNetworkClient,
-            backgroundExecutor,
+            /* liteExecutor= */ executor,
+            /* blockingExecutor= */ executor,
             mockRetryManager);
     Task<AppCheckToken> task = provider.getToken();
 
@@ -196,7 +200,8 @@ public void getToken_tokenExchangeFails_setsTaskException() throws Exception {
             PROJECT_NUMBER,
             mockIntegrityManager,
             mockNetworkClient,
-            backgroundExecutor,
+            /* liteExecutor= */ executor,
+            /* blockingExecutor= */ executor,
             mockRetryManager);
     Task<AppCheckToken> task = provider.getToken();