PagedListAdapter and similar classes can't coordinate list updates easily [73781068]

Verified

Bug

Status Update

No update yet.

Description

cc...@google.com

created issue #1

Feb 22, 2018 09:43PM

Jake suggested something like a lambda to listen for when the list is updated, e.g. when updating a list and its title simultaneously.

Jake, is this something like what you had in mind:

ViewModel:
public LiveData<Pair<String, PagedList<Item>>> list;
Activity:
viewModel.list.observe(this, it -> submitList(it.second, () -> setTitle(it.first)));

Comments

ja...@google.com <ja...@google.com> #2Feb 22, 2018 10:12PM

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

cc...@google.com <cc...@google.com> #3Feb 23, 2018 06:12PM

Reassigned to cc...@google.com.

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.

cc...@google.com <cc...@google.com> #4Apr 19, 2018 06:07PM

deleted

cc...@google.com <cc...@google.com> #5Aug 14, 2018 04:55PM

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?

cc...@google.com <cc...@google.com> #6Oct 15, 2018 09:57PM

Verified by cc...@google.com.

deleted

Issue 73781068