fix tests

Author: andrewseguin

src/lib/core/overlay/scroll/scroll-dispatcher.ts

@@ -8,7 +8,7 @@ import 'rxjs/add/operator/auditTime';
 
 
 /** Time in ms to throttle the scrolling events by default. */
-export const DEFAULT_AUDIT_TIME = 20;
+export const DEFAULT_SCROLL_TIME = 20;
 
 /**
  * Service contained all registered Scrollable references and emits an event when any one of the
@@ -57,7 +57,7 @@ export class ScrollDispatcher {
    * references (or window, document, or body) fire a scrolled event. Can provide a time in ms
    * to override the default "throttle" time.
    */
-  scrolled(auditTimeInMs: number = DEFAULT_AUDIT_TIME): Observable<void> {
+  scrolled(auditTimeInMs: number = DEFAULT_SCROLL_TIME): Observable<void> {
     // In the case of a 0ms delay, return the observable without auditTime since it does add
     // a perceptible delay in processing overhead.
     if (auditTimeInMs == 0) {

src/lib/tooltip/tooltip.spec.ts

@@ -313,7 +313,7 @@ describe('MdTooltip', () => {
           tooltipDirective = buttonDebugElement.injector.get(MdTooltip);
         });
 
-      it('should hide tooltip if clipped after changing positions', fakeAsync(() => {
+      fit('should hide tooltip if clipped after changing positions', fakeAsync(() => {
         expect(tooltipDirective._tooltipInstance).toBeUndefined();
 
         // Show the tooltip and tick for the show delay (default is 0)

src/lib/tooltip/tooltip.ts

@@ -32,14 +32,15 @@ import {Subject} from 'rxjs/Subject';
 import {Dir} from '../core/rtl/dir';
 import 'rxjs/add/operator/first';
 import {ScrollDispatcher} from '../core/overlay/scroll/scroll-dispatcher';
+import {Subscription} from 'rxjs';
 
 export type TooltipPosition = 'left' | 'right' | 'above' | 'below' | 'before' | 'after';
 
 /** Time in ms to delay before changing the tooltip visibility to hidden */
 export const TOUCHEND_HIDE_DELAY  = 1500;
 
 /** Time in ms to throttle repositioning after scroll events. */
-export const SCROLL_THROTTLE_MS = 0;
+export const SCROLL_THROTTLE_MS = 20;
 
 /**
  * Directive that attaches a material design tooltip to the host element. Animates the showing and
@@ -60,6 +61,7 @@ export const SCROLL_THROTTLE_MS = 0;
 export class MdTooltip implements OnInit, OnDestroy {
   _overlayRef: OverlayRef;
   _tooltipInstance: TooltipComponent;
+  scrollSubscription: Subscription;
 
   private _position: TooltipPosition = 'below';
 
@@ -135,7 +137,7 @@ export class MdTooltip implements OnInit, OnDestroy {
   ngOnInit() {
     // When a scroll on the page occurs, update the position in case this tooltip needs		
     // to be repositioned.
-    this._scrollDispatcher.scrolled(SCROLL_THROTTLE_MS).subscribe(() => {
+    this.scrollSubscription = this._scrollDispatcher.scrolled(SCROLL_THROTTLE_MS).subscribe(() => {
       if (this._overlayRef) {
         this._overlayRef.updatePosition();
       }
@@ -149,6 +151,8 @@ export class MdTooltip implements OnInit, OnDestroy {
     if (this._tooltipInstance) {
       this._disposeTooltip();
     }
+
+    this.scrollSubscription.unsubscribe();
   }
 
   /** Shows the tooltip after the delay in ms, defaults to tooltip-delay-show or 0ms if no input */