override onRenderProcessGone

lib/src/main/java/com/shopify/checkoutkit/CheckoutDialog.kt

@@ -107,7 +107,9 @@ internal class CheckoutDialog(
 
         setOnCancelListener {
             checkoutEventProcessor.onCheckoutCanceled()
-            (checkoutWebView.parent as ViewGroup).removeView(checkoutWebView)
+            checkoutWebView.parent?.let {
+                (checkoutWebView.parent as ViewGroup).removeView(checkoutWebView)
+            }
         }
 
         header.setOnMenuItemClickListener {

lib/src/main/java/com/shopify/checkoutkit/CheckoutWebView.kt

@@ -26,13 +26,16 @@ import android.annotation.SuppressLint
 import android.content.Context
 import android.graphics.Color.TRANSPARENT
 import android.net.Uri
+import android.os.Build
 import android.os.Handler
 import android.os.Looper
 import android.util.AttributeSet
 import android.view.KeyEvent
 import android.view.View
+import android.view.ViewGroup
 import android.view.ViewGroup.LayoutParams
 import android.view.ViewGroup.LayoutParams.MATCH_PARENT
+import android.webkit.RenderProcessGoneDetail
 import android.webkit.WebResourceError
 import android.webkit.WebResourceRequest
 import android.webkit.WebResourceResponse
@@ -116,6 +119,19 @@ internal class CheckoutWebView(context: Context, attributeSet: AttributeSet? = n
             checkoutBridge.getEventProcessor().onCheckoutViewLoadComplete()
         }
 
+        override fun onRenderProcessGone(view: WebView, detail: RenderProcessGoneDetail): Boolean {
+            return if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O && !detail.didCrash()) {
+                // Renderer was killed because system ran out of memory.
+
+                // Removing the view from `CheckoutWebViewContainer will trigger a cache clear
+                // and call webView.destroy()
+                (view.parent as ViewGroup).removeView(view)
+                true
+            } else {
+                false
+            }
+        }
+
         override fun shouldOverrideUrlLoading(
             view: WebView?,
             request: WebResourceRequest?

lib/src/test/java/com/shopify/checkoutkit/CheckoutWebViewClientTest.kt

@@ -23,6 +23,7 @@
 package com.shopify.checkoutkit
 
 import android.net.Uri
+import android.webkit.RenderProcessGoneDetail
 import android.webkit.WebResourceError
 import android.webkit.WebResourceRequest
 import android.webkit.WebResourceResponse
@@ -39,6 +40,7 @@ import org.mockito.kotlin.verify
 import org.mockito.kotlin.whenever
 import org.robolectric.Robolectric
 import org.robolectric.RobolectricTestRunner
+import org.robolectric.annotation.Config
 
 @RunWith(RobolectricTestRunner::class)
 class CheckoutWebViewClientTest {
@@ -147,6 +149,47 @@ class CheckoutWebViewClientTest {
         verify(checkoutWebViewEventProcessor).onCheckoutViewLoadComplete()
     }
 
+    @Test
+    fun `onRenderProcessGone should return false if sdk version is too low to check detail#didCrash()`() {
+        val view = viewWithProcessor(activity).withParent()
+        val webViewClient = view.CheckoutWebViewClient()
+        val detail = mock<RenderProcessGoneDetail>()
+        whenever(detail.didCrash()).thenReturn(false)
+
+        val result = webViewClient.onRenderProcessGone(view, detail)
+
+        assertThat(result).isFalse
+        assertThat(view.parent).isNotNull
+    }
+
+    @Config(sdk = [26])
+    @Test
+    fun `onRenderProcessGone should do nothing if the renderer crashed`() {
+        val view = viewWithProcessor(activity).withParent()
+        val webViewClient = view.CheckoutWebViewClient()
+        val detail = mock<RenderProcessGoneDetail>()
+        whenever(detail.didCrash()).thenReturn(true)
+
+        val result = webViewClient.onRenderProcessGone(view, detail)
+
+        assertThat(result).isFalse
+        assertThat(view.parent).isNotNull
+    }
+
+    @Config(sdk = [26])
+    @Test
+    fun `onRenderProcessGone should remove the view from its parent if the render process crashed due to low memory`() {
+        val view = viewWithProcessor(activity).withParent()
+        val webViewClient = view.CheckoutWebViewClient()
+        val detail = mock<RenderProcessGoneDetail>()
+        whenever(detail.didCrash()).thenReturn(false)
+
+        val result = webViewClient.onRenderProcessGone(view, detail)
+
+        assertThat(result).isTrue
+        assertThat(view.parent).isNull()
+    }
+
     private fun mockWebRequest(uri: Uri, forMainFrame: Boolean = false): WebResourceRequest {
         val mockRequest = mock<WebResourceRequest>()
         whenever(mockRequest.url).thenReturn(uri)
@@ -181,4 +224,10 @@ class CheckoutWebViewClientTest {
         whenever(mock.reasonPhrase).thenReturn(description)
         return mock
     }
+
+    private fun CheckoutWebView.withParent(): CheckoutWebView {
+        val container = CheckoutWebViewContainer(activity)
+        container.addView(this)
+        return this
+    }
 }