xbmc/xbmc — pull request #1220 by jmarshallnz (2012-07-26 03:16:43+00:00)
Title: Cleanup resume

This cleans up the resume handling to use m_resumePoint in the videoinfotag if available. It means we don't necessarily have to hit the database if we already have the information.

Further, it means that directory classes (eg plugins + upnp) can supply the details for resuming directly for those items that aren't in the local database (indeed, it would bypass the local database). The intention is that a handler (eg using JSON-RPC) for these directory classes would then update the source of that information (eg update a upnp server) when an item from that directory class is stopped part-way.

There is a question of whether resume points should be stored for such items locally at all - currently there is no way to differentiate those, but now that the resume point structure is being used, this could be changed.

Plugins can also override any resume information during setResolvedUrl() on playback should they wish (request from plugin authors).

Comments welcome.