chore(jqLite): remove the ready handlers instead of setting a flag

mgol · mgol · commit beab3baec372 · 2016-10-12T19:31:55.000+02:00
This change aligns jqLite with the jQuery implementation. Closes angular#15237
diff --git a/src/jqLite.js b/src/jqLite.js
@@ -522,11 +522,9 @@ function jqLiteDocumentLoaded(action, win) {
 }
 
 function jqLiteReady(fn) {
-  var fired = false;
-
   function trigger() {
-    if (fired) return;
-    fired = true;
+    window.document.removeEventListener('DOMContentLoaded', trigger);
+    window.removeEventListener('load', trigger);
     fn();
   }
 

Original file line number	Diff line number	Diff line change
`@@ -522,11 +522,9 @@ function jqLiteDocumentLoaded(action, win) {`
`522`	`522`	`}`
`523`	`523`
`524`	`524`	`function jqLiteReady(fn) {`
`525`		`- var fired = false;`
`526`		`-`
`527`	`525`	`function trigger() {`
`528`		`- if (fired) return;`
`529`		`- fired = true;`
	`526`	`+ window.document.removeEventListener('DOMContentLoaded', trigger);`
	`527`	`+ window.removeEventListener('load', trigger);`
`530`	`528`	`fn();`
`531`	`529`	`}`
`532`	`530`