bug#19903: 24.4; wrong-type-argument symbolp "bold" during enriched-encode

[Eli Zaretskii]

> From: Ivan Shmakov <ivan@siamics.net>
> Date: Sat, 21 Feb 2015 11:12:44 +0000
> 
>       Now, given that there’s a number of “internal” functions (such
>       as ‘internal-lisp-face-p’, for instance) which accept string
>       face names just fine, I wonder if it makes sense to just change
>       ‘internal-get-lisp-face-attribute’ accordingly?  An untested
>       patch is hereby MIMEd.

I don't think internal functions should cater to UI issues, unless
they are themselves interactive.

If we keep this confined to interactive functions, how many such
functions in facemenu.el will have to be changed?  If not too many,
I'm inclined to keep this there.