feat: leakcanary 调整

Author: OnionTiger

app/build.gradle.kts

@@ -163,7 +163,7 @@ dependencies {
     implementation(libs.coil.compose)
     implementation(libs.color.chooser)
     // performance
-    implementation(libs.leakcanary.android)
+    //implementation(libs.leakcanary.android)
     // local db
     implementation(libs.mmkv)
     //引入卡顿监控实现依赖库

app/src/main/java/com/omega/sun/ui/controller/base/BaseController.kt

@@ -7,9 +7,6 @@ import com.bluelinelabs.conductor.Controller
 
 abstract class BaseController(args: Bundle? = null) : Controller(args) {
 
-    init {
-        watchForLeaks()
-    }
 
     val fragmentActivity: FragmentActivity?
         get() {

app/src/main/java/com/omega/sun/ui/controller/base/BaseLifecycleController.kt

@@ -85,7 +85,6 @@ abstract class BaseLifecycleController(args: Bundle? = null) : BaseController(ar
     }
 
     init {
-        watchForLeaks()
         addLifecycleListener(object : LifecycleListener() {
             override fun preCreateView(controller: Controller) {
                 super.preCreateView(controller)

app/src/main/java/com/omega/sun/ui/controller/base/ControllerLeakWatcher.kt

@@ -1,44 +1,44 @@
-package com.omega.sun.ui.controller.base
-
-import android.view.View
-import com.bluelinelabs.conductor.Controller
-import com.bluelinelabs.conductor.ControllerChangeHandler
-import com.bluelinelabs.conductor.ControllerChangeType
-import com.omega.sun.BuildConfig
-import leakcanary.AppWatcher
-
-private class RefWatchingControllerLifecycleListener : Controller.LifecycleListener() {
-
-    private var hasExited = false
-
-    override fun postDestroy(controller: Controller) {
-        if (hasExited) {
-            controller.expectWeaklyReachable()
-        }
-    }
-
-    override fun preDestroyView(controller: Controller, view: View) {
-        AppWatcher.objectWatcher.expectWeaklyReachable(view, "A destroyed controller view should have only weak references.")
-    }
-
-    override fun onChangeEnd(
-        controller: Controller,
-        changeHandler: ControllerChangeHandler,
-        changeType: ControllerChangeType
-    ) {
-        hasExited = !changeType.isEnter
-        if (controller.isDestroyed) {
-            controller.expectWeaklyReachable()
-        }
-    }
-
-    private fun Controller.expectWeaklyReachable() {
-        AppWatcher.objectWatcher.expectWeaklyReachable(this, "A destroyed controller should have only weak references.")
-    }
-}
-
-fun Controller.watchForLeaks() {
-    if(BuildConfig.DEBUG){
-        addLifecycleListener(RefWatchingControllerLifecycleListener())
-    }
-}
+//package com.omega.sun.ui.controller.base
+//
+//import android.view.View
+//import com.bluelinelabs.conductor.Controller
+//import com.bluelinelabs.conductor.ControllerChangeHandler
+//import com.bluelinelabs.conductor.ControllerChangeType
+//import com.omega.sun.BuildConfig
+//import leakcanary.AppWatcher
+//
+//private class RefWatchingControllerLifecycleListener : Controller.LifecycleListener() {
+//
+//    private var hasExited = false
+//
+//    override fun postDestroy(controller: Controller) {
+//        if (hasExited) {
+//            controller.expectWeaklyReachable()
+//        }
+//    }
+//
+//    override fun preDestroyView(controller: Controller, view: View) {
+//        AppWatcher.objectWatcher.expectWeaklyReachable(view, "A destroyed controller view should have only weak references.")
+//    }
+//
+//    override fun onChangeEnd(
+//        controller: Controller,
+//        changeHandler: ControllerChangeHandler,
+//        changeType: ControllerChangeType
+//    ) {
+//        hasExited = !changeType.isEnter
+//        if (controller.isDestroyed) {
+//            controller.expectWeaklyReachable()
+//        }
+//    }
+//
+//    private fun Controller.expectWeaklyReachable() {
+//        AppWatcher.objectWatcher.expectWeaklyReachable(this, "A destroyed controller should have only weak references.")
+//    }
+//}
+//
+//fun Controller.watchForLeaks() {
+//    if(BuildConfig.DEBUG){
+//        addLifecycleListener(RefWatchingControllerLifecycleListener())
+//    }
+//}