From e8035622b3baa2aeb792df938a2bbd23e02ee41b Mon Sep 17 00:00:00 2001 From: Ben Weiss Date: Mon, 20 Nov 2023 09:07:30 +0100 Subject: [PATCH] =?UTF-8?q?Enable=20the=20thread=20police=20=F0=9F=91=AE?= =?UTF-8?q?=20for=20debug=20builds?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * App will now crash if someone does funky stuff on the main thread. Change-Id: I9026c100705f2fec6963a1d888b40906186f9d28 --- .../samples/apps/nowinandroid/NiaApplication.kt | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/app/src/main/kotlin/com/google/samples/apps/nowinandroid/NiaApplication.kt b/app/src/main/kotlin/com/google/samples/apps/nowinandroid/NiaApplication.kt index 8e3ad814a..775cfe8ee 100644 --- a/app/src/main/kotlin/com/google/samples/apps/nowinandroid/NiaApplication.kt +++ b/app/src/main/kotlin/com/google/samples/apps/nowinandroid/NiaApplication.kt @@ -17,6 +17,8 @@ package com.google.samples.apps.nowinandroid import android.app.Application +import android.content.pm.ApplicationInfo +import android.os.StrictMode import coil.ImageLoader import coil.ImageLoaderFactory import com.google.samples.apps.nowinandroid.sync.initializers.Sync @@ -37,10 +39,25 @@ class NiaApplication : Application(), ImageLoaderFactory { override fun onCreate() { super.onCreate() + + // Kill NiA if there are main thread policy violations and log the offending call. + if (isDebuggable()) { + StrictMode.setThreadPolicy( + StrictMode.ThreadPolicy.Builder().detectAll().penaltyLog().penaltyDeath().build(), + ) + } + // Initialize Sync; the system responsible for keeping data in the app up to date. Sync.initialize(context = this) profileVerifierLogger() } override fun newImageLoader(): ImageLoader = imageLoader.get() + + /** + * Check if the application is debuggable. + */ + private fun isDebuggable(): Boolean { + return 0 != applicationInfo.flags and ApplicationInfo.FLAG_DEBUGGABLE + } }