[MT-1531] - TagManagement: Fix terminate webView on Tealium shutdown

mobile/src/main/java/com/tealium/mobile/TealiumHelper.kt

@@ -155,6 +155,10 @@ object TealiumHelper : ActivityDataCollector {
         )
     }
 
+    fun shutdown() {
+        Tealium.destroy(BuildConfig.TEALIUM_INSTANCE)
+    }
+
     fun retrieveDatalayer(): Map<String, Any>? {
         return Tealium[BuildConfig.TEALIUM_INSTANCE]?.gatherTrackData()
     }

tagmanagementdispatcher/build.gradle

@@ -3,7 +3,7 @@ apply plugin: 'kotlin-android'
 apply plugin: 'maven-publish'
 apply from: '../jacoco.gradle'
 
-version = '1.2.2'
+version = '1.2.3'
 
 android {
     compileSdkVersion 33

tagmanagementdispatcher/src/main/java/com/tealium/tagmanagementdispatcher/WebViewLoader.kt

@@ -17,6 +17,7 @@ import com.tealium.remotecommands.RemoteCommandRequest
 import kotlinx.coroutines.*
 import kotlinx.coroutines.Dispatchers
 import org.json.JSONObject
+import java.lang.ref.WeakReference
 import java.util.*
 import java.util.concurrent.atomic.AtomicBoolean
 import java.util.concurrent.atomic.AtomicReference
@@ -28,7 +29,8 @@ class WebViewLoader(
     private val connectivityRetriever: Connectivity = ConnectivityRetriever.getInstance(context.config.application),
     private val webViewProvider: () -> WebView = { WebView(context.config.application) }
 ) : LibrarySettingsUpdatedListener,
-    SessionStartedListener {
+    SessionStartedListener,
+    InstanceShutdownListener {
 
     val webViewStatus = AtomicReference<PageStatus>(PageStatus.INIT)
     var lastUrlLoadTimestamp = 0L
@@ -410,6 +412,19 @@ class WebViewLoader(
         registerNewSessionIfNeeded(sessionId)
     }
 
+    override fun onInstanceShutdown(name: String, instance: WeakReference<Tealium>) {
+        mainScope.launch {
+            try {
+                webView.destroy()
+            } catch (t: Throwable) {
+                Logger.dev(
+                    BuildConfig.TAG,
+                    "Error destroying WebView on shutdown: ${t.localizedMessage}"
+                )
+            }
+        }
+    }
+
     private fun registerNewSessionIfNeeded(sessionId: Long) {
         if (sessionId == INVALID_SESSION_ID) {
             return

tagmanagementdispatcher/src/test/java/com/tealium/tagmanagementdispatcher/WebViewClientTest.kt

@@ -4,6 +4,7 @@ import android.app.Application
 import android.net.Uri
 import android.webkit.*
 import com.tealium.core.Logger
+import com.tealium.core.Tealium
 import com.tealium.core.TealiumConfig
 import com.tealium.core.TealiumContext
 import com.tealium.core.messaging.AfterDispatchSendCallbacks
@@ -21,6 +22,7 @@ import org.junit.Assert.*
 import org.junit.Before
 import org.junit.Test
 import java.io.File
+import java.lang.ref.WeakReference
 import java.util.*
 
 class WebViewClientTest {
@@ -290,4 +292,37 @@ class WebViewClientTest {
             mockWebView.destroy()
         }
     }
+
+    @Test
+    fun webViewClient_DestroyWebViewOnInstanceShutdown() = runBlocking {
+        webViewLoader = WebViewLoader(mockTealiumContext, "testUrl", mockDispatchSendCallbacks, mockConnectivity, mockWebViewProvider)
+        delay(1000)
+
+        webViewLoader.onInstanceShutdown("testName", WeakReference(Tealium::class.java.cast(null)))
+        delay(1000)
+
+        verify(exactly = 1, timeout = 1000) {
+            mockWebView.destroy()
+        }
+    }
+
+    @Test
+    fun webViewClient_DestroyWebViewHandlesExceptionOnInstanceShutdown() = runBlocking {
+        every { mockWebView.destroy() } throws RuntimeException("Test WebView destruction exception")
+
+        webViewLoader = WebViewLoader(mockTealiumContext, "testUrl", mockDispatchSendCallbacks, mockConnectivity, mockWebViewProvider)
+        delay(1000)
+
+        webViewLoader.onInstanceShutdown("testName", WeakReference(Tealium::class.java.cast(null)))
+        delay(1000)
+
+        verify(exactly = 1) {
+            mockWebView.destroy()
+        }
+        verify {
+            Logger.dev(any(), match {
+                it.startsWith("Error destroying WebView on shutdown:")
+            })
+        }
+    }
 }