More string -> names.Tag conversion

Shouldn't this accept a *names.UnitTag?

The new behaviour isn't comparable.  A null tag was an error before, now it isn't. We don't check for nil, and it is possible to pass something that isn't a Unit.