Merge pull request #315 from canjs/remove-document-element

phillipskevin · web-flow · commit f96f45a9e86a · 2018-11-19T15:45:11.000-06:00
fixing teardown when documentElement is removed
diff --git a/can-component.js b/can-component.js
@@ -490,7 +490,7 @@ var Component = Construct.extend(
 				var removalDisposal = domMutate.onNodeRemoval(el, function () {
 					var doc = el.ownerDocument;
 					var rootNode = doc.contains ? doc : doc.documentElement;
-					if (!rootNode.contains(el)) {
+					if (!rootNode || !rootNode.contains(el)) {
 						removalDisposal();
 						callTeardownFunctions();
 					}

Original file line number	Diff line number	Diff line change
`@@ -490,7 +490,7 @@ var Component = Construct.extend(`
`490`	`490`	`var removalDisposal = domMutate.onNodeRemoval(el, function () {`
`491`	`491`	`var doc = el.ownerDocument;`
`492`	`492`	`var rootNode = doc.contains ? doc : doc.documentElement;`
`493`		`- if (!rootNode.contains(el)) {`
	`493`	`+ if (!rootNode \|\| !rootNode.contains(el)) {`
`494`	`494`	`removalDisposal();`
`495`	`495`	`callTeardownFunctions();`
`496`	`496`	`}`