[url] use IsProtocol in place of GetProtocol
[fileitem] drop GetAsUrl() as GetURL() does the same thing
[dav] have GetStatusTag() iterate over XML elements, not nodes.

As things are, if there are no children of the passed element, we'll have a NULL dereference.
[dav] iterate over XML elements not Nodes in CDavDirectory::ParseResponse() as we assume elements inside the loops anyway
[rssdirectory] have a helper for getting the value of a child XML element rather than calling GetText() and assuming a non-NULL response
[edl] use FirstChild()->Value() rather than GetText() in parsing cut times
[pls playlist] use FirstChild()->Value() rather than GetText() without NULL checks
[advancedsettings] use FirstChild()->Value() rather than GetText() when reading DNS cache entries
[fanart] use FirstChild()->Value() rather than unchecked GetText().

NOTE: Potential change in behaviour in that if a url attribute is specified in the <fanart>
node and <thumb> children are specified without a value, it used to just take url as the
url.  This was not intended, however, as url is supposed to be the base url, not the full url.
[buttontranslator] tidy the MapRemote and MapJoystickActions functions to not use unchecked GetText()
[cosmetic] indenting
[buttontranslator] kill unnecessary ifdefs
fix: mapping for xbmc eject on device mceusb

Follows ksooo #4949 and #4979

Devices like
http://h10025.www1.hp.com/ewfrf/wc/document?cc=uk&lc=en&docname=c00844678
etc benefit form these keys.
[cosmetic] remove duplicate access modifier in CGUIMediaWindow
[pvr] refactor directory path preparation out of Update() method

This fixes an issue where we get an infinity loop cause by a fallback done
in GUIMediaWindow::Update() if GetDirectory() return false. This effects only
some of the PVR related windows, because there we override the directory path
directly within the Update() method and the strDirectory param is discarded.
Add audioencoder icons
Merge pull request #5057 from uNiversaI/icons

Add audioencoder icons
[pvr] drop call to CRenderManager::Preinit(), apparently it
breaks stuff
Merge pull request #5029 from jmarshallnz/protocol_filetype

Replace GetProtocol() usage with IsProtocol, drop CFileItem::GetAsUrl() duplicate
[videodb] no need to use GetEpisodeId() if we already have an episode id. fixes detour via infoscanner when displaying episode info