Add visit_attribute to Visitor, use it for unused_attribute

The lint was missing a *lot* of cases previously.
(That is, this method is now mostly incorrect. Maybe it should just be `attribute.get_str()` or something?)