I think we should make this take a DOMString and do the null/empty => Null check here, rather than in all the callers.