Extract 250ms timeout to a constant

Author: dvicory

android/src/main/java/com/reactnativecommunity/webview/RNCWebViewManager.java

@@ -13,6 +13,7 @@
 import android.net.Uri;
 import android.os.Build;
 import android.os.Environment;
+import android.os.SystemClock;
 import android.text.TextUtils;
 import android.util.Log;
 import android.view.Gravity;
@@ -88,7 +89,6 @@
 import java.util.HashMap;
 import java.util.Locale;
 import java.util.Map;
-import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicReference;
 
 /**
@@ -141,6 +141,7 @@ public class RNCWebViewManager extends SimpleViewManager<WebView> {
   // Use `webView.loadUrl("about:blank")` to reliably reset the view
   // state and release page resources (including any running JavaScript).
   protected static final String BLANK_URL = "about:blank";
+  protected static final int SHOULD_OVERRIDE_URL_LOADING_TIMEOUT = 250;
   protected WebViewConfig mWebViewConfig;
 
   protected RNCWebChromeClient mWebChromeClient = null;
@@ -806,14 +807,14 @@ public boolean shouldOverrideUrlLoading(WebView view, String url) {
         try {
           assert lockObject != null;
           synchronized (lockObject) {
-            final long startTime = System.nanoTime();
+            final long startTime = SystemClock.elapsedRealtime();
             while (lockObject.get() == ShouldOverrideCallbackState.UNDECIDED) {
-              if (TimeUnit.NANOSECONDS.toMillis(System.nanoTime() - startTime) > 250) {
+              if (SystemClock.elapsedRealtime() - startTime > SHOULD_OVERRIDE_URL_LOADING_TIMEOUT) {
                 FLog.w(TAG, "Did not receive response to shouldOverrideUrlLoading in time, defaulting to allow loading.");
                 RNCWebViewModule.shouldOverrideUrlLoadingLock.removeLock(lockIdentifier);
                 return false;
               }
-              lockObject.wait(250);
+              lockObject.wait(SHOULD_OVERRIDE_URL_LOADING_TIMEOUT);
             }
           }
         } catch (InterruptedException e) {