(PUP-2568) Remove unnecessary calls

There are several calls that can be removed from various paths. Downcase
isn't needed for the PResourceType's type_name, since that is already
downcased (removes a string copy). The other change is that the NAME pattern can handle leading
:: and so that only needs to be stripped off if it is a valid name
(removes a string copy).