Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Commit c3d5cc5

Browse files
jbedardNarretz
authored andcommitted
refactor($compile): remove workaround for chrome bug fixed in v50
Closes #16554
1 parent 6f85b3b commit c3d5cc5

File tree

1 file changed

+0
-11
lines changed

1 file changed

+0
-11
lines changed

src/ng/compile.js

-11
Original file line numberDiff line numberDiff line change
@@ -2549,17 +2549,6 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
25492549
compileNode = $compileNode[0];
25502550
replaceWith(jqCollection, sliceArgs($template), compileNode);
25512551

2552-
// Support: Chrome < 50
2553-
// https://github.com/angular/angular.js/issues/14041
2554-
2555-
// In the versions of V8 prior to Chrome 50, the document fragment that is created
2556-
// in the `replaceWith` function is improperly garbage collected despite still
2557-
// being referenced by the `parentNode` property of all of the child nodes. By adding
2558-
// a reference to the fragment via a different property, we can avoid that incorrect
2559-
// behavior.
2560-
// TODO: remove this line after Chrome 50 has been released
2561-
$template[0].$$parentNode = $template[0].parentNode;
2562-
25632552
childTranscludeFn = compilationGenerator(mightHaveMultipleTransclusionError, $template, transcludeFn, terminalPriority,
25642553
replaceDirective && replaceDirective.name, {
25652554
// Don't pass in:

0 commit comments

Comments
 (0)