weblayer: ignore version checks in some code paths

This allows calling these functions when running under roboelectric.
This is necessary as when running under roboelectric the implementation
is not loaded and it doesn't run on a device.

BUG=1128312
TEST=none

Change-Id: Ib96cd68b8ff22f0046d6a96f8b1ca6ba203f418c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2414775Reviewed-by: Bo <boliu@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#807964}

weblayer/public/java/org/chromium/weblayer/NavigateParams.java

@@ -59,7 +59,8 @@ public class NavigateParams {
          */
         @NonNull
         public Builder disableIntentProcessing() {
-            if (WebLayer.getSupportedMajorVersionInternal() < 86) {
+            if (WebLayer.shouldPerformVersionChecks()
+                    && WebLayer.getSupportedMajorVersionInternal() < 86) {
                 throw new UnsupportedOperationException();
             }
             mParams.mIntentProcessingDisabled = true;
@@ -74,7 +75,8 @@ public class NavigateParams {
          */
         @NonNull
         public Builder disableNetworkErrorAutoReload() {
-            if (WebLayer.getSupportedMajorVersionInternal() < 86) {
+            if (WebLayer.shouldPerformVersionChecks()
+                    && WebLayer.getSupportedMajorVersionInternal() < 86) {
                 throw new UnsupportedOperationException();
             }
             mParams.mNetworkErrorAutoReloadDisabled = true;
@@ -103,7 +105,8 @@ public class NavigateParams {
      * @since 86
      */
     public boolean isIntentProcessingDisabled() {
-        if (WebLayer.getSupportedMajorVersionInternal() < 86) {
+        if (WebLayer.shouldPerformVersionChecks()
+                && WebLayer.getSupportedMajorVersionInternal() < 86) {
             throw new UnsupportedOperationException();
         }
         return mIntentProcessingDisabled;
@@ -117,7 +120,8 @@ public class NavigateParams {
      * @since 86
      */
     public boolean isNetworkErrorAutoReloadDisabled() {
-        if (WebLayer.getSupportedMajorVersionInternal() < 86) {
+        if (WebLayer.shouldPerformVersionChecks()
+                && WebLayer.getSupportedMajorVersionInternal() < 86) {
             throw new UnsupportedOperationException();
         }
         return mNetworkErrorAutoReloadDisabled;

weblayer/public/java/org/chromium/weblayer/WebLayer.java

@@ -191,6 +191,14 @@ public class WebLayer {
         return getWebLayerLoader(context).getMajorVersion();
     }
 
+    // Returns true if version checks should be done. This is provided solely for testing, and
+    // specifically testing that does not run on device and load the implementation. It is only
+    // necessary to check this in code paths that don't require WebLayer to load the implementation
+    // and need to be callable in tests.
+    static boolean shouldPerformVersionChecks() {
+        return !"robolectric".equals(Build.FINGERPRINT);
+    }
+
     // Internal version of getSupportedMajorVersion(). This should only be used when you know
     // WebLayer has been initialized. Generally that means calling this from any non-static method.
     static int getSupportedMajorVersionInternal() {