refactor(overlay): inconsistent return type for FlexibleConnectedPositionStrategy.withScrollableContainers (#15610)

Fixes `FlexibleConnectedPositionStrategy.withScrollableContainers` returning `void`, whereas all other `with*` public methods are returning `this`.

Fixes #15607.

Author: crisbeto

src/cdk/overlay/position/flexible-connected-position-strategy.ts

@@ -304,15 +304,15 @@ export class FlexibleConnectedPositionStrategy implements PositionStrategy {
     this._applyPosition(fallback!.position, fallback!.originPoint);
   }
 
-  detach() {
+  detach(): void {
     this._clearPanelClasses();
     this._lastPosition = null;
     this._previousPushAmount = null;
     this._resizeSubscription.unsubscribe();
   }
 
   /** Cleanup after the element gets destroyed. */
-  dispose() {
+  dispose(): void {
     if (this._isDisposed) {
       return;
     }
@@ -369,8 +369,9 @@ export class FlexibleConnectedPositionStrategy implements PositionStrategy {
    * on reposition we can evaluate if it or the overlay has been clipped or outside view. Every
    * Scrollable must be an ancestor element of the strategy's origin element.
    */
-  withScrollableContainers(scrollables: CdkScrollable[]) {
+  withScrollableContainers(scrollables: CdkScrollable[]): this {
     this.scrollables = scrollables;
+    return this;
   }
 
   /**

src/lib/tooltip/tooltip.ts

@@ -339,17 +339,16 @@ export class MatTooltip implements OnDestroy {
       return this._overlayRef;
     }
 
+    const scrollableAncestors =
+        this._scrollDispatcher.getAncestorScrollContainers(this._elementRef);
+
     // Create connected position strategy that listens for scroll events to reposition.
     const strategy = this._overlay.position()
-      .flexibleConnectedTo(this._elementRef)
-      .withTransformOriginOn('.mat-tooltip')
-      .withFlexibleDimensions(false)
-      .withViewportMargin(8);
-
-    const scrollableAncestors = this._scrollDispatcher
-      .getAncestorScrollContainers(this._elementRef);
-
-    strategy.withScrollableContainers(scrollableAncestors);
+                         .flexibleConnectedTo(this._elementRef)
+                         .withTransformOriginOn('.mat-tooltip')
+                         .withFlexibleDimensions(false)
+                         .withViewportMargin(8)
+                         .withScrollableContainers(scrollableAncestors);
 
     strategy.positionChanges.pipe(takeUntil(this._destroyed)).subscribe(change => {
       if (this._tooltipInstance) {

tools/public_api_guard/cdk/overlay.d.ts

@@ -122,7 +122,7 @@ export declare class FlexibleConnectedPositionStrategy implements PositionStrate
     withLockedPosition(isLocked?: boolean): this;
     withPositions(positions: ConnectedPosition[]): this;
     withPush(canPush?: boolean): this;
-    withScrollableContainers(scrollables: CdkScrollable[]): void;
+    withScrollableContainers(scrollables: CdkScrollable[]): this;
     withTransformOriginOn(selector: string): this;
     withViewportMargin(margin: number): this;
 }