[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.
Removed protocol options from url at curl requests
[AML] Be rid of damned cpu type magic numbers
[AML] Don't assume M1 for unknown CPUs
[AML] Apply MX cpu scale hack only to MX
[AML] Use M1 digital_raw val only on M1
[AML] Ensure cputype range checks are valid.
[AML] Cosmetics

1) Sort device checks
2) Rename *cputype symbols to *device_type for consistency