Workaround for Kotlin/kotlinx.coroutines#3328

Author: csadilek

android-components/components/browser/storage-sync/src/main/java/mozilla/components/browser/storage/sync/PlacesStorage.kt

@@ -6,11 +6,7 @@ package mozilla.components.browser.storage.sync
 
 import android.content.Context
 import androidx.annotation.VisibleForTesting
-import kotlinx.coroutines.CoroutineScope
-import kotlinx.coroutines.Dispatchers
-import kotlinx.coroutines.asCoroutineDispatcher
-import kotlinx.coroutines.cancelChildren
-import kotlinx.coroutines.withContext
+import kotlinx.coroutines.*
 import mozilla.appservices.places.PlacesReaderConnection
 import mozilla.appservices.places.PlacesWriterConnection
 import mozilla.appservices.places.uniffi.PlacesApiException
@@ -36,10 +32,30 @@ abstract class PlacesStorage(
             Executors.newSingleThreadExecutor(
                 NamedThreadFactory("PlacesStorageWriteScope"),
             ).asCoroutineDispatcher(),
-        )
+        ) +
+        CoroutineExceptionHandler { _, throwable ->
+            // Workaround for https://github.com/Kotlin/kotlinx.coroutines/issues/3328 to prevent
+            // adding DiagnosticCoroutineContextException as suppressed exception which is not
+            // serializable in 1.6.4. We are otherwise not able to serialize the exception and
+            // pass it to our crash reporter. Without a handler on the context, the global handler
+            // will otherwise add:
+            // runCatching { exception.addSuppressed(DiagnosticCoroutineContextException(context)) }
+            val currentThread = Thread.currentThread()
+            currentThread.uncaughtExceptionHandler.uncaughtException(currentThread, throwable)
+        }
         @VisibleForTesting internal set
 
-    internal var readScope = CoroutineScope(Dispatchers.IO)
+    internal var readScope = CoroutineScope(Dispatchers.IO) +
+        CoroutineExceptionHandler { _, throwable ->
+            // Workaround for https://github.com/Kotlin/kotlinx.coroutines/issues/3328 to prevent
+            // adding DiagnosticCoroutineContextException as suppressed exception which is not
+            // serializable in 1.6.4. We are otherwise not able to serialize the exception and
+            // pass it to our crash reporter. Without a handler on the context, the global handler
+            // will otherwise add:
+            // runCatching { exception.addSuppressed(DiagnosticCoroutineContextException(context)) }
+            val currentThread = Thread.currentThread()
+            currentThread.uncaughtExceptionHandler.uncaughtException(currentThread, throwable)
+        }
         @VisibleForTesting internal set
     private val storageDir by lazy { context.filesDir }