Fix TypeError in _trimBreadcrumbs when data undefined (#929)

Author: benvinegar

src/raven.js

@@ -1356,7 +1356,7 @@ Raven.prototype = {
 
         for (var i = 0; i < breadcrumbs.values.length; ++i) {
             crumb = breadcrumbs.values[i];
-            if (!crumb.hasOwnProperty('data'))
+            if (!crumb.hasOwnProperty('data') || !isObject(crumb.data))
                 continue;
 
             data = crumb.data;

test/raven.test.js

@@ -151,6 +151,20 @@ describe('globals', function() {
                     }
                 }]
             });
+
+           breadcrumbs = {
+               values: [{
+                   data: undefined
+               }]
+           };
+
+           // see: https://github.com/getsentry/raven-js/issues/925
+           Raven._trimBreadcrumbs(breadcrumbs);
+           assert.deepEqual(breadcrumbs, {
+              values: [{
+                   data: undefined
+               }]
+            });
         });
     });