I think it's only bad because it would break with future changes to typeof. Of course I may be completely wrong on that.

This works just as well and is more future proof though:

    function isNull(x) { return x === null }

At which point you don't really need a function at all. Perhaps the version using typeof is from a time when there was no === operator. Was that pre-ES3? I think so but am not certain.