By the way, I did not realize that the docstring of 'set-face-attribute' 
says nothing about the evaluation order of its arguments.  I suggest the 
attached patch.

Eli, is this okay for the release branch?