Ошибка throw ('msg') против новой ошибки throw ('msg')

var err1 = Error('message');
var err2 = new Error('message');

Какая разница? Глядя на них в хромированной консоли, они выглядят одинаково. Одинаковые свойства объекта и одинаковые __proto__цепочки. Похоже, Errorдействует как фабрика.

Какой из них правильный и почему?

Оба в порядке; это явно указано в спецификации :

... Таким образом, вызов функции Error(…)эквивалентен выражению создания объекта new Error(…)с теми же аргументами.

Errorэто действует как фабрика, как и некоторые другие родные конструкторов: Array, Objectи т.д. все проверки что - то подобное if (!(this instanceof Array)) { return new Array(arguments); }. (Но учтите, что String(x)и new String(x)очень разные, и аналогично для Numberи Boolean.)

Тем не менее, в случае ошибки даже не требуется бросать Errorобъект ... throw 'Bad things happened';тоже будет работать.
Вы даже можете бросить литерал объекта для отладки:

throw {message:"You've been a naughty boy",
       context: this,
       args: arguments,
       more:'More custom info here'};