Tooltip: Clear the tracking interval on close. Fixes #8920 -Tooltip potential setinterval endless loop.

scottgonzalez · scottgonzalez · commit 9cbd4b422213 · 2012-12-18T08:38:36.000-05:00
diff --git a/ui/jquery.ui.tooltip.js b/ui/jquery.ui.tooltip.js
@@ -270,7 +270,7 @@ $.widget( "ui.tooltip", {
 		// as the tooltip is visible, position the tooltip using the most recent
 		// event.
 		if ( this.options.show && this.options.show.delay ) {
-			delayedShow = setInterval(function() {
+			delayedShow = this.delayedShow = setInterval(function() {
 				if ( tooltip.is( ":visible" ) ) {
 					position( positionOption.of );
 					clearInterval( delayedShow );
@@ -312,6 +312,9 @@ $.widget( "ui.tooltip", {
 			return;
 		}
 
+		// Clear the interval for delayed tracking tooltips
+		clearInterval( this.delayedShow );
+
 		// only set title if we had one before (see comment in _open())
 		if ( target.data( "ui-tooltip-title" ) ) {
 			target.attr( "title", target.data( "ui-tooltip-title" ) );
