Memory leak according to LeakCanary

Please complete the following information:

• Library Version: 1.2.0
• Affected Device(s): Xiaomi Poco F2 Pro

Describe the Bug: Whenever I add a com.skydoves.powerspinner.PowerSpinnerView to my fragment I get a notification from LeakCanary about a memory leak caused by the PowerSpinnerView. I don’t do anything else with it with code, it’s just this.

<com.skydoves.powerspinner.PowerSpinnerView
            android:id="@+id/genreSpinner"
            android:layout_width="80dp"
            android:layout_height="wrap_content"
            android:layout_margin="6dp"
            android:hint="Genre"
            app:layout_constraintStart_toStartOf="parent"
            app:layout_constraintTop_toTopOf="parent"/>

I tried setting the lifecycleowner

genreSpinner.lifecycleOwner = this <- (Fragment)

With or without it, I still get the memory leak notification.

When I add the PowerSpinnerView to my main activity layout file (my app uses 1 activity with multiple fragments) I don’t have this issue.

Expected Behavior: No memory leak 😃

Below you can see the notification from LeakCanary:

┬───
│ GC Root: Input or output parameters in native code
│
├─ android.os.FileObserver$ObserverThread instance
│    Leaking: NO (PathClassLoader↓ is not leaking)
│    Thread name: 'FileObserver'
│    ↓ Thread.contextClassLoader
├─ dalvik.system.PathClassLoader instance
│    Leaking: NO (InternalLeakCanary↓ is not leaking and A ClassLoader is never
│    leaking)
│    ↓ ClassLoader.runtimeInternalObjects
├─ java.lang.Object[] array
│    Leaking: NO (InternalLeakCanary↓ is not leaking)
│    ↓ Object[].[378]
├─ leakcanary.internal.InternalLeakCanary class
│    Leaking: NO (MainActivity↓ is not leaking and a class is never leaking)
│    ↓ static InternalLeakCanary.resumedActivity
├─ <my_package>.MainActivity instance
│    Leaking: NO (Activity#mDestroyed is false)
│    mApplication instance of <my_package>.MainApplication
│    mBase instance of androidx.appcompat.view.ContextThemeWrapper
│    ↓ ComponentActivity.mLifecycleRegistry
│                        ~~~~~~
├─ androidx.lifecycle.LifecycleRegistry instance
│    Leaking: UNKNOWN
│    Retaining 1.1 kB in 44 objects
│    ↓ LifecycleRegistry.mObserverMap
│                        ~~~~
├─ androidx.arch.core.internal.FastSafeIterableMap instance
│    Leaking: UNKNOWN
│    Retaining 932 B in 39 objects
│    ↓ SafeIterableMap.mEnd
│                      ~~
├─ androidx.arch.core.internal.SafeIterableMap$Entry instance
│    Leaking: UNKNOWN
│    Retaining 56 B in 3 objects
│    ↓ SafeIterableMap$Entry.mKey
│                            ~~
├─ com.skydoves.powerspinner.PowerSpinnerView instance
│    Leaking: UNKNOWN
│    Retaining 1.9 MB in 9635 objects
│    View not part of a window view hierarchy
│    View.mAttachInfo is null (view detached)
│    View.mID = R.id.null
│    View.mWindowAttachCount = 1
│    mContext instance of <my_package>.MainActivity with mDestroyed =
│    false
│    ↓ View.mParent
│           ~~~
├─ androidx.constraintlayout.widget.ConstraintLayout instance
│    Leaking: UNKNOWN
│    Retaining 1.5 MB in 5099 objects
│    View not part of a window view hierarchy
│    View.mAttachInfo is null (view detached)
│    View.mID = R.id.null
│    View.mWindowAttachCount = 1
│    mContext instance of <my_package>.MainActivity with mDestroyed =
│    false
│    ↓ View.mParent
│           ~~~
├─ androidx.constraintlayout.widget.ConstraintLayout instance
│    Leaking: UNKNOWN
│    Retaining 1.4 kB in 22 objects
│    View not part of a window view hierarchy
│    View.mAttachInfo is null (view detached)
│    View.mID = R.id.null
│    View.mWindowAttachCount = 1
│    mContext instance of <my_package>.MainActivity with mDestroyed =
│    false
│    ↓ View.mParent
│           ~~~
╰→ androidx.drawerlayout.widget.DrawerLayout instance
​     Leaking: YES (ObjectWatcher was watching this because <my_package>.
​     fragments.channelsFragment.ChannelsFragment received
​     Fragment#onDestroyView() callback (references to its views should be
​     cleared to prevent leaks))
​     Retaining 3.2 kB in 91 objects
​     key = b686def5-a840-41ef-931a-d2b1a1f03023
​     watchDurationMillis = 15996
​     retainedDurationMillis = 10996
​     View not part of a window view hierarchy
​     View.mAttachInfo is null (view detached)
​     View.mID = R.id.null
​     View.mWindowAttachCount = 1
​     mContext instance of <my_package>.MainActivity with mDestroyed =
​     false

METADATA

Build.VERSION.SDK_INT: 30
Build.MANUFACTURER: Xiaomi
LeakCanary version: 2.7
App process name: <my_package>
Stats: LruCache[maxSize=3000,hits=3371,misses=114403,hitRate=2%]
RandomAccess[bytes=5538838,reads=114403,travel=56894977315,range=28893688,size=3
6090270]
Heap dump reason: user request
Analysis duration: 5416 ms

[skydoves]

Hi, you must use viewLifecycleOwner in your Fragment instead of this.

[renezuidhof]

Fixed