Make java doc tense and language more consistent

Author: Rachel Prince

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

@@ -79,7 +79,7 @@ private void startInstallActivity(String path, Activity currentActivity) {
     Intent intent = new Intent(currentActivity, InstallActivity.class);
     intent.putExtra("INSTALL_PATH", path);
     currentActivity.startActivity(intent);
-    LogWrapper.getInstance().v(TAG + "Prompting user with install activity ");
+    LogWrapper.getInstance().v(TAG + "Prompting tester with install activity ");
   }
 
   void trySetInstallTaskError() {

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

@@ -30,19 +30,19 @@ static Builder builder() {
     return new com.google.firebase.appdistribution.AutoValue_AppDistributionRelease.Builder();
   }
 
-  /** The short bundle version of this build (example: 1.0.0). */
+  /** Returns the short bundle version of this build (example: 1.0.0). */
   @NonNull
   public abstract String getDisplayVersion();
 
-  /** The version code of this build (example: 123). */
+  /** Returns the version code of this build (example: 123). */
   @NonNull
   public abstract long getVersionCode();
 
-  /** The release notes for this build. */
+  /** Returns the release notes for this build. */
   @Nullable
   public abstract String getReleaseNotes();
 
-  /** The binary type for this build. */
+  /** Returns the binary type for this build. */
   @NonNull
   public abstract BinaryType getBinaryType();
 

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

@@ -141,15 +141,16 @@ public static FirebaseAppDistribution getInstance() {
   /**
    * Updates the app to the newest release, if one is available.
    *
-   * <p>Returns the release information or null if no update is found. Performs the following
-   * actions:
+   * <p>Returns the release information or {@code null} if no update is found. Performs the
+   * following actions:
    *
    * <ol>
-   *   <li>If tester is not signed in, presents the tester with a Google sign in UI.
+   *   <li>If tester is not signed in, presents the tester with a Google Sign-in UI.
    *   <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 Google Play app to complete the download and installation.
+   *   <li>If the newest release is an APK, downloads the binary and starts an installation. If the
+   *       newest release is an AAB, directs the tester to the Play app to complete the download and
+   *       installation.
    * </ol>
    */
   @NonNull

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

@@ -31,18 +31,20 @@ public enum Status {
      */
     AUTHENTICATION_FAILURE,
 
-    /** The authentication process was canceled (typically by the user). */
+    /** The authentication process was canceled (typically by the tester). */
     AUTHENTICATION_CANCELED,
 
     /**
-     * No network is available to make requests, or the request timed out. Check your internet
-     * connection and try again.
+     * No network was available to make requests, or the request timed out. Check the tester's
+     * internet connection and retrying the call to {@link FirebaseAppDistribution}.
      */
     NETWORK_FAILURE,
 
     /**
-     * The new release failed to download. This is a most likely due to a transient condition and
-     * may be corrected by retrying.
+     * The new release failed to download. This was a most likely due to a transient condition and
+     * may be corrected by retrying the call to {@link
+     * FirebaseAppDistribution#updateIfNewReleaseAvailable} or {@link
+     * FirebaseAppDistribution#updateApp}.
      */
     DOWNLOAD_FAILURE,
 
@@ -52,13 +54,13 @@ public enum Status {
      */
     INSTALLATION_FAILURE,
 
-    /** The installation was canceled (typically by the user). */
+    /** The installation was canceled (typically by the tester). */
     INSTALLATION_CANCELED,
 
     /**
-     * An update is not available for the current tester and app. Make sure that {@link
-     * FirebaseAppDistribution#checkForNewRelease()} returns with a non-null {@link
-     * AppDistributionRelease} before calling {@link FirebaseAppDistribution#updateApp()},
+     * An update was not available for the current tester and app. Make sure that {@link
+     * FirebaseAppDistribution#checkForNewRelease} returns with a non-null {@link
+     * AppDistributionRelease} before calling {@link FirebaseAppDistribution#updateApp},
      */
     UPDATE_NOT_AVAILABLE,
 
@@ -128,7 +130,7 @@ static class ErrorMessages {
         "Failed to fetch releases due to unknown network error.";
 
     public static final String JSON_PARSING_ERROR =
-        "Error parsing service response when checking for new release. This is a most likely due to a transient condition and may be corrected by retrying.";
+        "Error parsing service response when checking for new release. This was most likely due to a transient condition and may be corrected by retrying.";
 
     public static final String AUTHENTICATION_ERROR =
         "Failed to authenticate the tester. The tester was either not signed in, or something went wrong. Try signing in again.";
@@ -139,17 +141,17 @@ static class ErrorMessages {
     public static final String AUTHENTICATION_CANCELED = "Tester canceled the authentication flow.";
 
     public static final String NOT_FOUND_ERROR =
-        "Release not found. An update is not available for the current tester and app. Make sure that FirebaseAppDistribution#checkForNewRelease returns with a non-null  AppDistributionRelease before calling FirebaseAppDistribution#updateApp";
+        "Release not found. An update was not available for the current tester and app. Make sure that FirebaseAppDistribution#checkForNewRelease returns with a non-null  AppDistributionRelease before calling FirebaseAppDistribution#updateApp";
 
     public static final String TIMEOUT_ERROR =
-        "Failed to fetch releases due to timeout. Check your internet connection and try again.";
+        "Failed to fetch releases due to timeout. Check the tester's internet connection and try again.";
 
     public static final String UPDATE_CANCELED = "Tester canceled the update.";
 
     public static final String UNKNOWN_ERROR = "Unknown error.";
 
     public static final String DOWNLOAD_URL_NOT_FOUND =
-        "Download URL not found. This is a most likely due to a transient condition and may be corrected by retrying.";
+        "Download URL not found. This was a most likely due to a transient condition and may be corrected by retrying.";
 
     public static final String HOST_ACTIVITY_INTERRUPTED =
         "Host activity interrupted while dialog was showing. Try calling FirebaseAppDistribution#updateIfNewReleaseAvailable again.";

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

@@ -38,7 +38,8 @@ interface Operation<TResult> {
    *
    * <p>This allows long running operations to be chained together using {@link Task#onSuccessTask}.
    * If the operation throws an exception, the task will fail, and the exception will be surfaced to
-   * the user as {@code FirebaseAppDistributionException}, available via {@link Task#getException}.
+   * the caller as {@code FirebaseAppDistributionException}, available via {@link
+   * Task#getException}.
    *
    * <p>Exceptions that are not {@code FirebaseAppDistributionException} will be wrapped in one,
    * with {@link Status#UNKNOWN}.

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

@@ -28,7 +28,7 @@ static UpdateProgress.Builder builder() {
   }
 
   /**
-   * The number of bytes downloaded so far for an APK.
+   * Returns the number of bytes downloaded so far for an APK.
    *
    * @returns the number of bytes downloaded, or -1 if called when updating to an AAB or if no new
    *     release is available.
@@ -37,7 +37,7 @@ static UpdateProgress.Builder builder() {
   public abstract long getApkBytesDownloaded();
 
   /**
-   * The file size of the APK file to download in bytes.
+   * Returns the file size of the APK file to download in bytes.
    *
    * @returns the file size in bytes, or -1 if called when updating to an AAB or if no new release
    *     is available.

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/internal/InstallActivity.java

@@ -47,7 +47,7 @@ protected void onResume() {
     // Since we kick-off installation with FLAG_ACTIVITY_NEW_TASK (in a new task), we won't be able
     // to figure out if installation failed or was canceled.
     // If we re-enter InstallActivity after install is already kicked off, we can assume that either
-    // installation failure or user canceled the install.
+    // installation failure or tester canceled the install.
     if (installInProgress) {
       LogWrapper.getInstance()
           .e(

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/internal/SignInResultActivity.java

@@ -28,7 +28,7 @@ public class SignInResultActivity extends AppCompatActivity {
   @Override
   public void onCreate(@NonNull Bundle savedInstanceBundle) {
     super.onCreate(savedInstanceBundle);
-    LogWrapper.getInstance().v(TAG + "The User is signing in");
+    LogWrapper.getInstance().v(TAG + "The tester is signing in");
     // While this does not appear to be achieving much, handling the redirect in this way
     // ensures that we can remove the browser tab from the back stack. See the documentation
     // on AuthorizationManagementActivity for more details.