fix(perf): Fix event listeners/ DOM nodes memory leak and throttle resize events

Includes: - Unbind 'resize' event handlers on $destroy (as per issue 720kb#143). Not only this reduces the overall number of listeners being bound, but also allows DOM nodes to be GC'ed. - To alleviate some CPU churn, we now throttle the resize event handler. By doing so, we enable users to resize their browsers in a much smoother fashion.

Author: Omer

lib/angular-tooltips.js

@@ -21,9 +21,12 @@
  window.console.log('Skipped!');
  }
  }
+ , resizeTimeout
  , resize = function resize() {
-
- window.requestAnimationFrame(runCallbacks);
+ window.clearTimeout(resizeTimeout);
+ resizeTimeout = window.setTimeout(function onResizeTimeout() {
+ window.requestAnimationFrame(runCallbacks);
+ }, 500);
  }
  , addCallback = function addCallback(callback) {
 
@@ -41,6 +44,12 @@
  window.addEventListener('resize', resize);
  }
  addCallback(callback);
+ },
+ 'remove': function remove() {
+ if (!callbacks.length) {
+ window.clearTimeout(resizeTimeout);
+ window.removeEventListener('resize', resize);
+ }
  }
  };
  }())
@@ -767,6 +776,7 @@
  unregisterOnTooltipSizeChange();
  unregisterOnTooltipSpeedChange();
  unregisterTipContentChangeWatcher();
+ resizeObserver.remove();
  element.off($attrs.tooltipShowTrigger + ' ' + $attrs.tooltipHideTrigger);
  });
  });