Skip to content

Commit 44d6fb8

Browse files
committed
Add annotations and rearrange declarations and initializations
1 parent 6b2e8eb commit 44d6fb8

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/impl/FirebaseAppDistributionImpl.java

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -66,14 +66,14 @@ class FirebaseAppDistributionImpl implements FirebaseAppDistribution {
6666
private final ApkUpdater apkUpdater;
6767
private final AabUpdater aabUpdater;
6868
private final SignInStorage signInStorage;
69-
private final SequentialReference<AppDistributionReleaseInternal> cachedNewRelease;
7069
private final ReleaseIdentifier releaseIdentifier;
7170
private final ScreenshotTaker screenshotTaker;
72-
@Blocking private final Executor blockingExecutor;
7371
@Lightweight private Executor lightweightExecutor;
74-
private final FirebaseAppDistributionNotificationsManager notificationsManager;
75-
private UpdateTaskCache updateIfNewReleaseAvailableTaskCache;
72+
@Blocking private final Executor blockingExecutor;
73+
private final SequentialReference<AppDistributionReleaseInternal> cachedNewRelease;
7674
private TaskCache<AppDistributionRelease> checkForNewReleaseTaskCache;
75+
private UpdateTaskCache updateIfNewReleaseAvailableTaskCache;
76+
private final FirebaseAppDistributionNotificationsManager notificationsManager;
7777
private AlertDialog updateConfirmationDialog;
7878
private AlertDialog signInConfirmationDialog;
7979
@Nullable private Activity dialogHostActivity = null;
@@ -95,7 +95,7 @@ class FirebaseAppDistributionImpl implements FirebaseAppDistribution {
9595
@NonNull ReleaseIdentifier releaseIdentifier,
9696
@NonNull ScreenshotTaker screenshotTaker,
9797
@NonNull @Lightweight Executor lightweightExecutor,
98-
@NonNull Executor blockingExecutor) {
98+
@NonNull @Blocking Executor blockingExecutor) {
9999
this.firebaseApp = firebaseApp;
100100
this.testerSignInManager = testerSignInManager;
101101
this.newReleaseFetcher = newReleaseFetcher;
@@ -104,14 +104,14 @@ class FirebaseAppDistributionImpl implements FirebaseAppDistribution {
104104
this.signInStorage = signInStorage;
105105
this.releaseIdentifier = releaseIdentifier;
106106
this.lifecycleNotifier = lifecycleNotifier;
107-
this.cachedNewRelease = new SequentialReference<>(lightweightExecutor);
108-
this.lightweightExecutor = lightweightExecutor;
109107
this.screenshotTaker = screenshotTaker;
108+
this.lightweightExecutor = lightweightExecutor;
110109
this.blockingExecutor = blockingExecutor;
110+
this.cachedNewRelease = new SequentialReference<>(lightweightExecutor);
111+
this.checkForNewReleaseTaskCache = new TaskCache<>(lightweightExecutor);
112+
this.updateIfNewReleaseAvailableTaskCache = new UpdateTaskCache(lightweightExecutor);
111113
this.notificationsManager =
112114
new FirebaseAppDistributionNotificationsManager(firebaseApp.getApplicationContext());
113-
this.updateIfNewReleaseAvailableTaskCache = new UpdateTaskCache(lightweightExecutor);
114-
this.checkForNewReleaseTaskCache = new TaskCache<>(lightweightExecutor);
115115
lifecycleNotifier.addOnActivityDestroyedListener(this::onActivityDestroyed);
116116
lifecycleNotifier.addOnActivityPausedListener(this::onActivityPaused);
117117
lifecycleNotifier.addOnActivityResumedListener(this::onActivityResumed);

0 commit comments

Comments
 (0)