Error object needs to be added to class2type map

[tomgrohl]

If you were to do anything like the following

try
{

   alert(a);
   
}
catch(e)
{
   alert( Object.prototype.toString.call(e) );
}

It would alert [object Error], but this is not in the jQuery class2type map;

I think Error needs adding, unless it is not in there for a reason.

New code is below:

jQuery.each("Boolean Number String Function Array Date RegExp Error Object".split(" "), function(i, name) {

class2type[ "[object " + name + "]" ] = name.toLowerCase();

});

[jitter]

This has already been proposed but it can't hurt to have a bug report to remember it. For further reference ​the original request for enhancement

[danheberden]

​https://github.com/jquery/jquery/pull/305

[kitgoncharov]

While this is a useful, relatively minor addition, it's also implemented inconsistently in some WebKit-based browsers. Specifically, the Class? name of native Error objects is "Object," not "Error," in Safari 2.x. Since jQuery doesn't officially support Safari 2, this isn't a significant concern; however, the library does support some mobile browsers that are based on outdated WebKit versions. As such, the fact that we can't ensure complete cross-browser consistency for this change suggests that this ticket should be closed as "wontfix."

[john]

See my comments in the pull request, closing for now. Please reopen if we can prove that it's consistently working right.