hide protected fields [76109329]

Fixed

Internal Cleanup

androidx-bug-bounty

Status Update

No update yet.

Description

yb...@google.com

created issue #1

Mar 21, 2018 09:16PM

ROomDatabase has protected mCallbacks field which is leaking in the API docs, we should @Hide it.

it is not intended for public use.

https://developer.android.com/reference/android/arch/persistence/room/RoomDatabase.html#mCallbacks

Comments

yb...@google.com <yb...@google.com> #2Mar 25, 2018 08:11PM

Here is a leak example for a real-life use case. Again, the MediaSession was released, but the leak sticks around:

* GC ROOT android.support.v4.media.session.MediaSessionCompat$MediaSessionImplApi21$ExtraSession.this$0
* references android.support.v4.media.session.MediaSessionCompat$MediaSessionImplApi21.mSessionObj
* references android.media.session.MediaSession.mController
* references android.media.session.MediaController.mContext
* leaks com.bubenheimer.rucksack.d.q instance

yb...@google.com <yb...@google.com> Jun 15, 2018 05:28PM

Reassigned to da...@google.com.

fl...@google.com <fl...@google.com> Jul 19, 2019 08:42AM

Reassigned to fl...@google.com.

fl...@google.com <fl...@google.com> Jul 19, 2019 02:01PM

Accepted by fl...@google.com.

yb...@google.com <yb...@google.com> #3Jul 30, 2020 09:01PM

Thank you for your feedback. We assure you that we are doing our best to address the issue reported, however our product team has shifted work priority that doesn't include this issue. For now, we will be closing the issue as won't fix obsolete. If this issue currently still exists, we request that you log a new issue along with latest bug report here

https://goo.gl/TbMiIO.

[Deleted User] <[Deleted User]> #4Aug 9, 2020 10:54PM

deleted

ap...@google.com <ap...@google.com> #5Aug 24, 2020 06:09PM

Marked as fixed.

Can you please confirm whether this is still reproducible on API 33?

And if you happen to have access, also on the Developer Preview of API 34?