fix(service-worker): fix improper call of Observable.merge (#19962)

alxhub · matsko · commit 14016c781f09 · 2017-10-26T18:16:20.000-04:00
Observable.merge was called using .call() as if it were an operator and not an Observable factory. This removes the .call() and uses the factory properly. PR Close #19962
diff --git a/packages/service-worker/src/push.ts b/packages/service-worker/src/push.ts
@@ -43,7 +43,7 @@ export class SwPush {
 
     const workerDrivenSubscriptions = <Observable<PushSubscription|null>>(op_switchMap.call(
         this.pushManager, (pm: PushManager) => pm.getSubscription().then(sub => { return sub; })));
-    this.subscription = obs_merge.call(workerDrivenSubscriptions, this.subscriptionChanges);
+    this.subscription = obs_merge(workerDrivenSubscriptions, this.subscriptionChanges);
   }
 
   requestSubscription(options: {serverPublicKey: string}): Promise<PushSubscription> {

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ export class SwPush {`
`43`	`43`
`44`	`44`	`const workerDrivenSubscriptions = <Observable<PushSubscription\|null>>(op_switchMap.call(`
`45`	`45`	`this.pushManager, (pm: PushManager) => pm.getSubscription().then(sub => { return sub; })));`
`46`		`- this.subscription = obs_merge.call(workerDrivenSubscriptions, this.subscriptionChanges);`
	`46`	`+ this.subscription = obs_merge(workerDrivenSubscriptions, this.subscriptionChanges);`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`requestSubscription(options: {serverPublicKey: string}): Promise<PushSubscription> {`