[ForwardingCookieHandler] Adds Logic To Catch MissingWebViewPackageEx…

…ception

Summary:
We are seeing crash reports that the webview is missing. In this case
it should fail gracefully so that a missing webview does not block
from using an app built with React Native.

The contains could also be changed to check for "webview" in general
to catch all webview related exception. It's currenlty checking on
for the specific string that I'm seeing in our app's crashreporting tool.

ReactAndroid/src/main/java/com/facebook/react/modules/network/ForwardingCookieHandler.java

@@ -182,6 +182,18 @@ protected void doInBackgroundGuarded(Void... params) {
  } catch (IllegalArgumentException ex) {
  // https://bugs.chromium.org/p/chromium/issues/detail?id=559720
  return null;
+ } catch (Exception exception) {
+ String message = exception.getMessage();
+ // We cannot catch MissingWebViewPackageException as it is in a private / system API
+ // class. This validates the exception's message to ensure we are only handling this
+ // specific exception.
+ // https://android.googlesource.com/platform/frameworks/base/+/master/core/java/android/
+ // webkit/WebViewFactory.java#98
+ if (message != null && message.contains("No WebView installed")) {
+ return null;
+ } else {
+ throw exception;
+ }
  }
 
  if (USES_LEGACY_STORE) {