Fix interface declaration for logging status

Also renamed param from status to errorCode.

Bug: 1005802
Change-Id: Iee6991724fb10b9e30ca7919ae21e5a4829e3324
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1838415
Commit-Queue: Fred Mello <fredmello@chromium.org>
Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#702702}

components/module_installer/android/java/src/org/chromium/components/module_installer/logger/Logger.java

@@ -5,6 +5,7 @@
 package org.chromium.components.module_installer.logger;
 
 import com.google.android.play.core.splitinstall.model.SplitInstallErrorCode;
+import com.google.android.play.core.splitinstall.model.SplitInstallSessionStatus;
 
 /**
  * Logger for SplitCompat Engine.
@@ -14,17 +15,17 @@ public interface Logger {
      * Logs exceptions that happen during module request.
      *
      * @param moduleName The module name.
-     * @param status The error code.
+     * @param errorCode The error code.
      */
-    void logRequestFailure(String moduleName, @SplitInstallErrorCode int status);
+    void logRequestFailure(String moduleName, @SplitInstallErrorCode int errorCode);
 
     /**
      * Logs exceptions that happen during the installation process.
      *
      * @param moduleName The module name.
-     * @param status The error code.
+     * @param errorCode The error code.
      */
-    void logStatusFailure(String moduleName, @SplitInstallErrorCode int status);
+    void logStatusFailure(String moduleName, @SplitInstallErrorCode int errorCode);
 
     /**
      * Logs the status count and duration during a module installation process.
@@ -32,7 +33,7 @@ public interface Logger {
      * @param moduleName The module name
      * @param status The status code
      */
-    void logStatus(String moduleName, @SplitInstallErrorCode int status);
+    void logStatus(String moduleName, @SplitInstallSessionStatus int status);
 
     /**
      * Logs the request start time.

components/module_installer/android/java/src/org/chromium/components/module_installer/logger/PlayCoreLogger.java

@@ -28,13 +28,13 @@ public class PlayCoreLogger implements Logger {
     }
 
     @Override
-    public void logRequestFailure(String moduleName, @SplitInstallErrorCode int status) {
-        mFailureLogger.logRequestFailure(moduleName, status);
+    public void logRequestFailure(String moduleName, @SplitInstallErrorCode int errorCode) {
+        mFailureLogger.logRequestFailure(moduleName, errorCode);
     }
 
     @Override
-    public void logStatusFailure(String moduleName, @SplitInstallErrorCode int status) {
-        mFailureLogger.logStatusFailure(moduleName, status);
+    public void logStatusFailure(String moduleName, @SplitInstallErrorCode int errorCode) {
+        mFailureLogger.logStatusFailure(moduleName, errorCode);
     }
 
     @Override

components/module_installer/android/java/src/org/chromium/components/module_installer/logger/SplitInstallFailureLogger.java

@@ -87,13 +87,13 @@ class SplitInstallFailureLogger {
         log(moduleName, NO_SPLITCOMPAT);
     }
 
-    public void logStatusFailure(String moduleName, @SplitInstallErrorCode int status) {
-        Integer code = getHistogramCode(status);
+    public void logStatusFailure(String moduleName, @SplitInstallErrorCode int errorCode) {
+        Integer code = getHistogramCode(errorCode);
         log(moduleName, code == -1 ? UNKNOWN_SPLITINSTALL_ERROR : code);
     }
 
-    public void logRequestFailure(String moduleName, @SplitInstallErrorCode int status) {
-        Integer code = getHistogramCode(status);
+    public void logRequestFailure(String moduleName, @SplitInstallErrorCode int errorCode) {
+        Integer code = getHistogramCode(errorCode);
         log(moduleName, code == -1 ? UNKNOWN_REQUEST_ERROR : code);
     }