Add aria-label and alt support

Original PR was eaten by git :\

This adds a `role` to core-icon and allows the developer to apply either `aria-label` or `alt` attributes to support screen readers. As a last ditch fallback it will try to include the icon name if one was provided.

cc @alice
Ok, '' is no longer the default value. Can we make the default null instead of undefined?