MediaPlayer playlist should not recalculate file metadata in Draw()

[KapiX]

PlaylistItem tries to cache metadata using attributes, but that doesn't work on read-only volumes. This leads playlist to reopen the file each time it wants to Draw an item, which sometimes blocks playlist window for a few seconds.

Item::Duration() - ​https://git.haiku-os.org/haiku/tree/src/apps/mediaplayer/playlist/PlaylistItem.cpp#n145

Item::Draw() - ​https://git.haiku-os.org/haiku/tree/src/apps/mediaplayer/playlist/PlaylistListView.cpp#n137

[leavengood]

Well it should at least cache the duration in memory, that seems like a bad idea to hit the attribute on each draw, let alone reloading the file...

[KapiX]

​https://review.haiku-os.org/c/haiku/+/4597

[pulkomandy]

Patch merged in hrev55529. As mentionned in the commit message, probably the disk access should be moved out of the display thread. Should we keep this ticket open for that?

[KapiX]

Yes, the issue described here is still there (first draw), it just doesn't do it all the time anymore.