Strict error check using isError, fix regexes

Author: benvinegar

src/raven.js

@@ -3,6 +3,11 @@
 
 var TraceKit = require('../vendor/TraceKit/tracekit');
 var RavenConfigError = require('./configError');
+var utils = require('./utils');
+
+var isError = utils.isError,
+    isObject = utils.isObject;
+
 var stringify = require('json-stringify-safe');
 
 var wrapConsoleMethod = require('./console').wrapMethod;
@@ -1655,25 +1660,12 @@ function isString(what) {
     return objectPrototype.toString.call(what) === '[object String]';
 }
 
-function isObject(what) {
-    return typeof what === 'object' && what !== null;
-}
 
 function isEmptyObject(what) {
     for (var _ in what) return false;  // eslint-disable-line guard-for-in, no-unused-vars
     return true;
 }
 
-// Sorta yanked from https://github.com/joyent/node/blob/aa3b4b4/lib/util.js#L560
-// with some tiny modifications
-function isError(what) {
-    var toString = objectPrototype.toString.call(what);
-    return isObject(what) &&
-        toString === '[object Error]' ||
-        toString === '[object Exception]' || // Firefox NS_ERROR_FAILURE Exceptions
-        what instanceof Error;
-}
-
 function each(obj, callback) {
     var i, j;
 

src/utils.js

@@ -0,0 +1,20 @@
+'use strict';
+
+function isObject(what) {
+    return typeof what === 'object' && what !== null;
+}
+
+// Sorta yanked from https://github.com/joyent/node/blob/aa3b4b4/lib/util.js#L560
+// with some tiny modifications
+function isError(what) {
+    var toString = {}.toString.call(what);
+    return isObject(what) &&
+        toString === '[object Error]' ||
+        toString === '[object Exception]' || // Firefox NS_ERROR_FAILURE Exceptions
+        what instanceof Error;
+}
+
+module.exports = {
+    isObject: isObject,
+    isError: isError
+};

test/integration/test.js

@@ -275,7 +275,39 @@ describe('integration', function () {
                     var ravenData = iframe.contentWindow.ravenData[0];
                     assert.match(ravenData.exception.values[0].value, /stringError$/);
                     assert.equal(ravenData.exception.values[0].stacktrace.frames.length, 1);  // always 1 because thrown strings can't provide > 1 frame
-                    assert.match(ravenData.exception.values[0].stacktrace.frames[0].filename, /\/test\/integration\/throw-string\.js/)
+
+                    // some browsers extract proper url, line, and column for thrown strings
+                    // but not all - falls back to frame url
+                    assert.match(ravenData.exception.values[0].stacktrace.frames[0].filename, /\/test\/integration\//);
+                    assert.match(ravenData.exception.values[0].stacktrace.frames[0]['function'], /\?|global code/);
+                }
+            );
+        });
+
+        it('should catch thrown objects', function (done) {
+            var iframe = this.iframe;
+
+            iframeExecute(iframe, done,
+                function () {
+                    // intentionally loading this error via a script file to make
+                    // sure it is 1) not caught by instrumentation 2) doesn't trigger
+                    // "Script error"
+                    var script = document.createElement('script');
+                    script.src = 'throw-object.js';
+                    script.onload = function () {
+                        done();
+                    };
+                    document.head.appendChild(script);
+                },
+                function () {
+                    var ravenData = iframe.contentWindow.ravenData[0];
+                    assert.equal(ravenData.exception.values[0].type, undefined);
+                    assert.equal(ravenData.exception.values[0].value, '[object Object]');
+                    assert.equal(ravenData.exception.values[0].stacktrace.frames.length, 1); // always 1 because thrown objects can't provide > 1 frame
+
+                    // some browsers extract proper url, line, and column for thrown objects
+                    // but not all - falls back to frame url
+                    assert.match(ravenData.exception.values[0].stacktrace.frames[0].filename, /\/test\/integration\//);
                     assert.match(ravenData.exception.values[0].stacktrace.frames[0]['function'], /\?|global code/);
                 }
             );

test/integration/throw-object.js

@@ -0,0 +1,7 @@
+function throwStringError() {
+    // never do this; just making sure Raven.js handles this case
+    // gracefully
+    throw {error: 'stuff is broken'};
+}
+
+throwStringError();

vendor/TraceKit/tracekit.js

@@ -1,5 +1,7 @@
 'use strict';
 
+var utils = require('../../src/utils');
+
 /*
  TraceKit - Cross brower stack traces
 
@@ -26,7 +28,7 @@ var _slice = [].slice;
 var UNKNOWN_FUNCTION = '?';
 
 // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error#Error_types
-var ERROR_TYPES_RE = /^(?:Uncaught (?:exception: )?)?((?:Eval|Internal|Range|Reference|Syntax|Type|URI|)Error): ?(.*)$/;
+var ERROR_TYPES_RE = /^(?:[Uu]ncaught (?:exception: )?)?(?:((?:Eval|Internal|Range|Reference|Syntax|Type|URI|)Error): )?(.*)$/;
 
 function getLocationHref() {
     if (typeof document === 'undefined' || typeof document.location === 'undefined')
@@ -153,7 +155,7 @@ TraceKit.report = (function reportModuleWrapper() {
         if (lastExceptionStack) {
             TraceKit.computeStackTrace.augmentStackTraceWithInitialElement(lastExceptionStack, url, lineNo, message);
             processLastException();
-        } else if (ex && {}.toString.call(ex) !== '[object String]') {
+        } else if (ex && utils.isError(ex)) {
             // non-string `ex` arg; attempt to extract stack trace
 
             // New chrome and blink send along a real error object