Small syntax fixes; Standardize use of period in java docs

Author: Rachel Prince

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/AppDistributionRelease.java

@@ -30,7 +30,7 @@ static Builder builder() {
     return new com.google.firebase.appdistribution.AutoValue_AppDistributionRelease.Builder();
   }
 
-  /** The short bundle version of this build (example 1.0.0). */
+  /** The short bundle version of this build (example: 1.0.0). */
   @NonNull
   public abstract String getDisplayVersion();
 

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/AppDistributionReleaseInternal.java

@@ -19,12 +19,7 @@
 import com.google.auto.value.AutoValue;
 
 /**
- * This class represents the AppDistributionRelease object returned by the App Distribution backend
- *
- * <p>It is an immutable value class implemented by AutoValue.
- *
- * @see <a
- *     href="https://github.com/google/auto/tree/master/value">https://github.com/google/auto/tree/master/value</a>
+ * This class represents the AppDistributionRelease object returned by the App Distribution backend.
  */
 @AutoValue
 abstract class AppDistributionReleaseInternal {
@@ -34,38 +29,38 @@ static Builder builder() {
     return new AutoValue_AppDistributionReleaseInternal.Builder();
   }
 
-  /** The short bundle version of this build (example 1.0.0) */
+  /** The short bundle version of this build (example: 1.0.0). */
   @NonNull
   abstract String getDisplayVersion();
 
-  /** The bundle version of this build (example: 123) */
+  /** The bundle version of this build (example: 123). */
   @NonNull
   abstract String getBuildVersion();
 
-  /** The release notes for this build */
+  /** The release notes for this build. */
   @Nullable
   abstract String getReleaseNotes();
 
-  /** The binary type for this build */
+  /** The binary type for this build. */
   @NonNull
   abstract BinaryType getBinaryType();
 
-  /** Hash of binary of an Android app */
+  /** Hash of binary of an Android app. */
   @Nullable
   abstract String getCodeHash();
 
-  /** Efficient hash of an Android apk. Used to identify a release */
+  /** Efficient hash of an Android apk. Used to identify a release. */
   @Nullable
   abstract String getApkHash();
 
   /**
    * IAS artifact id. This value is inserted into the manifest of APK's installed via Used to map a
-   * release to an APK installed via an app bundle
+   * release to an APK installed via an app bundle.
    */
   @Nullable
   abstract String getIasArtifactId();
 
-  /** Short-lived download URL */
+  /** Short-lived download URL. */
   @Nullable
   abstract String getDownloadUrl();
 

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/BinaryType.java

@@ -14,7 +14,7 @@
 
 package com.google.firebase.appdistribution;
 
-/** Enum of Android app binary types, used in {@link AppDistributionRelease} */
+/** Enum of Android app binary types, used in {@link AppDistributionRelease}. */
 public enum BinaryType {
   /** Android App Bundle. */
   AAB,

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/FirebaseAppDistribution.java

@@ -81,7 +81,7 @@ public class FirebaseAppDistribution {
   private TaskCompletionSource<Void> showSignInDialogTask = null;
   private TaskCompletionSource<Void> showUpdateDialogTask = null;
 
-  /** Constructor for FirebaseAppDistribution */
+  /** Constructor for FirebaseAppDistribution. */
   @VisibleForTesting
   FirebaseAppDistribution(
       @NonNull FirebaseApp firebaseApp,
@@ -103,7 +103,7 @@ public class FirebaseAppDistribution {
     lifecycleNotifier.addOnActivityResumedListener(this::onActivityResumed);
   }
 
-  /** Constructor for FirebaseAppDistribution */
+  /** Constructor for FirebaseAppDistribution. */
   FirebaseAppDistribution(
       @NonNull FirebaseApp firebaseApp,
       @NonNull Provider<FirebaseInstallationsApi> firebaseInstallationsApiProvider,
@@ -122,7 +122,7 @@ public class FirebaseAppDistribution {
         lifecycleNotifier);
   }
 
-  /** Constructor for FirebaseAppDistribution */
+  /** Constructor for FirebaseAppDistribution. */
   FirebaseAppDistribution(
       @NonNull FirebaseApp firebaseApp,
       @NonNull Provider<FirebaseInstallationsApi> firebaseInstallationsApiProvider) {
@@ -150,7 +150,7 @@ public static FirebaseAppDistribution getInstance() {
    *   <li>Checks if a newer release is available. If so, presents the tester with a confirmation
    *       dialog to begin the download.
    *   <li>For APKs, downloads the binary and starts an installation intent. For AABs, directs the
-   *       tester to the Play app to complete the download and installation.
+   *       tester to the Google Play app to complete the download and installation.
    * </ol>
    */
   @NonNull
@@ -246,7 +246,7 @@ private Task<Void> showSignInConfirmationDialog(Activity hostActivity) {
     return showSignInDialogTask.getTask();
   }
 
-  /** Signs in the App Distribution tester. Presents the tester with a Google sign in UI */
+  /** Signs in the App Distribution tester. Presents the tester with a Google sign in UI. */
   @NonNull
   public Task<Void> signInTester() {
     return this.testerSignInManager.signInTester();
@@ -326,7 +326,7 @@ private UpdateTask updateApp(boolean showDownloadInNotificationManager) {
                 Constants.ErrorMessages.NOT_FOUND_ERROR, UPDATE_NOT_AVAILABLE));
       }
       if (cachedNewRelease.getDownloadUrl() == null) {
-        LogWrapper.getInstance().v("Download failed to execute");
+        LogWrapper.getInstance().v("Download failed to execute.");
         return getErrorUpdateTask(
             new FirebaseAppDistributionException(
                 Constants.ErrorMessages.DOWNLOAD_URL_NOT_FOUND,
@@ -346,7 +346,7 @@ public boolean isTesterSignedIn() {
     return this.signInStorage.getSignInStatus();
   }
 
-  /** Signs out the App Distribution tester */
+  /** Signs out the App Distribution tester. */
   public void signOutTester() {
     setCachedNewRelease(null);
     this.signInStorage.setSignInStatus(false);

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/FirebaseAppDistributionException.java

@@ -32,7 +32,7 @@ public enum Status {
     /** The authentication process was canceled. */
     AUTHENTICATION_CANCELED,
 
-    /** No network is available to make requests or the request timed out. */
+    /** No network is available to make requests, or the request timed out. */
     NETWORK_FAILURE,
 
     /** The new release failed to download. */

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/FirebaseAppDistributionLifecycleNotifier.java

@@ -45,23 +45,23 @@ interface ActivityConsumer<T> {
   @GuardedBy("lock")
   private Activity currentActivity;
 
-  /** A queue of listeners that trigger when the activity is foregrounded */
+  /** A queue of listeners that trigger when the activity is foregrounded. */
   @GuardedBy("lock")
   private final Queue<OnActivityCreatedListener> onActivityCreatedListeners = new ArrayDeque<>();
 
-  /** A queue of listeners that trigger when the activity is foregrounded */
+  /** A queue of listeners that trigger when the activity is foregrounded. */
   @GuardedBy("lock")
   private final Queue<OnActivityStartedListener> onActivityStartedListeners = new ArrayDeque<>();
 
-  /** A queue of listeners that trigger when the activity is resumed */
+  /** A queue of listeners that trigger when the activity is resumed. */
   @GuardedBy("lock")
   private final Queue<OnActivityResumedListener> onActivityResumedListeners = new ArrayDeque<>();
 
-  /** A queue of listeners that trigger when the activity is backgrounded */
+  /** A queue of listeners that trigger when the activity is backgrounded. */
   @GuardedBy("lock")
   private final Queue<OnActivityPausedListener> onActivityPausedListeners = new ArrayDeque<>();
 
-  /** A queue of listeners that trigger when the activity is destroyed */
+  /** A queue of listeners that trigger when the activity is destroyed. */
   @GuardedBy("lock")
   private final Queue<OnActivityDestroyedListener> onDestroyedListeners = new ArrayDeque<>();
 

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/FirebaseAppDistributionRegistrar.java

@@ -30,7 +30,7 @@
 import java.util.List;
 
 /**
- * Registers FirebaseAppDistribution
+ * Registers FirebaseAppDistribution.
  *
  * @hide
  */

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/FirebaseAppDistributionTesterApiClient.java

@@ -69,7 +69,7 @@ class FirebaseAppDistributionTesterApiClient {
   }
 
   /**
-   * Fetches and returns the lastest release for the app that the tester has access to, or null if
+   * Fetches and returns the latest release for the app that the tester has access to, or null if
    * the tester doesn't have access to any releases.
    */
   @Nullable

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/ReleaseIdentificationUtils.java

@@ -37,9 +37,9 @@ final class ReleaseIdentificationUtils {
   private static final int NO_FLAGS = 0;
 
   /**
-   * Get the package info for the currently installed app
+   * Get the package info for the currently installed app.
    *
-   * @throws FirebaseAppDistributionException if the package name can't be found
+   * @throws FirebaseAppDistributionException if the package name can't be found.
    */
   static PackageInfo getPackageInfo(Context context) throws FirebaseAppDistributionException {
     return getPackageInfoWithFlags(context, NO_FLAGS);
@@ -49,7 +49,7 @@ static PackageInfo getPackageInfo(Context context) throws FirebaseAppDistributio
    * Get the package info for the currently installed app, with the PackageManager.GET_META_DATA
    * flag set.
    *
-   * @throws FirebaseAppDistributionException if the package name can't be found
+   * @throws FirebaseAppDistributionException if the package name can't be found.
    */
   static PackageInfo getPackageInfoWithMetadata(Context context)
       throws FirebaseAppDistributionException {

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/TesterSignInManager.java

@@ -42,7 +42,7 @@
 import com.google.firebase.installations.FirebaseInstallationsApi;
 import java.util.List;
 
-/** Class that handles signing in the tester */
+/** Class that handles signing in the tester. */
 class TesterSignInManager {
   private static final String TAG = "TesterSignInManager:";
   private static final String SIGNIN_REDIRECT_URL =