[PATCHSET] Add missing part of BMediaRoster and other improvements in media_kit

[Barrett]

This patch improve various aspects of the media_kit and media_server, this is a little summary :

• Add BMediaRoster::IsRunning, SyncToNode (#11856) and GetFormatsFor
• General improvements in style and error checking
• Fix #5621
• Add a new service for detecting when media services are up/down and use it to remove code duplication, make BMediaRoster survive to media_server reboot by reestabilishing the connection, make media_server to watch when the media_addon_server quit and call the cleanup function, which was leaved here uncomplete and unused until i get on it.

[korli]

About 0002:

• port < B_OK is usually used to check for an error on create_port() call.
• status and reply.status should be checked for B_OK equality.
• "else status = B_OK;" can then be left out.
• shouldn't BMediaNode::TimerExpired() do something with the parameters notifyPoint and error?
• wait_port isn't a coding style valid c++ name. waitPort would be better.

[Barrett]

I'm preparing an update to the whole patchset. About BMediaNode::TimerExpired() the error argument is set to B_OK as default and i don't know what i could do with notifyPoint, do you have any suggestion? I suppose they were here to support non-BMediaEventLooper implementations.

[Barrett]

This new patch just fix a line going over 80 characters.

[Barrett]

Fix things suggested by korli

[Barrett]

Sorry for double upload of 0004 it's not really comfortable to update patches here. I've fixed what you suggested and made other updates on the other patches for minor details such as comments and code style.

[Barrett]

Don't rely on those patches, the updated ones are there (and follow-ups) :

​https://github.com/Barrett17/haiku/commits/mediaroster ​https://github.com/Barrett17/haiku/commits/btimesource_cleanup

[Barrett]

All remaining code applied in hrev49445.