Merge pull request #176 from canjs/swallow

Prevents errors thrown outside of the run promise from being swallowed

lib/zone.js

@@ -41,9 +41,11 @@ Task.prototype.run = function(ctx, args){
 			zone.execHookR("afterTask", this);
 	} catch(err) {
 		Zone.current = previousZone;
-		if(!this.nestedTask)
+		if(!this.nestedTask) {
 			zone.execHookR("afterTask", this);
-		if(this.catchErrors !== false) {
+		}
+
+		if(this.catchErrors !== false && !this.zone.isResolved) {
 			zone.errors.push(err);
 		} else {
 			throw err;

test/test.js

@@ -905,6 +905,27 @@ if(isBrowser && !isWorker) {
 			})
 			.then(done, done);
 		});
+
+		it("Event handlers ran after the Zone completes throw their error", function(done){
+			var el = document.createElement("div");
+
+			new Zone().run(function(){
+				el.addEventListener("some-event", function(){
+					throw new Error("Hello world!");
+				});
+			})
+			.then(function(){
+				var errorNotSwallowed = false;
+				window.onerror = function(){
+					window.onerror = null;
+					errorNotSwallowed = true;
+					return true;
+				};
+				el.dispatchEvent(new Event("some-event"));
+				assert.ok(errorNotSwallowed, "Threw an error, not swallowed");
+			})
+			.then(done, done);
+		});
 	});
 
 	describe("when third-party library (i.e. affirm) also wraps events (i.e. addEventListener) in strict-mode", function(){